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

third_party/WebKit/Source/platform/graphics/paint/PropertyTreeStateTest.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 "testing/gtest/include/gtest/gtest.h"
 
 namespace blink {
 
 class PropertyTreeStateTest : public ::testing::Test {};
 
+TEST_F(PropertyTreeStateTest, ClipBelowOutputClipOfEffect) {
+  RefPtr<ClipPaintPropertyNode> clip = ClipPaintPropertyNode::Create(
+      ClipPaintPropertyNode::Root(), TransformPaintPropertyNode::Root(),
+      FloatRoundedRect());
+
+  RefPtr<EffectPaintPropertyNode> effect = EffectPaintPropertyNode::Create(
+      EffectPaintPropertyNode::Root(), TransformPaintPropertyNode::Root(),
+      ClipPaintPropertyNode::Root(), kColorFilterNone,
+      CompositorFilterOperations(), 1.0, SkBlendMode::kSrcOver);
+
+  PropertyTreeState state(TransformPaintPropertyNode::Root(), clip.Get(),
+                          effect.Get());
+  EXPECT_EQ(PropertyTreeState::kEffect, state.GetInnermostNode());
+
+  PropertyTreeStateIterator iterator(state);
+  EXPECT_EQ(PropertyTreeState::kClip, iterator.Next()->GetInnermostNode());
+  EXPECT_EQ(PropertyTreeState::kNone, iterator.Next()->GetInnermostNode());
+}
+
+TEST_F(PropertyTreeStateTest, ClipBelowOutputClipOfEffectMovingPixels) {
+  RefPtr<ClipPaintPropertyNode> clip = ClipPaintPropertyNode::Create(
+      ClipPaintPropertyNode::Root(), TransformPaintPropertyNode::Root(),
+      FloatRoundedRect());
+
+  CompositorFilterOperations operations;
+  operations.AppendBlurFilter(2);
+  RefPtr<EffectPaintPropertyNode> effect = EffectPaintPropertyNode::Create(
+      EffectPaintPropertyNode::Root(), TransformPaintPropertyNode::Root(),
+      ClipPaintPropertyNode::Root(), kColorFilterNone, operations, 1.0,
+      SkBlendMode::kSrcOver);
+
+  PropertyTreeState state(TransformPaintPropertyNode::Root(), clip.Get(),
+                          effect.Get());
+  // If the effect moves pixels, the clip must happen first.
+  EXPECT_EQ(PropertyTreeState::kClip, state.GetInnermostNode());
+
+  PropertyTreeStateIterator iterator(state);
+  EXPECT_EQ(PropertyTreeState::kEffect, iterator.Next()->GetInnermostNode());
+  EXPECT_EQ(PropertyTreeState::kNone, iterator.Next()->GetInnermostNode());
+}
+
 TEST_F(PropertyTreeStateTest, TransformOnEffectOnClip) {
   RefPtr<TransformPaintPropertyNode> transform =
       TransformPaintPropertyNode::Create(TransformPaintPropertyNode::Root(),
                                          TransformationMatrix(),
                                          FloatPoint3D());
 
   RefPtr<ClipPaintPropertyNode> clip = ClipPaintPropertyNode::Create(
       ClipPaintPropertyNode::Root(), TransformPaintPropertyNode::Root(),
       FloatRoundedRect());
 
@@ skipping 49 matching lines @@
 
   RefPtr<ClipPaintPropertyNode> clip = ClipPaintPropertyNode::Create(
       ClipPaintPropertyNode::Root(), transform.Get(), FloatRoundedRect());
 
   RefPtr<EffectPaintPropertyNode> effect = EffectPaintPropertyNode::Create(
       EffectPaintPropertyNode::Root(), transform.Get(),
       ClipPaintPropertyNode::Root(), kColorFilterNone,
       CompositorFilterOperations(), 1.0, SkBlendMode::kSrcOver);
 
   PropertyTreeState state(transform.Get(), clip.Get(), effect.Get());
-  EXPECT_EQ(PropertyTreeState::kClip, state.GetInnermostNode());
+  EXPECT_EQ(PropertyTreeState::kEffect, state.GetInnermostNode());
 
   PropertyTreeStateIterator iterator(state);
-  EXPECT_EQ(PropertyTreeState::kEffect, iterator.Next()->GetInnermostNode());
+  EXPECT_EQ(PropertyTreeState::kClip, iterator.Next()->GetInnermostNode());
   EXPECT_EQ(PropertyTreeState::kTransform, iterator.Next()->GetInnermostNode());
   EXPECT_EQ(PropertyTreeState::kNone, iterator.Next()->GetInnermostNode());
 }
 
 TEST_F(PropertyTreeStateTest, ClipDescendantOfTransform) {
+  // CSS transform
   RefPtr<TransformPaintPropertyNode> transform =
       TransformPaintPropertyNode::Create(TransformPaintPropertyNode::Root(),
                                          TransformationMatrix(),
                                          FloatPoint3D());
 
+  // Scroll transform
   RefPtr<TransformPaintPropertyNode> transform2 =
       TransformPaintPropertyNode::Create(
           transform.Get(), TransformationMatrix(), FloatPoint3D());
 
+  // CSS clip
   RefPtr<ClipPaintPropertyNode> clip = ClipPaintPropertyNode::Create(
       ClipPaintPropertyNode::Root(), transform2.Get(), FloatRoundedRect());
 
   RefPtr<EffectPaintPropertyNode> effect = EffectPaintPropertyNode::Create(
       EffectPaintPropertyNode::Root(), TransformPaintPropertyNode::Root(),
       ClipPaintPropertyNode::Root(), kColorFilterNone,
       CompositorFilterOperations(), 1.0, SkBlendMode::kSrcOver);
 
-  // Here the clip is inside of its own transform, but the transform is an
-  // ancestor of the clip's transform. This models situations such as
-  // a clip inside a scroller that applies to an absolute-positioned element
-  // which escapes the scroll transform but not the clip.
+  // Here the clip is inside of its own transform, but the
+  // PropertyTreeState of the content is an ancestor of the clip's transform./
+  // This models situations such as a CSS clip inside a scroller that applies to
+  // an absolute-positioned element which escapes the scroll transforms but not
+  // the clip.
   PropertyTreeState state(transform.Get(), clip.Get(), effect.Get());
   EXPECT_EQ(PropertyTreeState::kClip, state.GetInnermostNode());
 
   PropertyTreeStateIterator iterator(state);
   EXPECT_EQ(PropertyTreeState::kTransform, iterator.Next()->GetInnermostNode());
   EXPECT_EQ(PropertyTreeState::kEffect, iterator.Next()->GetInnermostNode());
   EXPECT_EQ(PropertyTreeState::kNone, iterator.Next()->GetInnermostNode());
 }
 
 TEST_F(PropertyTreeStateTest, EffectDescendantOfTransform) {
-  RefPtr<TransformPaintPropertyNode> transform =
-      TransformPaintPropertyNode::Create(TransformPaintPropertyNode::Root(),
-                                         TransformationMatrix(),
-                                         FloatPoint3D());
-
   RefPtr<ClipPaintPropertyNode> clip = ClipPaintPropertyNode::Create(
       ClipPaintPropertyNode::Root(), TransformPaintPropertyNode::Root(),
       FloatRoundedRect());
 
   RefPtr<TransformPaintPropertyNode> transform2 =
       TransformPaintPropertyNode::Create(TransformPaintPropertyNode::Root(),
                                          TransformationMatrix(),
                                          FloatPoint3D());
 
   RefPtr<EffectPaintPropertyNode> effect = EffectPaintPropertyNode::Create(
       EffectPaintPropertyNode::Root(), transform2.Get(), clip.Get(),
       kColorFilterNone, CompositorFilterOperations(), 1.0,
       SkBlendMode::kSrcOver);
 
-  // Here the clip is inside of its own transform, but the transform is an
-  // ancestor of the clip's transform. This models situations such as
-  // a clip inside a scroller that applies to an absolute-positioned element
-  // which escapes the scroll transform but not the clip.
-  PropertyTreeState state(transform.Get(), clip.Get(), effect.Get());
+  PropertyTreeState state(transform2.Get(), clip.Get(), effect.Get());
   EXPECT_EQ(PropertyTreeState::kEffect, state.GetInnermostNode());
 
   PropertyTreeStateIterator iterator(state);
   EXPECT_EQ(PropertyTreeState::kTransform, iterator.Next()->GetInnermostNode());
   EXPECT_EQ(PropertyTreeState::kClip, iterator.Next()->GetInnermostNode());
   EXPECT_EQ(PropertyTreeState::kNone, iterator.Next()->GetInnermostNode());
 }
 
 TEST_F(PropertyTreeStateTest, CompositorElementIdNoElementIdOnAnyNode) {
   PropertyTreeState state(TransformPaintPropertyNode::Root(),
@@ skipping 71 matching lines @@
   EXPECT_EQ(second_compositor_element_id,
             state.GetCompositorElementId(composited_element_ids));
 
   composited_element_ids.clear();
   composited_element_ids.insert(second_compositor_element_id);
   EXPECT_EQ(first_compositor_element_id,
             state.GetCompositorElementId(composited_element_ids));
 }
 
 }  // namespace blink