[webkit-reviews] review denied: [Bug 6796] Reproducible CSS Crasher (infinite recursion adding/removing scrollbar) : [Attachment 7001] First attempt at halting infinte recursion

bugzilla-request-daemon at opendarwin.org bugzilla-request-daemon at opendarwin.org
Mon Mar 13 14:20:51 PST 2006


Dave Hyatt <hyatt at apple.com> has denied Dave Hyatt <hyatt at apple.com>'s request
for review:
Bug 6796: Reproducible CSS Crasher (infinite recursion adding/removing
scrollbar)
http://bugzilla.opendarwin.org/show_bug.cgi?id=6796

Attachment 7001: First attempt at halting infinte recursion
http://bugzilla.opendarwin.org/attachment.cgi?id=7001&action=edit

------- Additional Comments from Dave Hyatt <hyatt at apple.com>
Move the variables into RenderLayer and try to keep as much of the scrolling
logic there as you can.

I'd try to make the min/max width modification only be done if the infinite
recursion case was hit.  I think it would also be fine to simply not have this
code in an initial check-in as well.



More information about the webkit-reviews mailing list