Title: [109965] trunk/Source/WebCore
Revision
109965
Author
shawnsi...@chromium.org
Date
2012-03-06 15:09:37 -0800 (Tue, 06 Mar 2012)

Log Message

[chromium] Make compositeAndReadback and damage tracking play nicely together
https://bugs.webkit.org/show_bug.cgi?id=80199

Reviewed by James Robinson.

Requires system-level testing (pixels on front-buffer should be
observed for correct behavior) that cannot be done by layout tests
or unit tests, have to rely on manual testing.

* platform/graphics/chromium/cc/CCSingleThreadProxy.cpp:
(WebCore::CCSingleThreadProxy::compositeAndReadback):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (109964 => 109965)


--- trunk/Source/WebCore/ChangeLog	2012-03-06 23:06:51 UTC (rev 109964)
+++ trunk/Source/WebCore/ChangeLog	2012-03-06 23:09:37 UTC (rev 109965)
@@ -1,3 +1,17 @@
+2012-03-06  Shawn Singh  <shawnsi...@chromium.org>
+
+        [chromium] Make compositeAndReadback and damage tracking play nicely together
+        https://bugs.webkit.org/show_bug.cgi?id=80199
+
+        Reviewed by James Robinson.
+
+        Requires system-level testing (pixels on front-buffer should be
+        observed for correct behavior) that cannot be done by layout tests
+        or unit tests, have to rely on manual testing.
+
+        * platform/graphics/chromium/cc/CCSingleThreadProxy.cpp:
+        (WebCore::CCSingleThreadProxy::compositeAndReadback):
+
 2012-03-06  Luke Macpherson   <macpher...@chromium.org>
 
         Make CSSStyleSelector::convertToLength() behave more like CSSPrimitiveValue::convertToLength().

Modified: trunk/Source/WebCore/platform/graphics/chromium/cc/CCSingleThreadProxy.cpp (109964 => 109965)


--- trunk/Source/WebCore/platform/graphics/chromium/cc/CCSingleThreadProxy.cpp	2012-03-06 23:06:51 UTC (rev 109964)
+++ trunk/Source/WebCore/platform/graphics/chromium/cc/CCSingleThreadProxy.cpp	2012-03-06 23:09:37 UTC (rev 109965)
@@ -80,11 +80,12 @@
 
     m_layerTreeHostImpl->readback(pixels, rect);
 
-    didSwapFrame();
-
     if (m_layerTreeHostImpl->isContextLost())
         return false;
 
+    m_layerTreeHostImpl->swapBuffers();
+    didSwapFrame();
+
     return true;
 }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to