[Webkit-unassigned] [Bug 106963] Move TreeScope pointer to ContainerNode

[bugzilla-daemon at webkit.org]

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





--- Comment #11 from Antti Koivisto <koivisto at iki.fi>  2013-01-16 03:32:48 PST ---
(In reply to comment #6)
> I object to this change, it puts a branch into all the methods that go through the document pointer in the render tree (see RenderObject.h) which we know is a perf regression from the last time moritta tried replacing the document pointer. This also seems like a lot of complexity.

There is no branch if you operate on ContainerNodes. Looks like the new branchiness in the render tree could be largely eliminated by simple tightening (non-RenderTexts always have Element as their node())

Note that significant memory reductions can be performance progressions too in by themselves due to improved locality. I don't know how this is going to be without testing.

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