Revert "Add GetBoundingRectAfterMutations to EmbeddedViewParams to calculate the final bounding rect for platform view (#19170)" (#19204)

This reverts commit f5c315f.

Author: brianosman

ci/licenses_golden/licenses_flutter

@@ -26,7 +26,6 @@ FILE: ../../../flutter/common/task_runners.cc
 FILE: ../../../flutter/common/task_runners.h
 FILE: ../../../flutter/flow/compositor_context.cc
 FILE: ../../../flutter/flow/compositor_context.h
-FILE: ../../../flutter/flow/embedded_view_params_unittests.cc
 FILE: ../../../flutter/flow/embedded_views.cc
 FILE: ../../../flutter/flow/embedded_views.h
 FILE: ../../../flutter/flow/gl_context_switch.cc

flow/BUILD.gn

@@ -130,7 +130,6 @@ executable("flow_unittests") {
   testonly = true
 
   sources = [
-    "embedded_view_params_unittests.cc",
     "flow_run_all_unittests.cc",
     "flow_test_utils.cc",
     "flow_test_utils.h",

flow/embedded_view_params_unittests.cc

@@ -186,49 +186,21 @@ class EmbeddedViewParams {
  public:
   EmbeddedViewParams() = default;
 
-  EmbeddedViewParams(SkMatrix matrix,
-                     SkSize size_points,
-                     MutatorsStack mutators_stack)
-      : matrix_(matrix),
-        size_points_(size_points),
-        mutators_stack_(mutators_stack) {
-    SkPath path;
-    SkRect starting_rect = SkRect::MakeSize(size_points);
-    path.addRect(starting_rect);
-    path.transform(matrix);
-    final_bounding_rect_ = path.computeTightBounds();
-  }
-
   EmbeddedViewParams(const EmbeddedViewParams& other) {
-    size_points_ = other.size_points_;
-    mutators_stack_ = other.mutators_stack_;
-    matrix_ = other.matrix_;
-    final_bounding_rect_ = other.final_bounding_rect_;
+    offsetPixels = other.offsetPixels;
+    sizePoints = other.sizePoints;
+    mutatorsStack = other.mutatorsStack;
   };
 
-  // The original size of the platform view before any mutation matrix is
-  // applied.
-  const SkSize& sizePoints() const { return size_points_; };
-  // The mutators stack contains the detailed step by step mutations for this
-  // platform view.
-  const MutatorsStack& mutatorsStack() const { return mutators_stack_; };
-  // The bounding rect of the platform view after applying all the mutations.
-  //
-  // Clippings are ignored.
-  const SkRect& finalBoundingRect() const { return final_bounding_rect_; }
+  SkPoint offsetPixels;
+  SkSize sizePoints;
+  MutatorsStack mutatorsStack;
 
   bool operator==(const EmbeddedViewParams& other) const {
-    return size_points_ == other.size_points_ &&
-           mutators_stack_ == other.mutators_stack_ &&
-           final_bounding_rect_ == other.final_bounding_rect_ &&
-           matrix_ == other.matrix_;
+    return offsetPixels == other.offsetPixels &&
+           sizePoints == other.sizePoints &&
+           mutatorsStack == other.mutatorsStack;
   }
-
- private:
-  SkMatrix matrix_;
-  SkSize size_points_;
-  MutatorsStack mutators_stack_;
-  SkRect final_bounding_rect_;
 };
 
 enum class PostPrerollResult { kResubmitFrame, kSuccess };

flow/embedded_views.h

@@ -27,8 +27,11 @@ void PlatformViewLayer::Preroll(PrerollContext* context,
   }
   context->has_platform_view = true;
   std::unique_ptr<EmbeddedViewParams> params =
-      std::make_unique<EmbeddedViewParams>(matrix, size_,
-                                           context->mutators_stack);
+      std::make_unique<EmbeddedViewParams>();
+  params->offsetPixels =
+      SkPoint::Make(matrix.getTranslateX(), matrix.getTranslateY());
+  params->sizePoints = size_;
+  params->mutatorsStack = context->mutators_stack;
   context->view_embedder->PrerollCompositeEmbeddedView(view_id_,
                                                        std::move(params));
 }

flow/layers/platform_view_layer.cc

@@ -407,14 +407,13 @@
 
 void FlutterPlatformViewsController::CompositeWithParams(int view_id,
                                                          const EmbeddedViewParams& params) {
-  CGRect frame = CGRectMake(0, 0, params.sizePoints().width(), params.sizePoints().height());
+  CGRect frame = CGRectMake(0, 0, params.sizePoints.width(), params.sizePoints.height());
   UIView* touchInterceptor = touch_interceptors_[view_id].get();
   touchInterceptor.layer.transform = CATransform3DIdentity;
   touchInterceptor.frame = frame;
   touchInterceptor.alpha = 1;
 
-  const MutatorsStack& mutatorStack = params.mutatorsStack();
-  int currentClippingCount = CountClips(mutatorStack);
+  int currentClippingCount = CountClips(params.mutatorsStack);
   int previousClippingCount = clip_count_[view_id];
   if (currentClippingCount != previousClippingCount) {
     clip_count_[view_id] = currentClippingCount;
@@ -425,7 +424,7 @@
         ReconstructClipViewsChain(currentClippingCount, touchInterceptor, oldPlatformViewRoot);
     root_views_[view_id] = fml::scoped_nsobject<UIView>([newPlatformViewRoot retain]);
   }
-  ApplyMutators(mutatorStack, touchInterceptor);
+  ApplyMutators(params.mutatorsStack, touchInterceptor);
 }
 
 SkCanvas* FlutterPlatformViewsController::CompositeEmbeddedView(int view_id) {

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

@@ -108,7 +108,7 @@ void EmbedderLayers::PushPlatformViewLayer(
     view.struct_size = sizeof(FlutterPlatformView);
     view.identifier = identifier;
 
-    const auto& mutators = params.mutatorsStack();
+    const auto& mutators = params.mutatorsStack;
 
     std::vector<const FlutterPlatformViewMutation*> mutations_array;
 
@@ -180,10 +180,10 @@ void EmbedderLayers::PushPlatformViewLayer(
   layer.platform_view = platform_views_referenced_.back().get();
 
   const auto layer_bounds =
-      SkRect::MakeXYWH(params.finalBoundingRect().x(),                     //
-                       params.finalBoundingRect().y(),                     //
-                       params.sizePoints().width() * device_pixel_ratio_,  //
-                       params.sizePoints().height() * device_pixel_ratio_  //
+      SkRect::MakeXYWH(params.offsetPixels.x(),                          //
+                       params.offsetPixels.y(),                          //
+                       params.sizePoints.width() * device_pixel_ratio_,  //
+                       params.sizePoints.height() * device_pixel_ratio_  //
       );
 
   const auto transformed_layer_bounds =