Improve document.title behavior; cleanup.

* Implement HTML5-conformant behavior for document.title if there is no
  <title> element.
* Remove attribute/property "blacklist" code from TameIFrameElement
  which is no longer necessary since the whitelist is per-attribute.
* Move misplaced @param comment for taming membrane parameter.

@r5169

[felix8a, 2012-11-29 06:39:24]

https://codereview.appspot.com/6843122/diff/1/src/com/google/caja/plugin/doma...
File src/com/google/caja/plugin/domado.js (right):

https://codereview.appspot.com/6843122/diff/1/src/com/google/caja/plugin/doma...
src/com/google/caja/plugin/domado.js:4684: if (attrLc !== 'name' && attrLc !==
'src') {
doesn't this special case go away too?

https://codereview.appspot.com/6843122/diff/1/src/com/google/caja/plugin/doma...
src/com/google/caja/plugin/domado.js:4693: // ...
how about add a TODO here?

[kpreid2, 2012-11-29 20:02:45]

* Implement HTML5-conformant behavior for document.title if there is no
  <title> element.
* Remove attribute/property "blacklist" code from TameIFrameElement
  which is no longer necessary since the whitelist is per-attribute.
* Move misplaced @param comment for taming membrane parameter.

[kpreid2, 2012-11-29 20:03:08]

New snapshot. Note changes to testIframeShim.

https://codereview.appspot.com/6843122/diff/1/src/com/google/caja/plugin/doma...
File src/com/google/caja/plugin/domado.js (right):

https://codereview.appspot.com/6843122/diff/1/src/com/google/caja/plugin/doma...
src/com/google/caja/plugin/domado.js:4684: if (attrLc !== 'name' && attrLc !==
'src') {
On 2012/11/29 06:39:24, felix8a wrote:
> doesn't this special case go away too?

Yep, it should. Removed the method entirely.

https://codereview.appspot.com/6843122/diff/1/src/com/google/caja/plugin/doma...
src/com/google/caja/plugin/domado.js:4693: // ...
On 2012/11/29 06:39:24, felix8a wrote:
> how about add a TODO here?

Removed the method entirely, since it does nothing. This was an artifact of
these changes having been originally written as preliminary to implementing
iframe src=, but I can always re-add this setAttribute hook then.

[felix8a, 2012-11-29 20:10:42]

lgtm