Bug 122794 – The HTML parser should lazy attach

• Status: RESOLVED
• Resolution: CONFIGURATION CHANGED
• Component: Layout and Rendering

[Ryosuke Niwa]

Consider merging https://chromium.googlesource.com/chromium/blink/+/318b658082e57daf411681a088d3b0e725dca07d
or come up with a better fix.

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

[Ryosuke Niwa]

Also see https://chromium.googlesource.com/chromium/blink/+/60b3cb1dbf3c2f471e7fb450cc1072025db52326

[Ryosuke Niwa]

(In reply to comment #1)
> Also see https://chromium.googlesource.com/chromium/blink/+/60b3cb1dbf3c2f471e7fb450cc1072025db52326

Was reverted in https://chromium.googlesource.com/chromium/blink/+/1f4794b89a3ec4c799b4b4d7a70b308746aff2cc.

[Ahmad Saleem]

Already done - https://github.com/WebKit/WebKit/blob/cc6daf68c1caefd70cb363b0a02af7debb465a93/Source/WebCore/html/parser/HTMLConstructionSite.cpp#L147

More changed than this patch - https://github.com/WebKit/WebKit/blob/424de6f2e3e3d2b3a8ee9f50bd34f9caa3217edd/Source/WebCore/dom/CharacterData.cpp#L205

[e-mail] - Anything else required? Thanks!

[Ryosuke Niwa]

We don't need to merge this now.