draft

draft

format

format

clean

Author: Chris Yang

flow/embedded_views.cc

@@ -100,8 +100,10 @@ void MutatorsStack::PushOpacity(const int& alpha) {
 };
 
 void MutatorsStack::PushBackdropFilter(
-    const std::shared_ptr<const DlImageFilter>& filter) {
-  std::shared_ptr<Mutator> element = std::make_shared<Mutator>(filter);
+    const std::shared_ptr<const DlImageFilter>& filter,
+    const SkRect& filter_rect) {
+  std::shared_ptr<Mutator> element =
+      std::make_shared<Mutator>(filter, filter_rect);
   vector_.push_back(element);
 };
 

flow/embedded_views.h

@@ -32,6 +32,33 @@ enum MutatorType {
   kBackdropFilter
 };
 
+// Represents an image filter mutation.
+//
+// Should be used for image_filter_layer and backdrop_filter_layer.
+// TODO(cyanglaz): Refactor this into a ImageFilterMutator class.
+// https://github.com/flutter/flutter/issues/108470
+class ImageFilterMutation {
+ public:
+  ImageFilterMutation(std::shared_ptr<const DlImageFilter> filter,
+                      const SkRect& filter_rect)
+      : filter_(filter), filter_rect_(filter_rect) {}
+
+  const DlImageFilter& GetFilter() const { return *filter_; }
+  const SkRect& GetFilterRect() const { return filter_rect_; }
+
+  bool operator==(const ImageFilterMutation& other) const {
+    return *filter_ == *other.filter_ && filter_rect_ == other.filter_rect_;
+  }
+
+  bool operator!=(const ImageFilterMutation& other) const {
+    return !operator==(other);
+  }
+
+ private:
+  std::shared_ptr<const DlImageFilter> filter_;
+  const SkRect filter_rect_;
+};
+
 // Stores mutation information like clipping or kTransform.
 //
 // The `type` indicates the type of the mutation: kClipRect, kTransform and etc.
@@ -73,15 +100,17 @@ class Mutator {
   explicit Mutator(const SkMatrix& matrix)
       : type_(kTransform), matrix_(matrix) {}
   explicit Mutator(const int& alpha) : type_(kOpacity), alpha_(alpha) {}
-  explicit Mutator(std::shared_ptr<const DlImageFilter> filter)
-      : type_(kBackdropFilter), filter_(filter) {}
+  explicit Mutator(std::shared_ptr<const DlImageFilter> filter,
+                   const SkRect& filter_rect)
+      : type_(kBackdropFilter),
+        filter_(std::make_shared<ImageFilterMutation>(filter, filter_rect)) {}
 
   const MutatorType& GetType() const { return type_; }
   const SkRect& GetRect() const { return rect_; }
   const SkRRect& GetRRect() const { return rrect_; }
   const SkPath& GetPath() const { return *path_; }
   const SkMatrix& GetMatrix() const { return matrix_; }
-  const DlImageFilter& GetFilter() const { return *filter_; }
+  const ImageFilterMutation& GetFilter() const { return *filter_; }
   const int& GetAlpha() const { return alpha_; }
   float GetAlphaFloat() const { return (alpha_ / 255.0); }
 
@@ -132,8 +161,7 @@ class Mutator {
     int alpha_;
   };
 
-  std::shared_ptr<const DlImageFilter> filter_;
-
+  std::shared_ptr<ImageFilterMutation> filter_;
 };  // Mutator
 
 // A stack of mutators that can be applied to an embedded platform view.
