Bug 88887 - [chromium] Fix incorrect LayerChromium scroll position for RTL overflow pages
Summary: [chromium] Fix incorrect LayerChromium scroll position for RTL overflow pages
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Adrienne Walker
URL:
Keywords:
Depends on:
Blocks: 88145
  Show dependency tree / graph
 
Reported: 2012-06-12 10:27 PDT by Adrienne Walker
Modified: 2012-06-12 12:55 PDT (History)
3 users (show)

See Also:

Attachments
Patch (7.11 KB, patch)
2012-06-12 10:32 PDT, Adrienne Walker 		no flags 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 Adrienne Walker 2012-06-12 10:27:29 PDT 
[chromium] Fix incorrect LayerChromium scroll position for RTL overflow pages
Comment 1 Adrienne Walker 2012-06-12 10:32:23 PDT 
Created attachment 147111 [details]
Patch
Comment 2 James Robinson 2012-06-12 10:34:54 PDT 
Comment on attachment 147111 [details]
Patch

R=me

seems pretty testable though - make an RTL page with scrollbars and x-overflow, scroll it to the left a bit, dump pixels?
Comment 3 Adrienne Walker 2012-06-12 10:39:20 PDT 
This is the same as bug 88882: the problem only when you are using ScrollbarLayerChromium.

My latest patch in bug 88145 will turn on ScrollbarLayerChromium even when using single-threaded mode, so we will get the benefit of layout tests hitting this code.
Comment 4 WebKit Review Bot 2012-06-12 12:55:46 PDT 
Comment on attachment 147111 [details]
Patch

Clearing flags on attachment: 147111

Committed r120105: <http://trac.webkit.org/changeset/120105>
Comment 5 WebKit Review Bot 2012-06-12 12:55:50 PDT 
All reviewed patches have been landed.  Closing bug.