added coverage fix

Author: gaaclarke

impeller/entity/BUILD.gn

@@ -271,11 +271,8 @@ impeller_component("entity_unittests") {
   testonly = true
 
   sources = [
-<<<<<<< HEAD
     "contents/checkerboard_contents_unittests.cc",
-=======
     "contents/filters/directional_gaussian_blur_filter_contents_unittests.cc",
->>>>>>> 8b53b77552 (added directional_gaussian unittests to help)
     "contents/filters/gaussian_blur_filter_contents_unittests.cc",
     "contents/filters/inputs/filter_input_unittests.cc",
     "entity_playground.cc",

impeller/entity/contents/filters/directional_gaussian_blur_filter_contents.cc

@@ -313,7 +313,9 @@ std::optional<Rect> DirectionalGaussianBlurFilterContents::GetFilterCoverage(
 
   auto transform = inputs[0]->GetTransform(entity) * effect_transform.Basis();
   auto transformed_blur_vector =
-      transform.TransformDirection(blur_direction_ * Radius{blur_sigma_}.radius)
+      transform
+          .TransformDirection(blur_direction_ *
+                              Radius{ScaleSigma(blur_sigma_)}.radius)
           .Abs();
   return coverage->Expand(transformed_blur_vector);
 }

impeller/entity/contents/filters/directional_gaussian_blur_filter_contents_unittests.cc

@@ -89,6 +89,7 @@ class DirectionalGaussianBlurFilterContentsTest : public ::testing::Test {
         .WillRepeatedly(Invoke(([](const TextureDescriptor& desc) {
           auto result = std::make_shared<MockTexture>(desc);
           EXPECT_CALL(*result, IsValid()).WillRepeatedly(Return(true));
+          EXPECT_CALL(*result, GetSize()).WillRepeatedly(Return(desc.size));
           return result;
         })));
   }
@@ -174,11 +175,22 @@ TEST_F(DirectionalGaussianBlurFilterContentsTest, RenderSomething) {
   ContentContext renderer(mock_context_, mock_typographer_context_,
                           mock_render_target_allocator_);
   Entity entity;
-  Rect coverage_hint = Rect::MakeLTRB(0, 0, 0, 0);
   std::optional<Entity> result =
-      contents->GetEntity(renderer, entity, coverage_hint);
+      contents->GetEntity(renderer, entity, /*coverage_hint=*/{});
   ASSERT_TRUE(result.has_value());
-  ASSERT_EQ(result.value().GetBlendMode(), BlendMode::kSourceOver);
+  EXPECT_EQ(result.value().GetBlendMode(), BlendMode::kSourceOver);
+  std::optional<Rect> result_coverage = result.value().GetCoverage();
+  std::optional<Rect> contents_coverage = contents->GetCoverage(entity);
+  EXPECT_TRUE(result_coverage.has_value());
+  EXPECT_TRUE(contents_coverage.has_value());
+  EXPECT_NEAR(result_coverage.value().GetLeft(),
+              contents_coverage.value().GetLeft(), kEhCloseEnough);
+  EXPECT_NEAR(result_coverage.value().GetTop(),
+              contents_coverage.value().GetTop(), kEhCloseEnough);
+  EXPECT_NEAR(result_coverage.value().GetRight(),
+              contents_coverage.value().GetRight(), kEhCloseEnough);
+  EXPECT_NEAR(result_coverage.value().GetBottom(),
+              contents_coverage.value().GetBottom(), kEhCloseEnough);
 }
 
 }  // namespace testing