Bug 130474 - REGRESSION(r156316): Scrollbar on a large document is broken
Summary: REGRESSION(r156316): Scrollbar on a large document is broken
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ryosuke Niwa
URL: http://www.whatwg.org/specs/web-apps/...
Keywords: InRadar, Regression
Depends on:
Blocks:
 
Reported: 2014-03-19 14:15 PDT by Ryosuke Niwa
Modified: 2014-03-19 14:53 PDT (History)
7 users (show)

See Also:

Attachments
Fixes the bug (2.47 KB, patch)
2014-03-19 14:32 PDT, Ryosuke Niwa 		andersca: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ryosuke Niwa 2014-03-19 14:15:57 PDT 
Reproduction steps
1. Go to http://www.whatwg.org/specs/web-apps/current-work/multipage/
2. Scroll all the way down to the bottom via wheel/trackpad
3. Scroll up so that the scollbar is above "Help" button via wheel/trackpad.
4. Try to drag the scrollbar.

Expected result:
Scrollbar moves to the direction I dragged

Actual result:
Scrollbar always moves downwards.
Comment 1 Radar WebKit Bug Importer 2014-03-19 14:16:39 PDT 
<rdar://problem/16371523>
Comment 2 Ryosuke Niwa 2014-03-19 14:29:48 PDT 
The problem appears that updateArrowPlacement is no longer exiting early in ToT:
http://trac.webkit.org/changeset/156316/trunk/Source/WebCore/platform/mac/ScrollbarThemeMac.mm

Since isScrollbarOverlayAPIAvailable() is always true, we should be returning there.
Comment 3 Ryosuke Niwa 2014-03-19 14:32:40 PDT 
Created attachment 227219 [details]
Fixes the bug
Comment 4 Ryosuke Niwa 2014-03-19 14:37:46 PDT 
Committed r165918: <http://trac.webkit.org/changeset/165918>