[SPv2] Apply effects before clips for when the effect does not move pixels

third_party/WebKit/Source/platform/graphics/paint/PropertyTreeState.cpp

 // Copyright 2016 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.
 
 #include "platform/graphics/paint/PropertyTreeState.h"
 
 #include "platform/graphics/paint/GeometryMapper.h"
 
 namespace blink {
 
@@ skipping 74 matching lines @@
       IsAncestorOf<TransformPaintPropertyNode>(effect_->LocalTransformSpace(),
                                                transform_.Get());
 
   if (!transform_->IsRoot() && clip_transform_strict_ancestor_of_transform &&
       effect_transform_strict_ancestor_of_transform)
     return kTransform;
 
   bool clip_ancestor_of_effect =
       IsAncestorOf<ClipPaintPropertyNode>(clip_.Get(), effect_->OutputClip());
 
-  if (!effect_->IsRoot() && clip_ancestor_of_effect) {
+  if (!effect_->IsRoot() &&
+      (clip_ancestor_of_effect ||
+       // Effects that don't move pixels commute with all clips, so always apply
+       // them first when inside compatible transforms.
+       (!effect_->HasFilterThatMovesPixels() &&
+        !effect_transform_strict_ancestor_of_transform))) {
     return kEffect;
   }
   if (!clip_->IsRoot())
     return kClip;
   return kNone;
 }
 
 const PropertyTreeState* PropertyTreeStateIterator::Next() {
   switch (properties_.GetInnermostNode()) {
     case PropertyTreeState::kTransform:
@@ skipping 14 matching lines @@
 #if DCHECK_IS_ON()
 
 String PropertyTreeState::ToTreeString() const {
   return Transform()->ToTreeString() + "\n" + Clip()->ToTreeString() + "\n" +
          Effect()->ToTreeString();
 }
 
 #endif
 
 }  // namespace blink