Bug 33848 - Horizontal scrollbar works in reverse at milliondollarcu.be
Summary: Horizontal scrollbar works in reverse at milliondollarcu.be
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Windows XP
: P2 Normal
Assignee: Nobody
URL: http://milliondollarcu.be/
Keywords: InRadar, PlatformOnly
Depends on:
Blocks:
 
Reported: 2010-01-19 09:15 PST by Adam Roben (:aroben)
Modified: 2010-02-05 14:41 PST (History)
3 users (show)

See Also:


Attachments
Patch (2.89 KB, patch)
2010-02-05 14:16 PST, Enrica Casucci
simon.fraser: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Roben (:aroben) 2010-01-19 09:15:21 PST
To reproduce:

1. Go to http://milliondollarcu.be/
2. Spoof as Safari Mac (to get the page to kick into 3D mode)
3. Resize the window so there's a horizontal scrollbar
4. Scroll using the horizontal scrollbar

When you move the horizontal scrollbar, the page scrolls in the wrong direction.
Comment 1 Adam Roben (:aroben) 2010-01-19 09:15:46 PST
<rdar://problem/7556121>
Comment 2 Enrica Casucci 2010-02-05 14:16:48 PST
Created attachment 48259 [details]
Patch
Comment 3 WebKit Review Bot 2010-02-05 14:23:31 PST
Attachment 48259 [details] did not pass style-queue:

Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
WebCore/platform/graphics/win/WKCACFLayerRenderer.cpp:159:  Tab found; better to use spaces  [whitespace/tab] [1]
Total errors found: 1


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 4 Simon Fraser (smfr) 2010-02-05 14:30:03 PST
Comment on attachment 48259 [details]
Patch

r=me but you have to remove the tabs before committing.
Comment 5 Enrica Casucci 2010-02-05 14:34:01 PST
(In reply to comment #4)
> (From update of attachment 48259 [details])
> r=me but you have to remove the tabs before committing.

Sorry, I forgot to setup Visual studio correctly.
I'll do that.
Comment 6 Enrica Casucci 2010-02-05 14:41:52 PST
Committed revision 54446.