Remove some unused RenderLayer code
authorsimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 28 Sep 2018 21:03:23 +0000 (21:03 +0000)
committersimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 28 Sep 2018 21:03:23 +0000 (21:03 +0000)
https://bugs.webkit.org/show_bug.cgi?id=190078

Reviewed by Zalan Bujtas.

The 'outOfFlowDescendantContainingBlocks' code was related to the accelerated overflow scrolling code that
I removed recently.

updateDescendantsLayerListsIfNeeded() is never called.

* rendering/RenderLayer.cpp:
(WebCore::RenderLayer::updateDescendantDependentFlags):
(WebCore::RenderLayer::calculateClipRects const):
* rendering/RenderLayer.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@236611 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/rendering/RenderLayer.cpp
Source/WebCore/rendering/RenderLayer.h

index 620a867..ecc4f38 100644 (file)
@@ -1,3 +1,20 @@
+2018-09-28  Simon Fraser  <simon.fraser@apple.com>
+
+        Remove some unused RenderLayer code
+        https://bugs.webkit.org/show_bug.cgi?id=190078
+
+        Reviewed by Zalan Bujtas.
+
+        The 'outOfFlowDescendantContainingBlocks' code was related to the accelerated overflow scrolling code that
+        I removed recently.
+        
+        updateDescendantsLayerListsIfNeeded() is never called.
+
+        * rendering/RenderLayer.cpp:
+        (WebCore::RenderLayer::updateDescendantDependentFlags):
+        (WebCore::RenderLayer::calculateClipRects const):
+        * rendering/RenderLayer.h:
+
 2018-09-28  Commit Queue  <commit-queue@webkit.org>
 
         Unreviewed, rolling out r236605.
index ef43440..38d7e61 100644 (file)
@@ -1004,7 +1004,7 @@ void RenderLayer::setAncestorChainHasVisibleDescendant()
     }
 }
 
-void RenderLayer::updateDescendantDependentFlags(HashSet<const RenderObject*>* outOfFlowDescendantContainingBlocks)
+void RenderLayer::updateDescendantDependentFlags()
 {
     if (m_visibleDescendantStatusDirty || m_hasSelfPaintingLayerDescendantDirty || hasNotIsolatedBlendingDescendantsStatusDirty()) {
         bool hasVisibleDescendant = false;
@@ -1013,20 +1013,8 @@ void RenderLayer::updateDescendantDependentFlags(HashSet<const RenderObject*>* o
         bool hasNotIsolatedBlendingDescendants = false;
 #endif
 
-        HashSet<const RenderObject*> childOutOfFlowDescendantContainingBlocks;
         for (RenderLayer* child = firstChild(); child; child = child->nextSibling()) {
-            childOutOfFlowDescendantContainingBlocks.clear();
-            child->updateDescendantDependentFlags(&childOutOfFlowDescendantContainingBlocks);
-
-            bool childIsOutOfFlowPositioned = child->renderer().isOutOfFlowPositioned();
-            if (childIsOutOfFlowPositioned)
-                childOutOfFlowDescendantContainingBlocks.add(child->renderer().containingBlock());
-
-            if (outOfFlowDescendantContainingBlocks) {
-                HashSet<const RenderObject*>::const_iterator it = childOutOfFlowDescendantContainingBlocks.begin();
-                for (; it != childOutOfFlowDescendantContainingBlocks.end(); ++it)
-                    outOfFlowDescendantContainingBlocks->add(*it);
-            }
+            child->updateDescendantDependentFlags();
 
             hasVisibleDescendant |= child->m_hasVisibleContent || child->m_hasVisibleDescendant;
             hasSelfPaintingLayerDescendant |= child->isSelfPaintingLayer() || child->hasSelfPaintingLayerDescendant();
@@ -1042,9 +1030,6 @@ void RenderLayer::updateDescendantDependentFlags(HashSet<const RenderObject*>* o
                 break;
         }
 
-        if (outOfFlowDescendantContainingBlocks)
-            outOfFlowDescendantContainingBlocks->remove(&renderer());
-
         m_hasVisibleDescendant = hasVisibleDescendant;
         m_visibleDescendantStatusDirty = false;
         m_hasSelfPaintingLayerDescendant = hasSelfPaintingLayerDescendant;
@@ -5968,32 +5953,6 @@ void RenderLayer::updateLayerListsIfNeeded()
     }
 }
 
-void RenderLayer::updateDescendantsLayerListsIfNeeded(bool recursive)
-{
-    Vector<RenderLayer*> layersToUpdate;
-    
-    if (auto* list = negZOrderList()) {
-        for (auto* childLayer : *list)
-            layersToUpdate.append(childLayer);
-    }
-    
-    if (auto* list = normalFlowList()) {
-        for (auto* childLayer : *list)
-            layersToUpdate.append(childLayer);
-    }
-    
-    if (auto* list = posZOrderList()) {
-        for (auto* childLayer : *list)
-            layersToUpdate.append(childLayer);
-    }
-    
-    for (auto* childLayer : layersToUpdate) {
-        childLayer->updateLayerListsIfNeeded();
-        if (recursive)
-            childLayer->updateDescendantsLayerListsIfNeeded(true);
-    }
-}
-
 void RenderLayer::updateCompositingAndLayerListsIfNeeded()
 {
     if (compositor().inCompositingMode()) {
index 68d7e93..27e935b 100644 (file)
@@ -378,9 +378,6 @@ public:
     // Update our normal and z-index lists.
     void updateLayerListsIfNeeded();
 
-    // Update the normal and z-index lists of our descendants.
-    void updateDescendantsLayerListsIfNeeded(bool recursive);
-
     // FIXME: We should ASSERT(!m_visibleContentStatusDirty) here, but see https://bugs.webkit.org/show_bug.cgi?id=71044
     // ditto for hasVisibleDescendant(), see https://bugs.webkit.org/show_bug.cgi?id=71277
     bool hasVisibleContent() const { return m_hasVisibleContent; }
@@ -913,7 +910,7 @@ private:
     void dirtyAncestorChainVisibleDescendantStatus();
     void setAncestorChainHasVisibleDescendant();
 
-    void updateDescendantDependentFlags(HashSet<const RenderObject*>* outOfFlowDescendantContainingBlocks = nullptr);
+    void updateDescendantDependentFlags();
 
     bool has3DTransformedDescendant() const { return m_has3DTransformedDescendant; }