Title: [88288] trunk
- Revision
- 88288
- Author
- [email protected]
- Date
- 2011-06-07 16:59:14 -0700 (Tue, 07 Jun 2011)
Log Message
2011-06-07 Vangelis Kokkevis <[email protected]>
Reviewed by James Robinson.
Add a test to verify that RTL pages that trigger the compositor
render correctly.
https://bugs.webkit.org/show_bug.cgi?id=62246
* compositing/geometry/rtl-composited-expected.png: Added.
* compositing/geometry/rtl-composited-expected.txt: Added.
* compositing/geometry/rtl-composited.html: Added.
2011-06-07 Vangelis Kokkevis <[email protected]>
Reviewed by James Robinson.
Disable compositing for RTL pages in the RenderLayerCompositor rather
than later on in the WebViewImpl.
https://bugs.webkit.org/show_bug.cgi?id=62246
* src/ChromeClientImpl.cpp:
(WebKit::ChromeClientImpl::allowedCompositingTriggers):
* src/WebViewImpl.cpp:
(WebKit::WebViewImpl::layout):
(WebKit::WebViewImpl::setRootGraphicsLayer):
Modified Paths
Added Paths
Diff
Modified: trunk/LayoutTests/ChangeLog (88287 => 88288)
--- trunk/LayoutTests/ChangeLog 2011-06-07 23:58:19 UTC (rev 88287)
+++ trunk/LayoutTests/ChangeLog 2011-06-07 23:59:14 UTC (rev 88288)
@@ -1,3 +1,15 @@
+2011-06-07 Vangelis Kokkevis <[email protected]>
+
+ Reviewed by James Robinson.
+
+ Add a test to verify that RTL pages that trigger the compositor
+ render correctly.
+ https://bugs.webkit.org/show_bug.cgi?id=62246
+
+ * compositing/geometry/rtl-composited-expected.png: Added.
+ * compositing/geometry/rtl-composited-expected.txt: Added.
+ * compositing/geometry/rtl-composited.html: Added.
+
2011-06-07 Ryosuke Niwa <[email protected]>
Gtk rebaseline after r88266.
Added: trunk/LayoutTests/compositing/geometry/rtl-composited-expected.png
(Binary files differ)
Added: svn:mime-type
Added: trunk/LayoutTests/compositing/geometry/rtl-composited-expected.txt (0 => 88288)
--- trunk/LayoutTests/compositing/geometry/rtl-composited-expected.txt (rev 0)
+++ trunk/LayoutTests/compositing/geometry/rtl-composited-expected.txt 2011-06-07 23:59:14 UTC (rev 88288)
@@ -0,0 +1 @@
+
Added: trunk/LayoutTests/compositing/geometry/rtl-composited.html (0 => 88288)
--- trunk/LayoutTests/compositing/geometry/rtl-composited.html (rev 0)
+++ trunk/LayoutTests/compositing/geometry/rtl-composited.html 2011-06-07 23:59:14 UTC (rev 88288)
@@ -0,0 +1,32 @@
+<!DOCTYPE>
+<html>
+<head>
+ <title>RTL page with composited elements</title>
+
+<style type="text/css" media="screen">
+body {
+ direction: rtl;
+}
+#wrapper {
+ position: relative;
+ background: green;
+ width: 200px;
+ height: 200px;
+ z-index: 5;
+}
+.accelerated {
+ -webkit-transform: translate3d(0,0,0);
+}
+</style>
+<script type="text/_javascript_" charset="utf-8">
+ if (window.layoutTestController)
+ layoutTestController.dumpAsText(true);
+</script>
+</head>
+<body>
+<body>
+<!-- There should be a green box at the top right corner of the window -->
+ <div id="wrapper"></div>
+ <div class="accelerated"></div>
+</body>
+</html>
\ No newline at end of file
Modified: trunk/Source/WebKit/chromium/ChangeLog (88287 => 88288)
--- trunk/Source/WebKit/chromium/ChangeLog 2011-06-07 23:58:19 UTC (rev 88287)
+++ trunk/Source/WebKit/chromium/ChangeLog 2011-06-07 23:59:14 UTC (rev 88288)
@@ -1,3 +1,17 @@
+2011-06-07 Vangelis Kokkevis <[email protected]>
+
+ Reviewed by James Robinson.
+
+ Disable compositing for RTL pages in the RenderLayerCompositor rather
+ than later on in the WebViewImpl.
+ https://bugs.webkit.org/show_bug.cgi?id=62246
+
+ * src/ChromeClientImpl.cpp:
+ (WebKit::ChromeClientImpl::allowedCompositingTriggers):
+ * src/WebViewImpl.cpp:
+ (WebKit::WebViewImpl::layout):
+ (WebKit::WebViewImpl::setRootGraphicsLayer):
+
2011-06-07 Sheriff Bot <[email protected]>
Unreviewed, rolling out r88254, r88262, and r88268.
Modified: trunk/Source/WebKit/chromium/src/ChromeClientImpl.cpp (88287 => 88288)
--- trunk/Source/WebKit/chromium/src/ChromeClientImpl.cpp 2011-06-07 23:58:19 UTC (rev 88287)
+++ trunk/Source/WebKit/chromium/src/ChromeClientImpl.cpp 2011-06-07 23:59:14 UTC (rev 88288)
@@ -846,7 +846,8 @@
ChromeClient::CompositingTriggerFlags ChromeClientImpl::allowedCompositingTriggers() const
{
- if (!m_webView->allowsAcceleratedCompositing())
+ // FIXME: RTL style not supported by the compositor yet.
+ if (!m_webView->allowsAcceleratedCompositing() || m_webView->pageHasRTLStyle())
return 0;
CompositingTriggerFlags flags = 0;
Modified: trunk/Source/WebKit/chromium/src/WebViewImpl.cpp (88287 => 88288)
--- trunk/Source/WebKit/chromium/src/WebViewImpl.cpp 2011-06-07 23:58:19 UTC (rev 88287)
+++ trunk/Source/WebKit/chromium/src/WebViewImpl.cpp 2011-06-07 23:59:14 UTC (rev 88288)
@@ -1036,11 +1036,6 @@
void WebViewImpl::layout()
{
TRACE_EVENT("WebViewImpl::layout", this, 0);
-#if USE(ACCELERATED_COMPOSITING)
- // FIXME: RTL style not supported by the compositor yet.
- if (isAcceleratedCompositingActive() && pageHasRTLStyle())
- setIsAcceleratedCompositingActive(false);
-#endif
WebFrameImpl* webframe = mainFrameImpl();
if (webframe) {
@@ -2379,8 +2374,7 @@
void WebViewImpl::setRootGraphicsLayer(WebCore::PlatformLayer* layer)
{
- // FIXME: RTL style not supported by the compositor yet.
- setIsAcceleratedCompositingActive(layer && !pageHasRTLStyle() ? true : false);
+ setIsAcceleratedCompositingActive(layer);
if (m_layerRenderer)
m_layerRenderer->setRootLayer(layer);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
