Title: [135622] trunk/Source/WebKit2
Revision
135622
Author
[email protected]
Date
2012-11-23 08:57:59 -0800 (Fri, 23 Nov 2012)

Log Message

Coordinated Graphics: LayerTreeRenderer::removeTile() does not remove tiles actually.
https://bugs.webkit.org/show_bug.cgi?id=103030

Patch by Huang Dongsung <[email protected]> on 2012-11-23
Reviewed by Noam Rosenthal.

LayerTreeRenderer::removeTile() must add a backing store into
m_backingStoresWithPendingBuffers. Otherwise, tiles are not removed.

* UIProcess/CoordinatedGraphics/LayerTreeRenderer.cpp:
(WebKit::LayerTreeRenderer::removeTile):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (135621 => 135622)


--- trunk/Source/WebKit2/ChangeLog	2012-11-23 16:54:35 UTC (rev 135621)
+++ trunk/Source/WebKit2/ChangeLog	2012-11-23 16:57:59 UTC (rev 135622)
@@ -1,5 +1,18 @@
 2012-11-23  Huang Dongsung  <[email protected]>
 
+        Coordinated Graphics: LayerTreeRenderer::removeTile() does not remove tiles actually.
+        https://bugs.webkit.org/show_bug.cgi?id=103030
+
+        Reviewed by Noam Rosenthal.
+
+        LayerTreeRenderer::removeTile() must add a backing store into
+        m_backingStoresWithPendingBuffers. Otherwise, tiles are not removed.
+
+        * UIProcess/CoordinatedGraphics/LayerTreeRenderer.cpp:
+        (WebKit::LayerTreeRenderer::removeTile):
+
+2012-11-23  Huang Dongsung  <[email protected]>
+
         [TexMap] Remove dependency of TextureMapperLayer on GraphicsLayerTextureMapper.
         https://bugs.webkit.org/show_bug.cgi?id=103046
 

Modified: trunk/Source/WebKit2/UIProcess/CoordinatedGraphics/LayerTreeRenderer.cpp (135621 => 135622)


--- trunk/Source/WebKit2/UIProcess/CoordinatedGraphics/LayerTreeRenderer.cpp	2012-11-23 16:54:35 UTC (rev 135621)
+++ trunk/Source/WebKit2/UIProcess/CoordinatedGraphics/LayerTreeRenderer.cpp	2012-11-23 16:57:59 UTC (rev 135622)
@@ -451,6 +451,7 @@
     RefPtr<CoordinatedBackingStore> backingStore = getBackingStore(layer);
     backingStore->removeTile(tileID);
     resetBackingStoreSizeToLayerSize(layer);
+    m_backingStoresWithPendingBuffers.add(backingStore);
     removeBackingStoreIfNeeded(layer);
 }
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to