[WinCairo][AcceleratedCompositing] Rendering issues on www.bbc.com.
authorpeavo@outlook.com <peavo@outlook.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 7 Mar 2016 18:51:45 +0000 (18:51 +0000)
committerpeavo@outlook.com <peavo@outlook.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 7 Mar 2016 18:51:45 +0000 (18:51 +0000)
https://bugs.webkit.org/show_bug.cgi?id=154912

Reviewed by Michael Catanzaro.

If we have a scroll layer, we should not include the scroll position in the visible rectangle,
since the scroll position is already reflected in the position of the scroll layer.

* WebCoreSupport/AcceleratedCompositingContext.cpp:
(AcceleratedCompositingContext::flushPendingLayerChanges):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@197693 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit/win/ChangeLog
Source/WebKit/win/WebCoreSupport/AcceleratedCompositingContext.cpp

index 7f51cb7..a7d50f2 100644 (file)
@@ -1,3 +1,16 @@
+2016-03-07  Per Arne Vollan  <peavo@outlook.com>
+
+        [WinCairo][AcceleratedCompositing] Rendering issues on www.bbc.com.
+        https://bugs.webkit.org/show_bug.cgi?id=154912
+
+        Reviewed by Michael Catanzaro.
+
+        If we have a scroll layer, we should not include the scroll position in the visible rectangle,
+        since the scroll position is already reflected in the position of the scroll layer.
+
+        * WebCoreSupport/AcceleratedCompositingContext.cpp:
+        (AcceleratedCompositingContext::flushPendingLayerChanges):
+
 2016-03-06  Gavin Barraclough  <barraclough@apple.com>
 
         Convert DOMTimer to std::chrono::milliseconds
index 81701e9..e08215b 100644 (file)
@@ -361,7 +361,9 @@ bool AcceleratedCompositingContext::flushPendingLayerChanges()
     if (!frameView->flushCompositingStateIncludingSubframes())
         return false;
 
-    FloatRect visibleRect(frameView->scrollPosition(), getWebViewSize(m_webView));
+    // On WinCairo, we always have a scroll layer, and should not include the scroll position in the visible rectangle,
+    // since the scroll position is already reflected in the position of the scroll layer.
+    FloatRect visibleRect(FloatPoint(0, 0), getWebViewSize(m_webView));
     downcast<GraphicsLayerTextureMapper>(*m_rootLayer).updateBackingStoreIncludingSubLayers(visibleRect);
     return true;
 }