[webkit-reviews] review requested: [Bug 65668] Optimize floating elements lookup : [Attachment 103242] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Aug 8 07:26:41 PDT 2011


Chiculita Alexandru <achicu at adobe.com> has asked  for review:
Bug 65668: Optimize floating elements lookup
https://bugs.webkit.org/show_bug.cgi?id=65668

Attachment 103242: Patch
https://bugs.webkit.org/attachment.cgi?id=103242&action=review

------- Additional Comments from Chiculita Alexandru <achicu at adobe.com>
Updated the floats test to also create some empty nested divs. It seems that
the lazy init optimization makes a big difference. Thanks for the tip.

Also addressed the other comments.

Regarding "You could move setIsPlaced inside addPlacedObject" comment. I think
it's ok to move it, even if I changed to lazy init now.


More information about the webkit-reviews mailing list