Early-out from the prepaint tree walk

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

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef ObjectPaintProperties_h
 #define ObjectPaintProperties_h
 
 #include "core/CoreExport.h"
 #include "platform/geometry/LayoutPoint.h"
 #include "platform/graphics/paint/ClipPaintPropertyNode.h"
 #include "platform/graphics/paint/EffectPaintPropertyNode.h"
 #include "platform/graphics/paint/PaintChunkProperties.h"
 #include "platform/graphics/paint/PropertyTreeState.h"
 #include "platform/graphics/paint/ScrollPaintPropertyNode.h"
 #include "platform/graphics/paint/TransformPaintPropertyNode.h"
 #include "wtf/PassRefPtr.h"
 #include "wtf/PtrUtil.h"
 #include "wtf/RefPtr.h"
 #include <memory>
 
 namespace blink {
 
+enum PropertyWasCreated {
+  DidNotCreateProperty = false,
+  CreatedProperty = true
+};
+
+enum ExistingPropertyWasCleared {
+  DidNotClearExistingProperty = false,
+  ClearedExistingProperty = true
+};
+
 // This class stores property tree related information associated with a
 // LayoutObject.
 // Currently there are two groups of information:
 // 1. The set of property nodes created locally by this LayoutObject.
 // 2. The set of property nodes (inherited, or created locally) and paint offset
 //    that can be used to paint the border box of this LayoutObject (see:
 //    localBorderBoxProperties).
 class CORE_EXPORT ObjectPaintProperties {
   WTF_MAKE_NONCOPYABLE(ObjectPaintProperties);
   USING_FAST_MALLOC(ObjectPaintProperties);
@@ skipping 88 matching lines @@
   // would be an effect node with opacity of 0.3 which was created by the div
   // itself. Note that propertyTreeState.transform() would not be null but would
   // instead point to the transform space setup by div's ancestors.
   const PropertyTreeStateWithOffset* localBorderBoxProperties() const {
     return m_localBorderBoxProperties.get();
   }
   void setLocalBorderBoxProperties(
       std::unique_ptr<PropertyTreeStateWithOffset> properties) {
     m_localBorderBoxProperties = std::move(properties);
   }
+  void clearLocalBorderBoxProperties() { m_localBorderBoxProperties = nullptr; }
 
   // This is the complete set of property nodes and paint offset that can be
   // used to paint the contents of this object. It is similar to
   // localBorderBoxProperties but includes properties (e.g., overflow clip,
   // scroll translation) that apply to contents. This is suitable for paint
   // invalidation.
   ObjectPaintProperties::PropertyTreeStateWithOffset contentsProperties() const;
 
-  void clearPaintOffsetTranslation() { m_paintOffsetTranslation = nullptr; }
-  void clearTransform() { m_transform = nullptr; }
-  void clearEffect() { m_effect = nullptr; }
-  void clearCssClip() { m_cssClip = nullptr; }
-  void clearCssClipFixedPosition() { m_cssClipFixedPosition = nullptr; }
-  void clearInnerBorderRadiusClip() { m_innerBorderRadiusClip = nullptr; }
-  void clearOverflowClip() { m_overflowClip = nullptr; }
-  void clearLocalBorderBoxProperties() { m_localBorderBoxProperties = nullptr; }
-  void clearPerspective() { m_perspective = nullptr; }
-  void clearSvgLocalToBorderBoxTransform() {
-    m_svgLocalToBorderBoxTransform = nullptr;
+  ExistingPropertyWasCleared clearPaintOffsetTranslation() {
+    return clear(m_paintOffsetTranslation);
   }
-  void clearScrollTranslation() { m_scrollTranslation = nullptr; }
-  void clearScrollbarPaintOffset() { m_scrollbarPaintOffset = nullptr; }
-  void clearScroll() { m_scroll = nullptr; }
+  ExistingPropertyWasCleared clearTransform() { return clear(m_transform); }
+  ExistingPropertyWasCleared clearEffect() { return clear(m_effect); }
+  ExistingPropertyWasCleared clearCssClip() { return clear(m_cssClip); }
+  ExistingPropertyWasCleared clearCssClipFixedPosition() {
+    return clear(m_cssClipFixedPosition);
+  }
+  ExistingPropertyWasCleared clearInnerBorderRadiusClip() {
+    return clear(m_innerBorderRadiusClip);
+  }
+  ExistingPropertyWasCleared clearOverflowClip() {
+    return clear(m_overflowClip);
+  }
+  ExistingPropertyWasCleared clearPerspective() { return clear(m_perspective); }
+  ExistingPropertyWasCleared clearSvgLocalToBorderBoxTransform() {
+    return clear(m_svgLocalToBorderBoxTransform);
+  }
+  ExistingPropertyWasCleared clearScrollTranslation() {
+    return clear(m_scrollTranslation);
+  }
+  ExistingPropertyWasCleared clearScrollbarPaintOffset() {
+    return clear(m_scrollbarPaintOffset);
+  }
+  ExistingPropertyWasCleared clearScroll() { return clear(m_scroll); }
 
   template <typename... Args>
-  void updatePaintOffsetTranslation(Args&&... args) {
-    updateProperty(m_paintOffsetTranslation, std::forward<Args>(args)...);
+  PropertyWasCreated updatePaintOffsetTranslation(Args&&... args) {
+    return update(m_paintOffsetTranslation, std::forward<Args>(args)...);
   }
   template <typename... Args>
-  void updateTransform(Args&&... args) {
-    updateProperty(m_transform, std::forward<Args>(args)...);
+  PropertyWasCreated updateTransform(Args&&... args) {
+    return update(m_transform, std::forward<Args>(args)...);
   }
   template <typename... Args>
-  void updatePerspective(Args&&... args) {
-    updateProperty(m_perspective, std::forward<Args>(args)...);
+  PropertyWasCreated updatePerspective(Args&&... args) {
+    return update(m_perspective, std::forward<Args>(args)...);
   }
   template <typename... Args>
-  void updateSvgLocalToBorderBoxTransform(Args&&... args) {
+  PropertyWasCreated updateSvgLocalToBorderBoxTransform(Args&&... args) {
     DCHECK(!scrollTranslation()) << "SVG elements cannot scroll so there "
                                     "should never be both a scroll translation "
                                     "and an SVG local to border box transform.";
-    updateProperty(m_svgLocalToBorderBoxTransform, std::forward<Args>(args)...);
+    return update(m_svgLocalToBorderBoxTransform, std::forward<Args>(args)...);
   }
   template <typename... Args>
-  void updateScrollTranslation(Args&&... args) {
+  PropertyWasCreated updateScrollTranslation(Args&&... args) {
     DCHECK(!svgLocalToBorderBoxTransform())
         << "SVG elements cannot scroll so there should never be both a scroll "
            "translation and an SVG local to border box transform.";
-    updateProperty(m_scrollTranslation, std::forward<Args>(args)...);
+    return update(m_scrollTranslation, std::forward<Args>(args)...);
   }
   template <typename... Args>
-  void updateScrollbarPaintOffset(Args&&... args) {
-    updateProperty(m_scrollbarPaintOffset, std::forward<Args>(args)...);
+  PropertyWasCreated updateScrollbarPaintOffset(Args&&... args) {
+    return update(m_scrollbarPaintOffset, std::forward<Args>(args)...);
   }
   template <typename... Args>
-  void updateScroll(Args&&... args) {
-    updateProperty(m_scroll, std::forward<Args>(args)...);
+  PropertyWasCreated updateScroll(Args&&... args) {
+    return update(m_scroll, std::forward<Args>(args)...);
   }
   template <typename... Args>
-  void updateEffect(Args&&... args) {
-    updateProperty(m_effect, std::forward<Args>(args)...);
+  PropertyWasCreated updateEffect(Args&&... args) {
+    return update(m_effect, std::forward<Args>(args)...);
   }
   template <typename... Args>
-  void updateCssClip(Args&&... args) {
-    updateProperty(m_cssClip, std::forward<Args>(args)...);
+  PropertyWasCreated updateCssClip(Args&&... args) {
+    return update(m_cssClip, std::forward<Args>(args)...);
   }
   template <typename... Args>
-  void updateCssClipFixedPosition(Args&&... args) {
-    updateProperty(m_cssClipFixedPosition, std::forward<Args>(args)...);
+  PropertyWasCreated updateCssClipFixedPosition(Args&&... args) {
+    return update(m_cssClipFixedPosition, std::forward<Args>(args)...);
   }
   template <typename... Args>
-  void updateInnerBorderRadiusClip(Args&&... args) {
-    updateProperty(m_innerBorderRadiusClip, std::forward<Args>(args)...);
+  PropertyWasCreated updateInnerBorderRadiusClip(Args&&... args) {
+    return update(m_innerBorderRadiusClip, std::forward<Args>(args)...);
   }
   template <typename... Args>
-  void updateOverflowClip(Args&&... args) {
-    updateProperty(m_overflowClip, std::forward<Args>(args)...);
+  PropertyWasCreated updateOverflowClip(Args&&... args) {
+    return update(m_overflowClip, std::forward<Args>(args)...);
   }
 
 #if DCHECK_IS_ON()
   // Used by FindPropertiesNeedingUpdate.h for recording the current properties.
   std::unique_ptr<ObjectPaintProperties> clone() const {
     std::unique_ptr<ObjectPaintProperties> cloned = create();
     if (m_paintOffsetTranslation)
       cloned->m_paintOffsetTranslation = m_paintOffsetTranslation->clone();
     if (m_transform)
       cloned->m_transform = m_transform->clone();
@@ skipping 28 matching lines @@
                                 state.scroll())));
     }
     return cloned;
   }
 #endif
 
  private:
   ObjectPaintProperties() {}
 
   template <typename PaintPropertyNode, typename... Args>
-  void updateProperty(RefPtr<PaintPropertyNode>& field, Args&&... args) {
-    if (field)
+  PropertyWasCreated update(RefPtr<PaintPropertyNode>& field, Args&&... args) {
+    if (field) {
       field->update(std::forward<Args>(args)...);
-    else
-      field = PaintPropertyNode::create(std::forward<Args>(args)...);
+      return DidNotCreateProperty;
+    }
+    field = PaintPropertyNode::create(std::forward<Args>(args)...);
+    return CreatedProperty;
+  }
+
+  template <typename PaintPropertyNode>
+  ExistingPropertyWasCleared clear(RefPtr<PaintPropertyNode>& field) {
+    if (field) {
+      field = nullptr;
+      return ClearedExistingProperty;
+    }
+    return DidNotClearExistingProperty;
   }
 
   RefPtr<TransformPaintPropertyNode> m_paintOffsetTranslation;
   RefPtr<TransformPaintPropertyNode> m_transform;
   RefPtr<EffectPaintPropertyNode> m_effect;
   RefPtr<ClipPaintPropertyNode> m_cssClip;
   RefPtr<ClipPaintPropertyNode> m_cssClipFixedPosition;
   RefPtr<ClipPaintPropertyNode> m_innerBorderRadiusClip;
   RefPtr<ClipPaintPropertyNode> m_overflowClip;
   RefPtr<TransformPaintPropertyNode> m_perspective;
   // TODO(pdr): Only LayoutSVGRoot needs this and it should be moved there.
   RefPtr<TransformPaintPropertyNode> m_svgLocalToBorderBoxTransform;
   RefPtr<TransformPaintPropertyNode> m_scrollTranslation;
   RefPtr<TransformPaintPropertyNode> m_scrollbarPaintOffset;
   RefPtr<ScrollPaintPropertyNode> m_scroll;
 
   std::unique_ptr<PropertyTreeStateWithOffset> m_localBorderBoxProperties;
 };
 
 }  // namespace blink
 
 #endif  // ObjectPaintProperties_h