Reland 160869 "Move innerHTML and outerHTML to Element"

Move innerHTML and outerHTML to Element

Per the DOM Parsing specification, innerHTML and outerHTML should be
available on Element, not HTMLElement. Since HTMLElement and SVGElement
are the only interfaces that inherit from Element, the only Web visible
difference should be that innerHTML and outerHTML are now available for
SVG elements.

For <svg>.innerHTML = "<!-- svg elements here -->" to insert elements
with correct namespace URI, one adjustment has to be made to the tree
builder step of the html parsing algorithm. The changes are described in
the spec change <http://html5.org/r/7768>;. The new concept of
'adjustedCurrentNode' is used when processing and when deciding to
process foreign content and makes sure that parsed elements go into the
correct namespace.

Some layout tests that (perhaps accidentally) use innerHTML and
outerHTML on SVG elements are adjusted slightly.

This is a re-land of 160869 https://codereview.chromium.org/44333002/

BUG=311080
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=160981

[davve, 2013-10-30 14:51:02]

The browser test is being disabled in https://codereview.chromium.org/45733012/

[Mike West, 2013-10-30 20:34:25]

rs=me

I think carrying over abarth's and pdr's LGTM is fine.

Please do copy/paste the description from the original patch, however, don't
just say "relanding X". The fact that this is a reland should just be a sentence
at the end of the description. :)

I'd like to ensure no one has to click through a few revisions/reverts to
determine what the change actually does.

[Mike West, 2013-10-30 20:35:19]

On 2013/10/30 20:34:25, Mike West wrote:
> rs=me
> 
> I think carrying over abarth's and pdr's LGTM is fine.
> 
> Please do copy/paste the description from the original patch, however, don't
> just say "relanding X". The fact that this is a reland should just be a
sentence
> at the end of the description. :)
> 
> I'd like to ensure no one has to click through a few revisions/reverts to
> determine what the change actually does.

(And please link to the original review in the description as well:
https://codereview.chromium.org/44333002)

[davve, 2013-10-30 20:57:21]

On 2013/10/30 20:35:19, Mike West wrote:
> On 2013/10/30 20:34:25, Mike West wrote:
> > rs=me
> > 
> > I think carrying over abarth's and pdr's LGTM is fine.
> > 
> > Please do copy/paste the description from the original patch, however, don't
> > just say "relanding X". The fact that this is a reland should just be a
> sentence
> > at the end of the description. :)
> > 
> > I'd like to ensure no one has to click through a few revisions/reverts to
> > determine what the change actually does.
> 
> (And please link to the original review in the description as well:
> https://codereview.chromium.org/44333002)

Ok, I fully agree with this. I looked at and tried to mimic a couple of others
re-lands, but perhaps misunderstood them or looked at the wrong ones. Will
update the description.

[Mike West, 2013-10-30 21:18:28]

On 2013/10/30 20:57:21, davve wrote:
> Ok, I fully agree with this. I looked at and tried to mimic a couple of others
> re-lands, but perhaps misunderstood them or looked at the wrong ones. Will
> update the description.

If you found relands that just said "reland XXX", they were Doing It Wrong™. :)

Thanks for updating the description, totally LGTM.

[commit-bot: I haz the power, 2013-10-30 21:39:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/davve@opera.com/52443003/1

[commit-bot: I haz the power, 2013-10-30 22:58:02]

Change committed as 160981

[dglazkov, 2013-10-31 01:39:48]

It still broke browsertests. Please revert. Apologies :-\

http://build.chromium.org/p/chromium.webkit/builders/Linux%20Tests/builds/32150

[davve, 2013-10-31 08:14:36]

On 2013/10/31 01:39:48, Dimitri Glazkov wrote:
> It still broke browsertests. Please revert. Apologies :-\
> 
>
http://build.chromium.org/p/chromium.webkit/builders/Linux%2520Tests/builds/3...

Was that build from the one hour window between this change going in and the
test being disabled? This change went in faster than I thought, and
https://codereview.chromium.org/45733012/ took way longer. Apologies for that.

[yurys, 2013-10-31 15:20:06]

On 2013/10/31 08:14:36, davve wrote:
> On 2013/10/31 01:39:48, Dimitri Glazkov wrote:
> > It still broke browsertests. Please revert. Apologies :-\
> > 
> >
>
http://build.chromium.org/p/chromium.webkit/builders/Linux%252520Tests/builds...
> 
> Was that build from the one hour window between this change going in and the
> test being disabled? This change went in faster than I thought, and
> https://codereview.chromium.org/45733012/ took way longer. Apologies for that.

Blink rev has been rolled to r161048 so it should be fine to rebaseline and
enable the test.

[davve, 2013-10-31 15:51:58]

On 2013/10/31 15:20:06, Yury Semikhatsky wrote:

> Blink rev has been rolled to r161048 so it should be fine to rebaseline and
> enable the test.

I've posted https://codereview.chromium.org/54933002/ to address this.