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>
Severity: Normal CC: anilsson, mifenton, rwlbuis, staikos, tonikitoo, webkit.review.bot, yong.li.webkit
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Description Flags
Patch none

Description Arvid Nilsson 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.
Comment 1 Arvid Nilsson 2013-01-25 03:06:24 PST
Created attachment 184713 [details]
Comment 2 Arvid Nilsson 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.
Comment 3 Arvid Nilsson 2013-01-25 04:34:50 PST
Created attachment 184725 [details]
Comment 4 WebKit Review Bot 2013-01-25 05:06:35 PST
Comment on attachment 184725 [details]

Clearing flags on attachment: 184725

Committed r140816: <http://trac.webkit.org/changeset/140816>
Comment 5 WebKit Review Bot 2013-01-25 05:06:39 PST
All reviewed patches have been landed.  Closing bug.