Generalize visible descendant dirty bits to prepare for more properties.

third_party/WebKit/Source/core/paint/PaintLayer.h

 /*
  * Copyright (C) 2003, 2009, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2013 Intel Corporation. All rights reserved.
  *
  * Portions are Copyright (C) 1998 Netscape Communications Corporation.
  *
  * Other contributors:
  *   Robert O'Callahan <roc+@cs.cmu.edu>
  *   David Baron <dbaron@fas.harvard.edu>
  *   Christian Biesinger <cbiesinger@web.de>
@@ skipping 297 matching lines @@
 
   PaintLayerStackingNode* stackingNode() { return m_stackingNode.get(); }
   const PaintLayerStackingNode* stackingNode() const {
     return m_stackingNode.get();
   }
 
   bool subtreeIsInvisible() const {
     return !hasVisibleContent() && !hasVisibleDescendant();
   }
 
-  // FIXME: hasVisibleContent() should call updateDescendantDependentFlags() if
-  // m_isVisibleContentDirty.
   bool hasVisibleContent() const {
-    DCHECK(!m_isVisibleContentDirty);
+    DCHECK(!m_needsDescendantDependentFlagsUpdate);
     return m_hasVisibleContent;
   }
 
-  // FIXME: hasVisibleDescendant() should call updateDescendantDependentFlags()
-  // if m_isVisibleDescendantDirty.
   bool hasVisibleDescendant() const {
-    DCHECK(!m_isVisibleDescendantDirty);
+    DCHECK(!m_needsDescendantDependentFlagsUpdate);
     return m_hasVisibleDescendant;
   }
 
   void dirtyVisibleContentStatus();
-  void potentiallyDirtyVisibleContentStatus(EVisibility);
 
   bool hasBoxDecorationsOrBackground() const;
   bool hasVisibleBoxDecorations() const;
   // True if this layer container layoutObjects that paint.
   bool hasNonEmptyChildLayoutObjects() const;
 
   // Will ensure that isAllScrollingContentComposited() is up to date.
   void updateScrollingStateAfterCompositingChange();
   bool isAllScrollingContentComposited() const {
     return m_isAllScrollingContentComposited;
@@ skipping 722 matching lines @@
   void updateStackingNode();
 
   FilterOperations addReflectionToFilterOperations(const ComputedStyle&) const;
 
   // FIXME: We could lazily allocate our ScrollableArea based on style
   // properties ('overflow', ...) but for now, we are always allocating it for
   // LayoutBox as it's safer.  crbug.com/467721.
   bool requiresScrollableArea() const { return layoutBox(); }
   void updateScrollableArea();
 
-  void dirtyAncestorChainVisibleDescendantStatus();
+  void markAncestorChainForDescendantDependentFlagsUpdate();
 
   bool attemptDirectCompositingUpdate(StyleDifference,
                                       const ComputedStyle* oldStyle);
   void updateTransform(const ComputedStyle* oldStyle,
                        const ComputedStyle& newStyle);
 
   void removeAncestorOverflowLayer(const PaintLayer* removedLayer);
 
   void updatePaginationRecursive(bool needsPaginationUpdate = false);
   void clearPaginationRecursive();
@@ skipping 39 matching lines @@
 
   // If have no self-painting descendants, we don't have to walk our children
   // during painting. This can lead to significant savings, especially if the
   // tree has lots of non-self-painting layers grouped together (e.g. table
   // cells).
   mutable unsigned m_hasSelfPaintingLayerDescendant : 1;
   mutable unsigned m_hasSelfPaintingLayerDescendantDirty : 1;
 
   const unsigned m_isRootLayer : 1;
 
-  unsigned m_isVisibleContentDirty : 1;
   unsigned m_hasVisibleContent : 1;
-  unsigned m_isVisibleDescendantDirty : 1;
+  unsigned m_needsDescendantDependentFlagsUpdate : 1;
   unsigned m_hasVisibleDescendant : 1;
 
 #if DCHECK_IS_ON()
   unsigned m_needsPositionUpdate : 1;
 #endif
 
   unsigned m_is3DTransformedDescendantDirty : 1;
   // Set on a stacking context layer that has 3D descendants anywhere
   // in a preserves3D hierarchy. Hint to do 3D-aware hit testing.
   unsigned m_has3DTransformedDescendant : 1;
@@ skipping 83 matching lines @@
 
 }  // namespace blink
 
 #ifndef NDEBUG
 // Outside the WebCore namespace for ease of invocation from gdb.
 CORE_EXPORT void showLayerTree(const blink::PaintLayer*);
 CORE_EXPORT void showLayerTree(const blink::LayoutObject*);
 #endif
 
 #endif  // Layer_h