Address comments

Author: bdero

impeller/entity/entity_pass.cc

@@ -129,13 +129,8 @@ std::optional<Rect> EntityPass::GetElementsCoverage(
         if (backdrop_filter) {
           auto backdrop_coverage = backdrop_filter->GetCoverage({});
           if (backdrop_coverage.has_value()) {
-            backdrop_coverage->origin += accumulated_coverage->origin;
-            if (unfiltered_coverage.has_value()) {
-              unfiltered_coverage =
-                  unfiltered_coverage->Union(*backdrop_coverage);
-            } else {
-              unfiltered_coverage = backdrop_coverage;
-            }
+            unfiltered_coverage =
+                Union(backdrop_coverage.value(), unfiltered_coverage);
           }
         } else {
           VALIDATION_LOG << "The EntityPass backdrop filter proc didn't return "
@@ -166,24 +161,12 @@ std::optional<Rect> EntityPass::GetElementsCoverage(
         element_coverage = unfiltered_coverage;
       }
 
-      if (element_coverage.has_value() && coverage_limit.has_value() &&
-          (!image_filter || image_filter->IsTranslationOnly())) {
-        element_coverage =
-            element_coverage->Intersection(coverage_limit.value());
-      }
+      element_coverage = element_coverage->Intersection(coverage_limit.value());
     } else {
       FML_UNREACHABLE();
     }
 
-    if (!accumulated_coverage.has_value() && element_coverage.has_value()) {
-      accumulated_coverage = element_coverage;
-      continue;
-    }
-    if (!element_coverage.has_value()) {
-      continue;
-    }
-    accumulated_coverage =
-        accumulated_coverage->Union(element_coverage.value());
+    accumulated_coverage = Union(accumulated_coverage, element_coverage);
   }
   return accumulated_coverage;
 }