@@ -154,7 +182,8 @@ class MutatorsStack {
   void PushClipPath(const SkPath& path);
   void PushTransform(const SkMatrix& matrix);
   void PushOpacity(const int& alpha);
-  void PushBackdropFilter(const std::shared_ptr<const DlImageFilter>& filter);
+  void PushBackdropFilter(const std::shared_ptr<const DlImageFilter>& filter,
+                          const SkRect& filter_rect);
 
   // Removes the `Mutator` on the top of the stack
   // and destroys it.
@@ -249,8 +278,9 @@ class EmbeddedViewParams {
   const SkRect& finalBoundingRect() const { return final_bounding_rect_; }
 
   // Pushes the stored DlImageFilter object to the mutators stack.
-  void PushImageFilter(std::shared_ptr<const DlImageFilter> filter) {
-    mutators_stack_.PushBackdropFilter(filter);
+  void PushImageFilter(std::shared_ptr<const DlImageFilter> filter,
+                       const SkRect& filter_rect) {
+    mutators_stack_.PushBackdropFilter(filter, filter_rect);
   }
 
   // Whether the embedder should construct DisplayList objects to hold the
@@ -454,7 +484,8 @@ class ExternalViewEmbedder {
   // See also: |PushVisitedPlatformView| for pushing platform view ids to the
   // visited platform views list.
   virtual void PushFilterToVisitedPlatformViews(
-      std::shared_ptr<const DlImageFilter> filter) {}
+      std::shared_ptr<const DlImageFilter> filter,
+      const SkRect& filter_rect) {}
 
  private:
   bool used_this_frame_ = false;

flow/layers/backdrop_filter_layer.cc

@@ -44,7 +44,8 @@ void BackdropFilterLayer::Preroll(PrerollContext* context,
   Layer::AutoPrerollSaveLayerState save =
       Layer::AutoPrerollSaveLayerState::Create(context, true, bool(filter_));
   if (context->view_embedder != nullptr) {
-    context->view_embedder->PushFilterToVisitedPlatformViews(filter_);
+    context->view_embedder->PushFilterToVisitedPlatformViews(
+        filter_, context->cull_rect);
   }
   SkRect child_paint_bounds = SkRect::MakeEmpty();
   PrerollChildren(context, matrix, &child_paint_bounds);

flow/mutators_stack_unittests.cc

@@ -94,14 +94,14 @@ TEST(MutatorsStack, PushBackdropFilter) {
   const int num_of_mutators = 10;
   for (int i = 0; i < num_of_mutators; i++) {
     auto filter = std::make_shared<DlBlurImageFilter>(i, 5, DlTileMode::kClamp);
-    stack.PushBackdropFilter(filter);
+    stack.PushBackdropFilter(filter, SkRect::MakeEmpty());
   }
 
   auto iter = stack.Begin();
   int i = 0;
   while (iter != stack.End()) {
     ASSERT_EQ(iter->get()->GetType(), MutatorType::kBackdropFilter);
-    ASSERT_EQ(iter->get()->GetFilter().asBlur()->sigma_x(), i);
+    ASSERT_EQ(iter->get()->GetFilter().GetFilter().asBlur()->sigma_x(), i);
     ++iter;
     ++i;
   }
@@ -164,7 +164,7 @@ TEST(MutatorsStack, Equality) {
   int alpha = 240;
   stack.PushOpacity(alpha);
   auto filter = std::make_shared<DlBlurImageFilter>(5, 5, DlTileMode::kClamp);
-  stack.PushBackdropFilter(filter);
+  stack.PushBackdropFilter(filter, SkRect::MakeEmpty());
 
   MutatorsStack stack_other;
   SkMatrix matrix_other = SkMatrix::Scale(1, 1);
@@ -179,7 +179,7 @@ TEST(MutatorsStack, Equality) {
   stack_other.PushOpacity(other_alpha);
   auto other_filter =
       std::make_shared<DlBlurImageFilter>(5, 5, DlTileMode::kClamp);
-  stack_other.PushBackdropFilter(other_filter);
+  stack_other.PushBackdropFilter(other_filter, SkRect::MakeEmpty());
 
   ASSERT_TRUE(stack == stack_other);
 }
@@ -211,9 +211,9 @@ TEST(Mutator, Initialization) {
   ASSERT_TRUE(mutator5.GetType() == MutatorType::kOpacity);
 
   auto filter = std::make_shared<DlBlurImageFilter>(5, 5, DlTileMode::kClamp);
-  Mutator mutator6 = Mutator(filter);
+  Mutator mutator6 = Mutator(filter, SkRect::MakeEmpty());
   ASSERT_TRUE(mutator6.GetType() == MutatorType::kBackdropFilter);
-  ASSERT_TRUE(mutator6.GetFilter() == *filter);
+  ASSERT_TRUE(mutator6.GetFilter().GetFilter() == *filter);
 }
 
 TEST(Mutator, CopyConstructor) {
@@ -244,7 +244,7 @@ TEST(Mutator, CopyConstructor) {
   ASSERT_TRUE(mutator5 == copy5);
 
   auto filter = std::make_shared<DlBlurImageFilter>(5, 5, DlTileMode::kClamp);
-  Mutator mutator6 = Mutator(filter);
+  Mutator mutator6 = Mutator(filter, SkRect::MakeEmpty());
   Mutator copy6 = Mutator(mutator6);
   ASSERT_TRUE(mutator6 == copy6);
 }
@@ -277,8 +277,8 @@ TEST(Mutator, Equality) {
   ASSERT_TRUE(mutator5 == other_mutator5);
 
   auto filter = std::make_shared<DlBlurImageFilter>(5, 5, DlTileMode::kClamp);
-  Mutator mutator6 = Mutator(filter);
-  Mutator other_mutator6 = Mutator(filter);
+  Mutator mutator6 = Mutator(filter, SkRect::MakeEmpty());
+  Mutator other_mutator6 = Mutator(filter, SkRect::MakeEmpty());
   ASSERT_TRUE(mutator6 == other_mutator6);
 }
 
@@ -299,8 +299,8 @@ TEST(Mutator, UnEquality) {
   auto filter = std::make_shared<DlBlurImageFilter>(5, 5, DlTileMode::kClamp);
   auto filter2 =
       std::make_shared<DlBlurImageFilter>(10, 10, DlTileMode::kClamp);
-  Mutator mutator3 = Mutator(filter);
-  Mutator other_mutator3 = Mutator(filter2);
+  Mutator mutator3 = Mutator(filter, SkRect::MakeEmpty());
+  Mutator other_mutator3 = Mutator(filter2, SkRect::MakeEmpty());
   ASSERT_TRUE(mutator3 != other_mutator3);
 }
 

shell/common/shell_test_external_view_embedder.cc

@@ -89,10 +89,11 @@ void ShellTestExternalViewEmbedder::PushVisitedPlatformView(int64_t view_id) {
 
 // |ExternalViewEmbedder|
 void ShellTestExternalViewEmbedder::PushFilterToVisitedPlatformViews(
-    std::shared_ptr<const DlImageFilter> filter) {
+    std::shared_ptr<const DlImageFilter> filter,
+    const SkRect& filter_rect) {
   for (int64_t id : visited_platform_views_) {
     EmbeddedViewParams params = current_composition_params_[id];
-    params.PushImageFilter(filter);
+    params.PushImageFilter(filter, filter_rect);
     current_composition_params_[id] = params;
     mutators_stacks_[id] = params.mutatorsStack();
   }

shell/common/shell_test_external_view_embedder.h

@@ -76,7 +76,8 @@ class ShellTestExternalViewEmbedder final : public ExternalViewEmbedder {
 
   // |ExternalViewEmbedder|
   void PushFilterToVisitedPlatformViews(
-      std::shared_ptr<const DlImageFilter> filter) override;
+      std::shared_ptr<const DlImageFilter> filter,
+      const SkRect& filter_rect) override;
 
   // |ExternalViewEmbedder|
   void SubmitFrame(GrDirectContext* context,

shell/common/shell_unittests.cc

@@ -844,7 +844,7 @@ TEST_F(ShellTest, PushBackdropFilterToVisitedPlatformViews) {
   auto filter = DlBlurImageFilter(5, 5, DlTileMode::kClamp);
   auto mutator = *external_view_embedder->GetStack(50).Begin();
   ASSERT_EQ(mutator->GetType(), MutatorType::kBackdropFilter);
-  ASSERT_EQ(mutator->GetFilter(), filter);
+  ASSERT_EQ(mutator->GetFilter().GetFilter(), filter);
 
   DestroyShell(std::move(shell));
 }

shell/platform/darwin/ios/framework/Source/FlutterPlatformViews.mm

@@ -321,10 +321,11 @@ - (BOOL)flt_hasFirstResponderInViewHierarchySubtree {
 }
 
 void FlutterPlatformViewsController::PushFilterToVisitedPlatformViews(
-    std::shared_ptr<const DlImageFilter> filter) {
+    std::shared_ptr<const DlImageFilter> filter,
+    const SkRect& filter_rect) {
   for (int64_t id : visited_platform_views_) {
     EmbeddedViewParams params = current_composition_params_[id];
-    params.PushImageFilter(filter);
+    params.PushImageFilter(filter, filter_rect);
     current_composition_params_[id] = params;
   }
 }
@@ -425,7 +426,7 @@ - (BOOL)flt_hasFirstResponderInViewHierarchySubtree {
                                CGRectGetWidth(flutter_view.bounds),
                                CGRectGetHeight(flutter_view.bounds))] autorelease];
 
-  NSMutableArray* blurRadii = [[[NSMutableArray alloc] init] autorelease];
+  NSMutableArray* blurFilters = [[[NSMutableArray alloc] init] autorelease];
 
   auto iter = mutators_stack.Begin();
   while (iter != mutators_stack.End()) {
@@ -448,13 +449,24 @@ - (BOOL)flt_hasFirstResponderInViewHierarchySubtree {
         embedded_view.alpha = (*iter)->GetAlphaFloat() * embedded_view.alpha;
         break;
       case kBackdropFilter: {
-        // We only support DlBlurImageFilter for BackdropFilter.
-        if ((*iter)->GetFilter().asBlur() && canApplyBlurBackdrop) {
+        // Only support DlBlurImageFilter for BackdropFilter.
+        if ((*iter)->GetFilter().GetFilter().asBlur() && canApplyBlurBackdrop) {
           // sigma_x is arbitrarily chosen as the radius value because Quartz sets
           // sigma_x and sigma_y equal to each other. DlBlurImageFilter's Tile Mode
           // is not supported in Quartz's gaussianBlur CAFilter, so it is not used
           // to blur the PlatformView.
-          [blurRadii addObject:@((*iter)->GetFilter().asBlur()->sigma_x())];
+          CGFloat blurRadius = (*iter)->GetFilter().GetFilter().asBlur()->sigma_x();
+          CGRect filterRect = flutter::GetCGRectFromSkRect((*iter)->GetFilter().GetFilterRect());
+          // `filterRect` reprents the rect that shouuld be filtered inside the `flutter_view_`.
+          // The `PlatformViewFilter` needs the frame inside the `clipView` that needs to be
+          // filtered.
+          CGRect intersection = CGRectIntersection(filterRect, clipView.frame);
+          if (CGRectContainsRect(clipView.frame, intersection)) {
+            CGRect frameInClipView = [flutter_view_.get() convertRect:intersection toView:clipView];
+            PlatformViewFilter* filter = [[PlatformViewFilter alloc] initWithFrame:frameInClipView
+                                                                        blurRadius:blurRadius];
+            [blurFilters addObject:filter];
+          }
         }
         break;
       }
@@ -463,15 +475,16 @@ - (BOOL)flt_hasFirstResponderInViewHierarchySubtree {
   }
 
   if (canApplyBlurBackdrop) {
-    canApplyBlurBackdrop = [clipView applyBlurBackdropFilters:blurRadii];
+    canApplyBlurBackdrop = [clipView applyBlurBackdropFilters:blurFilters];
   }
 
   // Reverse the offset of the clipView.
   // The clipView's frame includes the final translate of the final transform matrix.
-  // So we need to revese this translate so the platform view can layout at the correct offset.
+  // Thus, this translate needs to be reversed so the platform view can layout at the correct
+  // offset.
   //
-  // Note that we don't apply this transform matrix the clippings because clippings happen on the
-  // mask view, whose origin is always (0,0) to the flutter_view.
+  // Note that the transforms are not applied to the clipping paths because clipping paths happen on
+  // the mask view, whose origin is always (0,0) to the flutter_view.
   CATransform3D reverseTranslate =
       CATransform3DMakeTranslation(-clipView.frame.origin.x, -clipView.frame.origin.y, 0);
   embedded_view.layer.transform = CATransform3DConcat(finalTransform, reverseTranslate);