Remove StyleDifference::needsPaintPropertyUpdate()

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

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2000 Dirk Mueller (mueller@kde.org)
  *           (C) 2004 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2011 Apple Inc.
  *               All rights reserved.
  * Copyright (C) 2009 Google Inc. All rights reserved.
  * Copyright (C) 2009 Torch Mobile Inc. All rights reserved.
  *               (http://www.torchmobile.com/)
@@ skipping 1709 matching lines @@
 
   if (diff.needsPaintInvalidationSubtree() ||
       updatedDiff.needsPaintInvalidationSubtree())
     setShouldDoFullPaintInvalidationIncludingNonCompositingDescendants();
   else if (diff.needsPaintInvalidationObject() ||
            updatedDiff.needsPaintInvalidationObject())
     setShouldDoFullPaintInvalidation();
 
   // Text nodes share style with their parents but the paint properties don't
   // apply to them, hence the !isText() check.
-  if (RuntimeEnabledFeatures::slimmingPaintInvalidationEnabled() &&
-      diff.needsPaintPropertyUpdate() && !isText()) {
+  if (RuntimeEnabledFeatures::slimmingPaintInvalidationEnabled() && !isText() &&
+      (diff.transformChanged() || diff.opacityChanged() ||
+       diff.zIndexChanged() || diff.filterChanged() ||
+       diff.backdropFilterChanged() || diff.cssClipChanged())) {
     setNeedsPaintPropertyUpdate();
 
     // We don't need to invalidate paint of objects on SPv2 when only paint
     // property or paint order change. Mark the painting layer needing repaint
     // for changed paint property or paint order. Raster invalidation will be
     // issued if needed during paint.
-    if (RuntimeEnabledFeatures::slimmingPaintV2Enabled())
+    if (RuntimeEnabledFeatures::slimmingPaintV2Enabled() &&
+        !shouldDoFullPaintInvalidation())
       ObjectPaintInvalidator(*this).slowSetPaintingLayerNeedsRepaint();
   }
 }
 
 void LayoutObject::styleWillChange(StyleDifference diff,
                                    const ComputedStyle& newStyle) {
   if (m_style) {
     // If our z-index changes value or our visibility changes,
     // we need to dirty our stacking context's z-order list.
     bool visibilityChanged =
@@ skipping 1762 matching lines @@
     const blink::LayoutObject* root = object1;
     while (root->parent())
       root = root->parent();
     root->showLayoutTreeAndMark(object1, "*", object2, "-", 0);
   } else {
     WTFLogAlways("%s", "Cannot showLayoutTree. Root is (nil)");
   }
 }
 
 #endif