[Webkit-unassigned] [Bug 46761] Remove calls to Document::updateStyleForAllDocuments()

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=46761





--- Comment #12 from Pratik Solanki <psolanki at apple.com>  2011-01-18 17:53:08 PST ---
I thought making the HTML parser use lazyAttach() (bug 47316) would fix my issue. It didn't. This is because we end up with the following (partial) dom tree

*    #text    0x1059da2c0 "PASS"
    BR    0x1059c2c90
    #text    0x113ec25f0 "\n\n"

At this point all 3 nodes are attached via lazyAttach() but none have a renderer. We start by calculating style for the PASS element and we create its renderer. And then we come to Node::attach(). The loop in Node::attach() looks at the nodes siblings and decides to create the renderer for the empty text node. And since the BR node has not been created (i.e. its renderer is NULL), the optimization in Text::rendererIsNeeded() eludes us again. The order for creating renderers is 

- PASS text node
- empty text node
- BR node

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.