Bug 33848

Summary: Horizontal scrollbar works in reverse at milliondollarcu.be
Product: WebKit Reporter: Adam Roben (:aroben) <aroben>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: cmarrin, enrica, webkit.review.bot
Priority: P2 Keywords: InRadar, PlatformOnly
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Windows XP   
URL: http://milliondollarcu.be/
Attachments:
Description Flags
Patch simon.fraser: review+

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.