Bug 107930

Summary: [BlackBerry] AC layers appear in the wrong place on RTL page
Product: WebKit Reporter: Arvid Nilsson <anilsson>
Component: WebKit BlackBerryAssignee: Arvid Nilsson <anilsson>
Status: RESOLVED FIXED    
Severity: Normal CC: anilsson, mifenton, rwlbuis, staikos, tonikitoo, webkit.review.bot, yong.li.webkit
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

Arvid Nilsson
Reported 2013-01-25 02:48:46 PST
PR #280229 The public API of the BlackBerry port always reports a minimum scroll position of (0, 0), even on RTL pages with left overflow. We accomplish this by translating the WebCore scroll position by an amount equal in size to the minimum scroll position, to obtain the API scroll position reported to the API client. This means the API client will ask us to render a rect that needs to be corrected for the minimum scroll position, or we'll render the wrong parts. This is done for BackingStore, but not for WebPageCompositor.
Attachments
Patch (9.66 KB, patch)
2013-01-25 03:06 PST, Arvid Nilsson 		no flags
DetailsFormatted DiffDiff
Patch (9.66 KB, patch)
2013-01-25 04:34 PST, Arvid Nilsson 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Arvid Nilsson
Comment 1 2013-01-25 03:06:24 PST
Created attachment 184713 [details] Patch
Arvid Nilsson
Comment 2 2013-01-25 04:31:38 PST
Since George is the internal reviewer, and this patch is identical to the internal patch, I'll mark him as upstream reviewer, too.
Arvid Nilsson
Comment 3 2013-01-25 04:34:50 PST
Created attachment 184725 [details] Patch
WebKit Review Bot
Comment 4 2013-01-25 05:06:35 PST
Comment on attachment 184725 [details] Patch Clearing flags on attachment: 184725 Committed r140816: <http://trac.webkit.org/changeset/140816>
WebKit Review Bot
Comment 5 2013-01-25 05:06:39 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.