Title: [229319] trunk/Source/WebKit
Revision
229319
Author
zandober...@gmail.com
Date
2018-03-06 04:17:36 -0800 (Tue, 06 Mar 2018)

Log Message

Unreviewed. Addressing further review feedback for patch landed in r229315.

* Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp:
(WebKit::CoordinatedGraphicsScene::commitSceneState): There's no need for
explicit calls of the clear() method on both containers in the CommitScope
objects, the destructors invoked for these objects from the CommitScope
destructor will have the same effect.

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (229318 => 229319)


--- trunk/Source/WebKit/ChangeLog	2018-03-06 12:12:04 UTC (rev 229318)
+++ trunk/Source/WebKit/ChangeLog	2018-03-06 12:17:36 UTC (rev 229319)
@@ -1,5 +1,15 @@
 2018-03-06  Zan Dobersek  <zdober...@igalia.com>
 
+        Unreviewed. Addressing further review feedback for patch landed in r229315.
+
+        * Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp:
+        (WebKit::CoordinatedGraphicsScene::commitSceneState): There's no need for
+        explicit calls of the clear() method on both containers in the CommitScope
+        objects, the destructors invoked for these objects from the CommitScope
+        destructor will have the same effect.
+
+2018-03-06  Zan Dobersek  <zdober...@igalia.com>
+
         [CoordGraphics] Remove unused scrolling-related code in TextureMapperLayer, CoordinatedGraphics stack
         https://bugs.webkit.org/show_bug.cgi?id=183340
 

Modified: trunk/Source/WebKit/Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp (229318 => 229319)


--- trunk/Source/WebKit/Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp	2018-03-06 12:12:04 UTC (rev 229318)
+++ trunk/Source/WebKit/Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp	2018-03-06 12:17:36 UTC (rev 229319)
@@ -540,9 +540,6 @@
 
     for (auto& backingStore : commitScope.backingStoresWithPendingBuffers)
         backingStore->commitTileOperations(*m_textureMapper);
-
-    commitScope.releasedImageBackings.clear();
-    commitScope.backingStoresWithPendingBuffers.clear();
 }
 
 void CoordinatedGraphicsScene::renderNextFrame()
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to