Revert of Skip paint property update and visual rect update if no geometry change

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

Index: third_party/WebKit/Source/core/paint/FindPropertiesNeedingUpdate.h
diff --git a/third_party/WebKit/Source/core/paint/FindPropertiesNeedingUpdate.h b/third_party/WebKit/Source/core/paint/FindPropertiesNeedingUpdate.h
index 3c6fee9cbf1e2692d208f1c0e9035f4f71c8697a..079ba73b700ee250539c33764427f850a44015ad 100644
--- a/third_party/WebKit/Source/core/paint/FindPropertiesNeedingUpdate.h
+++ b/third_party/WebKit/Source/core/paint/FindPropertiesNeedingUpdate.h
@@ -6,12 +6,6 @@
 #define FindPropertiesNeedingUpdate_h
 
 #if DCHECK_IS_ON()
-
-#include "core/frame/FrameView.h"
-#include "core/layout/LayoutObject.h"
-#include "core/paint/ObjectPaintProperties.h"
-#include "core/paint/PaintPropertyTreeBuilder.h"
-
 namespace blink {
 
 // This file contains two scope classes for catching cases where paint
@@ -96,9 +90,9 @@
   Persistent<FrameView> m_frameView;
   bool m_neededPaintPropertyUpdate;
   bool m_neededForcedSubtreeUpdate;
-  RefPtr<const TransformPaintPropertyNode> m_originalPreTranslation;
-  RefPtr<const ClipPaintPropertyNode> m_originalContentClip;
-  RefPtr<const TransformPaintPropertyNode> m_originalScrollTranslation;
+  RefPtr<TransformPaintPropertyNode> m_originalPreTranslation;
+  RefPtr<ClipPaintPropertyNode> m_originalContentClip;
+  RefPtr<TransformPaintPropertyNode> m_originalScrollTranslation;
 };
 
 #define DCHECK_OBJECT_PROPERTY_EQ(object, original, updated)            \
@@ -132,18 +126,6 @@
   }
 
   ~FindObjectPropertiesNeedingUpdateScope() {
-    // Paint offset and paintOffsetTranslation should not change under
-    // FindObjectPropertiesNeedingUpdateScope no matter if we needed paint
-    // property update.
-    DCHECK_OBJECT_PROPERTY_EQ(m_object, &m_originalPaintOffset,
-                              &m_object.paintOffset());
-    const auto* objectProperties = m_object.paintProperties();
-    if (m_originalProperties && objectProperties) {
-      DCHECK_OBJECT_PROPERTY_EQ(m_object,
-                                m_originalProperties->paintOffsetTranslation(),
-                                objectProperties->paintOffsetTranslation());
-    }
-
     // No need to check if an update was already needed.
     if (m_neededPaintPropertyUpdate || m_neededForcedSubtreeUpdate)
       return;
@@ -154,7 +136,13 @@
     //    LayoutObject::setNeedsPaintPropertyUpdate().
     // 2) The PrePaintTreeWalk should have had a forced subtree update (see:
     //    PaintPropertyTreeBuilderContext::forceSubtreeUpdate).
+    DCHECK_OBJECT_PROPERTY_EQ(m_object, &m_originalPaintOffset,
+                              &m_object.paintOffset());
+    const auto* objectProperties = m_object.paintProperties();
     if (m_originalProperties && objectProperties) {
+      DCHECK_OBJECT_PROPERTY_EQ(m_object,
+                                m_originalProperties->paintOffsetTranslation(),
+                                objectProperties->paintOffsetTranslation());
       DCHECK_OBJECT_PROPERTY_EQ(m_object, m_originalProperties->transform(),
                                 objectProperties->transform());
       DCHECK_OBJECT_PROPERTY_EQ(m_object, m_originalProperties->effect(),