Title: [231426] releases/WebKitGTK/webkit-2.20/Source/WebKit
Revision
231426
Author
[email protected]
Date
2018-05-07 02:18:09 -0700 (Mon, 07 May 2018)

Log Message

Merge r230954 - REGRESSION(r230950): Faulty commit sequencing in CoordinatedGraphicsScene
https://bugs.webkit.org/show_bug.cgi?id=184917

Reviewed by Michael Catanzaro.

After r230950, current animation state for a given layer is also taken
into account when determining whether or not the layer requires a
backing store. For that to work properly, all the animation state has
to be updated before the backing store work. This patch changes the
order of helper method invocations in
CoordinatedGraphicsScene::setLayerState() to address that.

* Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp:
(WebKit::CoordinatedGraphicsScene::setLayerState):

Modified Paths

Diff

Modified: releases/WebKitGTK/webkit-2.20/Source/WebKit/ChangeLog (231425 => 231426)


--- releases/WebKitGTK/webkit-2.20/Source/WebKit/ChangeLog	2018-05-07 09:18:04 UTC (rev 231425)
+++ releases/WebKitGTK/webkit-2.20/Source/WebKit/ChangeLog	2018-05-07 09:18:09 UTC (rev 231426)
@@ -1,5 +1,22 @@
 2018-04-24  Zan Dobersek  <[email protected]>
 
+        REGRESSION(r230950): Faulty commit sequencing in CoordinatedGraphicsScene
+        https://bugs.webkit.org/show_bug.cgi?id=184917
+
+        Reviewed by Michael Catanzaro.
+
+        After r230950, current animation state for a given layer is also taken
+        into account when determining whether or not the layer requires a
+        backing store. For that to work properly, all the animation state has
+        to be updated before the backing store work. This patch changes the
+        order of helper method invocations in
+        CoordinatedGraphicsScene::setLayerState() to address that.
+
+        * Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp:
+        (WebKit::CoordinatedGraphicsScene::setLayerState):
+
+2018-04-24  Zan Dobersek  <[email protected]>
+
         [CoordGraphics] Avoid painting backing stores for zero-opacity layers
         https://bugs.webkit.org/show_bug.cgi?id=184143
 

Modified: releases/WebKitGTK/webkit-2.20/Source/WebKit/Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp (231425 => 231426)


--- releases/WebKitGTK/webkit-2.20/Source/WebKit/Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp	2018-05-07 09:18:04 UTC (rev 231425)
+++ releases/WebKitGTK/webkit-2.20/Source/WebKit/Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp	2018-05-07 09:18:09 UTC (rev 231426)
@@ -282,15 +282,14 @@
     if (layerState.committedScrollOffsetChanged)
         layer->didCommitScrollOffset(layerState.committedScrollOffset);
 
-    prepareContentBackingStore(layer);
-
     // Apply Operations.
     setLayerChildrenIfNeeded(layer, layerState);
+    setLayerFiltersIfNeeded(layer, layerState);
+    setLayerAnimationsIfNeeded(layer, layerState);
+    prepareContentBackingStore(layer);
     createTilesIfNeeded(layer, layerState);
     removeTilesIfNeeded(layer, layerState);
     updateTilesIfNeeded(layer, layerState);
-    setLayerFiltersIfNeeded(layer, layerState);
-    setLayerAnimationsIfNeeded(layer, layerState);
     syncPlatformLayerIfNeeded(layer, layerState);
     setLayerRepaintCountIfNeeded(layer, layerState);
 }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to