Title: [106052] trunk/Source/WebCore
Revision
106052
Author
al...@chromium.org
Date
2012-01-26 15:13:44 -0800 (Thu, 26 Jan 2012)

Log Message

[chromium] Remove dead and unnecessary code related to LayerChromium::cleanupResources
https://bugs.webkit.org/show_bug.cgi?id=77137

Reviewed by James Robinson.

No new test needed. Only removing dead code.

* platform/graphics/chromium/LayerChromium.cpp:
* platform/graphics/chromium/LayerChromium.h:
(WebCore::LayerChromium::layerTreeHost):
* platform/graphics/chromium/TiledLayerChromium.cpp:
(WebCore::TiledLayerChromium::setLayerTreeHost):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (106051 => 106052)


--- trunk/Source/WebCore/ChangeLog	2012-01-26 23:06:01 UTC (rev 106051)
+++ trunk/Source/WebCore/ChangeLog	2012-01-26 23:13:44 UTC (rev 106052)
@@ -1,3 +1,18 @@
+2012-01-26  Alok Priyadarshi  <al...@chromium.org>
+
+        [chromium] Remove dead and unnecessary code related to LayerChromium::cleanupResources
+        https://bugs.webkit.org/show_bug.cgi?id=77137
+
+        Reviewed by James Robinson.
+
+        No new test needed. Only removing dead code.
+
+        * platform/graphics/chromium/LayerChromium.cpp:
+        * platform/graphics/chromium/LayerChromium.h:
+        (WebCore::LayerChromium::layerTreeHost):
+        * platform/graphics/chromium/TiledLayerChromium.cpp:
+        (WebCore::TiledLayerChromium::setLayerTreeHost):
+
 2012-01-26  Anders Carlsson  <ander...@apple.com>
 
         Move horizontal rubber-band checks to ScrollElasticityController::handleWheelEvent

Modified: trunk/Source/WebCore/platform/graphics/chromium/LayerChromium.cpp (106051 => 106052)


--- trunk/Source/WebCore/platform/graphics/chromium/LayerChromium.cpp	2012-01-26 23:06:01 UTC (rev 106051)
+++ trunk/Source/WebCore/platform/graphics/chromium/LayerChromium.cpp	2012-01-26 23:13:44 UTC (rev 106052)
@@ -94,19 +94,6 @@
 {
 }
 
-void LayerChromium::cleanupResourcesRecursive()
-{
-    for (size_t i = 0; i < children().size(); ++i)
-        children()[i]->cleanupResourcesRecursive();
-
-    if (maskLayer())
-        maskLayer()->cleanupResourcesRecursive();
-    if (replicaLayer())
-        replicaLayer()->cleanupResourcesRecursive();
-
-    cleanupResources();
-}
-
 void LayerChromium::setIsNonCompositedContent(bool isNonCompositedContent)
 {
     m_isNonCompositedContent = isNonCompositedContent;

Modified: trunk/Source/WebCore/platform/graphics/chromium/LayerChromium.h (106051 => 106052)


--- trunk/Source/WebCore/platform/graphics/chromium/LayerChromium.h	2012-01-26 23:06:01 UTC (rev 106051)
+++ trunk/Source/WebCore/platform/graphics/chromium/LayerChromium.h	2012-01-26 23:13:44 UTC (rev 106052)
@@ -196,7 +196,6 @@
     virtual void contentChanged() { }
 
     CCLayerTreeHost* layerTreeHost() const { return m_layerTreeHost.get(); }
-    void cleanupResourcesRecursive();
 
 protected:
     LayerChromium();

Modified: trunk/Source/WebCore/platform/graphics/chromium/TiledLayerChromium.cpp (106051 => 106052)


--- trunk/Source/WebCore/platform/graphics/chromium/TiledLayerChromium.cpp	2012-01-26 23:06:01 UTC (rev 106051)
+++ trunk/Source/WebCore/platform/graphics/chromium/TiledLayerChromium.cpp	2012-01-26 23:13:44 UTC (rev 106052)
@@ -178,9 +178,6 @@
     if (host == layerTreeHost())
         return;
 
-    if (layerTreeHost())
-        cleanupResources();
-
     LayerChromium::setLayerTreeHost(host);
 
     if (!host)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to