Remove caching of contents paint properties

third_party/WebKit/Source/core/layout/LayoutObject.cpp

Index: third_party/WebKit/Source/core/layout/LayoutObject.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutObject.cpp b/third_party/WebKit/Source/core/layout/LayoutObject.cpp
index 22a31999a0631cfd3d6b81222a611cbd7168140b..b306602e4a9c3848e1369064b2e4f5c329a057e7 100644
--- a/third_party/WebKit/Source/core/layout/LayoutObject.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutObject.cpp
@@ -3580,6 +3580,13 @@ void LayoutObject::SetIsBackgroundAttachmentFixedObject(
     GetFrameView()->RemoveBackgroundAttachmentFixedObject(this);
 }
 
+PropertyTreeState LayoutObject::ContentsProperties() const {
+  DCHECK(RuntimeEnabledFeatures::slimmingPaintInvalidationEnabled());
+  if (rare_paint_data_)
+    return rare_paint_data_->ContentsProperties();
+  return PropertyTreeState::Root();

[chrishtr, 2017/04/10 19:44:50]

Why would 3585 ever fail if the code is correct?

[pdr., 2017/04/10 20:05:34]

No. In the latest patch I refactored this to make it more clear.

+}
+
 RarePaintData& LayoutObject::EnsureRarePaintData() {
   if (!rare_paint_data_)
     rare_paint_data_ = WTF::MakeUnique<RarePaintData>();