Title: [129116] trunk/Source/WebKit2
Revision
129116
Author
[email protected]
Date
2012-09-20 04:14:10 -0700 (Thu, 20 Sep 2012)

Log Message

[CoordinatedGraphics] Don't reset m_shouldSyncFrame in flushPendingLayerChanges
https://bugs.webkit.org/show_bug.cgi?id=97108

Reviewed by Noam Rosenthal.

Stop ignoring if m_shouldSyncFrame has been set between the two
layer flush. It can be set during layout in several situations,
for example when a layer is deleted or changed state. We want to
send the DidRenderFrame message at the next flush in those situations
so the UI process will apply the changes as soon as possible.

* WebProcess/WebPage/CoordinatedGraphics/LayerTreeCoordinator.cpp:
(WebKit::LayerTreeCoordinator::flushPendingLayerChanges):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (129115 => 129116)


--- trunk/Source/WebKit2/ChangeLog	2012-09-20 11:06:35 UTC (rev 129115)
+++ trunk/Source/WebKit2/ChangeLog	2012-09-20 11:14:10 UTC (rev 129116)
@@ -1,3 +1,19 @@
+2012-09-20  Balazs Kelemen  <[email protected]>
+
+        [CoordinatedGraphics] Don't reset m_shouldSyncFrame in flushPendingLayerChanges
+        https://bugs.webkit.org/show_bug.cgi?id=97108
+
+        Reviewed by Noam Rosenthal.
+
+        Stop ignoring if m_shouldSyncFrame has been set between the two
+        layer flush. It can be set during layout in several situations,
+        for example when a layer is deleted or changed state. We want to
+        send the DidRenderFrame message at the next flush in those situations
+        so the UI process will apply the changes as soon as possible.
+
+        * WebProcess/WebPage/CoordinatedGraphics/LayerTreeCoordinator.cpp:
+        (WebKit::LayerTreeCoordinator::flushPendingLayerChanges):
+
 2012-09-20  Byungwoo Lee  <[email protected]>
 
         [EFL][WK2] Check timeout on waitUntilLoadFinished() and waitUntilTitleChangedTo().

Modified: trunk/Source/WebKit2/WebProcess/WebPage/CoordinatedGraphics/LayerTreeCoordinator.cpp (129115 => 129116)


--- trunk/Source/WebKit2/WebProcess/WebPage/CoordinatedGraphics/LayerTreeCoordinator.cpp	2012-09-20 11:06:35 UTC (rev 129115)
+++ trunk/Source/WebKit2/WebProcess/WebPage/CoordinatedGraphics/LayerTreeCoordinator.cpp	2012-09-20 11:14:10 UTC (rev 129116)
@@ -254,8 +254,6 @@
     if (m_waitingForUIProcess)
         return false;
 
-    m_shouldSyncFrame = false;
-
     for (size_t i = 0; i < m_detachedLayers.size(); ++i)
         m_webPage->send(Messages::LayerTreeCoordinatorProxy::DeleteCompositingLayer(m_detachedLayers[i]));
     m_detachedLayers.clear();
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to