[Impeller] Reorder blend filter checks to avoid unnecessary destination snapshot

Author: bdero

impeller/entity/contents/filters/blend_filter_contents.cc

@@ -357,21 +357,10 @@ std::optional<Entity> BlendFilterContents::CreateForegroundPorterDuffBlend(
     BlendMode blend_mode,
     std::optional<Scalar> alpha,
     bool absorb_opacity) const {
-  auto dst_snapshot =
-      input->GetSnapshot("ForegroundPorterDuffBlend", renderer, entity);
-  if (!dst_snapshot.has_value()) {
-    return std::nullopt;
-  }
-
   if (blend_mode == BlendMode::kClear) {
     return std::nullopt;
   }
 
-  if (blend_mode == BlendMode::kDestination) {
-    return Entity::FromSnapshot(dst_snapshot, entity.GetBlendMode(),
-                                entity.GetStencilDepth());
-  }
-
   if (blend_mode == BlendMode::kSource) {
     auto contents = std::make_shared<SolidColorContents>();
     contents->SetGeometry(Geometry::MakeRect(coverage));
@@ -384,6 +373,17 @@ std::optional<Entity> BlendFilterContents::CreateForegroundPorterDuffBlend(
     return foreground_entity;
   }
 
+  auto dst_snapshot =
+      input->GetSnapshot("ForegroundPorterDuffBlend", renderer, entity);
+  if (!dst_snapshot.has_value()) {
+    return std::nullopt;
+  }
+
+  if (blend_mode == BlendMode::kDestination) {
+    return Entity::FromSnapshot(dst_snapshot, entity.GetBlendMode(),
+                                entity.GetStencilDepth());
+  }
+
   RenderProc render_proc = [foreground_color, coverage, dst_snapshot,
                             blend_mode, absorb_opacity, alpha](
                                const ContentContext& renderer,