The HTML parser should lazy attach.

The HTML parser should lazy attach.

The rebaselines are one of the following:
-empty whitespace text node changes
-changes in anonymous wrapper blocks that don't affect the resulting layout
-existing race conditions that are exposed by lazy attaching. These are the ones we've deemed OK to land.

Some detailed explanations:
-fast/dom/tabindex-defaults.html: FAIL is because of FIXME inside HTMLElement::supportsFocus crbug.com/285335
-svg/custom/getsvgdocument.html: crbug.com/284734

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

[eseidel, 2013-09-04 18:39:14]

lgtm

I'm comfortable with these changes.

https://codereview.chromium.org/23475019/diff/1/Source/core/html/parser/HTMLC...
File Source/core/html/parser/HTMLConstructionSite.cpp (left):

https://codereview.chromium.org/23475019/diff/1/Source/core/html/parser/HTMLC...
Source/core/html/parser/HTMLConstructionSite.cpp:108: insert(task,
DeprecatedAttachNow);
I assume you meant to include this? :)

[ojan, 2013-09-04 18:43:59]

https://codereview.chromium.org/23475019/diff/1/Source/core/html/parser/HTMLC...
File Source/core/html/parser/HTMLConstructionSite.cpp (left):

https://codereview.chromium.org/23475019/diff/1/Source/core/html/parser/HTMLC...
Source/core/html/parser/HTMLConstructionSite.cpp:108: insert(task,
DeprecatedAttachNow);
On 2013/09/04 18:39:14, eseidel wrote:
> I assume you meant to include this? :)

This is *the* change. Stupid rietveld doesn't update the subject of the issue
when you change the change description.

[commit-bot: I haz the power, 2013-09-04 18:44:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ojan@chromium.org/23475019/1

[commit-bot: I haz the power, 2013-09-04 19:28:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ojan@chromium.org/23475019/1

[commit-bot: I haz the power, 2013-09-04 20:41:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ojan@chromium.org/23475019/18001

[commit-bot: I haz the power, 2013-09-04 22:47:06]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ojan@chromium.org/23475019/31001

[commit-bot: I haz the power, 2013-09-04 23:15:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ojan@chromium.org/23475019/37001

[commit-bot: I haz the power, 2013-09-05 02:22:11]

Retried try job too often on linux_blink_rel for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_blin...

[commit-bot: I haz the power, 2013-09-05 18:34:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ojan@chromium.org/23475019/47001

[commit-bot: I haz the power, 2013-09-05 20:09:50]

Change committed as 157319