[Webkit-unassigned] [Bug 136433] REGRESSION (r172832): Poor 2-finger scrolling performance at theverge.com articles (all tiles repaint)

[bugzilla-daemon at webkit.org]

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


Tim Horton <thorton at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #237607|review?                     |review+
               Flag|                            |




--- Comment #8 from Tim Horton <thorton at apple.com>  2014-09-04 13:43:33 PST ---
(From update of attachment 237607)
View in context: https://bugs.webkit.org/attachment.cgi?id=237607&action=review

> Source/WebCore/rendering/RenderLayer.cpp:6401
> +    bool needsHorizontalScrollbar = renderer().hasOverflowClip() && ((hasHorizontalScrollbar() && overflowDefinesAutomaticScrollbar(overflowX)) || overflowRequiresScrollbar(overflowX));

we already have our RenderBox in a local, perhaps use that instead?

>> LayoutTests/platform/mac-wk2/tiled-drawing/scrolling/root-overflow-with-mousewheel.html:35
>> +    setTimeout(checkForScroll, 500);
> 
> Oh, I can get rid of this setTimeout completely, so that's good.

Please do! 500ms tests are unwelcome :)

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