[webkit-reviews] review denied: [Bug 33520] Mapping from local to container coord space is O(N^2) : [Attachment 50531] fix ChangeLog

[bugzilla-daemon at webkit.org]

Dave Hyatt <hyatt at apple.com> has denied James Robinson <jamesr at chromium.org>'s
request for review:
Bug 33520: Mapping from local to container coord space is O(N^2)
https://bugs.webkit.org/show_bug.cgi?id=33520

Attachment 50531: fix ChangeLog
https://bugs.webkit.org/attachment.cgi?id=50531&action=review

------- Additional Comments from Dave Hyatt <hyatt at apple.com>
The logic for disabling the cache is wrong...

<div style="position:absolute;">  .... <div style="overflow:auto">.... <div
style="position:absolute;left:100;top:100">

You'd have the cache enabled and add in the intermediate x/y of the overflow
section, but that isn't relevant.

I think a simple cache will have to check containingBlock of the child layer's
renderer to make sure it matches the parent layer's renderer in order to do a
cache like this.