Title: [155045] trunk/Source/WebCore
Revision
155045
Author
[email protected]
Date
2013-09-04 09:39:53 -0700 (Wed, 04 Sep 2013)

Log Message

[Coordinated Graphics] Remove unused method in CoordinatedGraphicsLayer
https://bugs.webkit.org/show_bug.cgi?id=120664

Patch by Jae Hyun Park <[email protected]> on 2013-09-04
Reviewed by Darin Adler.

CoordinatedGraphicsLayer::hasPendingVisibleChanges is not used as of
r148952.

* platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp:
* platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.h:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (155044 => 155045)


--- trunk/Source/WebCore/ChangeLog	2013-09-04 15:43:34 UTC (rev 155044)
+++ trunk/Source/WebCore/ChangeLog	2013-09-04 16:39:53 UTC (rev 155045)
@@ -1,3 +1,16 @@
+2013-09-04  Jae Hyun Park  <[email protected]>
+
+        [Coordinated Graphics] Remove unused method in CoordinatedGraphicsLayer
+        https://bugs.webkit.org/show_bug.cgi?id=120664
+
+        Reviewed by Darin Adler.
+
+        CoordinatedGraphicsLayer::hasPendingVisibleChanges is not used as of
+        r148952.
+
+        * platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp:
+        * platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.h:
+
 2013-09-04  Hans Muller  <[email protected]>
 
         [CSS Shapes] Revise the ShapeInterval set operations' implementation

Modified: trunk/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp (155044 => 155045)


--- trunk/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp	2013-09-04 15:43:34 UTC (rev 155044)
+++ trunk/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp	2013-09-04 16:39:53 UTC (rev 155045)
@@ -1069,27 +1069,6 @@
         m_pendingVisibleRectAdjustment = true;
 }
 
-bool CoordinatedGraphicsLayer::hasPendingVisibleChanges()
-{
-    if (opacity() < 0.01 && !m_animations.hasActiveAnimationsOfType(AnimatedPropertyOpacity))
-        return false;
-
-    for (size_t i = 0; i < children().size(); ++i) {
-        if (toCoordinatedGraphicsLayer(children()[i])->hasPendingVisibleChanges())
-            return true;
-    }
-
-    bool shouldSyncCanvas = false;
-#if USE(GRAPHICS_SURFACE)
-    shouldSyncCanvas = m_pendingCanvasOperation & SyncCanvas;
-#endif
-
-    if (!m_shouldSyncLayerState && !m_shouldSyncChildren && !m_shouldSyncFilters && !m_shouldSyncImageBacking && !m_shouldSyncAnimations && !shouldSyncCanvas)
-        return false;
-
-    return tiledBackingStoreVisibleRect().intersects(tiledBackingStoreContentsRect());
-}
-
 static inline bool isIntegral(float value)
 {
     return static_cast<int>(value) == value;

Modified: trunk/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.h (155044 => 155045)


--- trunk/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.h	2013-09-04 15:43:34 UTC (rev 155044)
+++ trunk/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.h	2013-09-04 16:39:53 UTC (rev 155045)
@@ -149,7 +149,6 @@
 
     void setNeedsVisibleRectAdjustment();
     void purgeBackingStores();
-    bool hasPendingVisibleChanges();
 
     static void setShouldSupportContentsTiling(bool);
     CoordinatedGraphicsLayer* findFirstDescendantWithContentsRecursively();
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to