Bug 56591 – [chromium] Support RTL pages and elements in the compositor

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Adrienne Walker]

Due to ScrollView::visibleContentRect having different semantics on RTL pages than on LTR ones (width of the visibleContentRect goes towards negative x instead of towards positive x), RTL pages don't render properly when accelerated compositing is turned on in the Chromium compositor.  There are some other assumptions in LayerTilerChromium and in TilingData where pages are only assumed to grow right and down instead of potentially growing left and down instead.

[Adrienne Walker]

Created attachment 99403 [details]
Patch

[James Robinson]

Comment on attachment 99403 [details]
Patch

Clearing review flag since this needs some mergin' love now that the edge AA patch has landed.

[Jungshik Shin]

There's a Chromium-OS bug related to this issue : http://crosbug.com/20666

[Adrienne Walker]

Created attachment 120982 [details]
Patch

[WebKit Review Bot]

Attachment 120982 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/update-webkit']" exit_code: 9

Updating OpenSource
Index mismatch: 3a7674dadd24ecd6a1b023eba203ecf2ed62fc59 != edb861612940a3244431d239dacdbc36317b627c
rereading e1e2538cff7d320c2b7cc22c0b75c9369ce9cdb2
	A	LayoutTests/svg/custom/webkit-transform-crash.html
	A	LayoutTests/svg/custom/webkit-transform-crash-expected.txt
	M	LayoutTests/ChangeLog
	M	Source/WebCore/ChangeLog
	M	Source/WebCore/svg/SVGStyledTransformableElement.cpp
103950 = 8282a1d85ad097ce4064a5896912bdb211b115e6 already exists! Why are we refetching it?
 at /usr/lib/git-core/git-svn line 5210

Died at Tools/Scripts/update-webkit line 158.


If any of these errors are false positives, please file a bug against check-webkit-style.

[Adrienne Walker]

(In reply to comment #5)
> Attachment 120982 [details] did not pass style-queue:
> 
> Failed to run "['Tools/Scripts/update-webkit']" exit_code: 9

I wonder if the git mirror needs to be kicked.  Mine seems stuck at r103949 and these errors are files in r103950.

[James Robinson]

Comment on attachment 120982 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=120982&action=review

Sweet! That's so easy.

> Source/WebKit/chromium/src/NonCompositedContentHost.cpp:138
> +void NonCompositedContentHost::paintContents(const WebCore::GraphicsLayer* graphicsLayer, WebCore::GraphicsContext& context, WebCore::GraphicsLayerPaintingPhase, const WebCore::IntRect& clipRect)

nit: no need to name the graphicsLayer parameter here

[Adrienne Walker]

Committed r103968: <http://trac.webkit.org/changeset/103968>