Don't skip transparent draw calls

Author: bdero

impeller/entity/contents/solid_color_contents.cc

@@ -63,10 +63,6 @@ VertexBuffer SolidColorContents::CreateSolidFillVertices(const Path& path,
 bool SolidColorContents::Render(const ContentContext& renderer,
                                 const Entity& entity,
                                 RenderPass& pass) const {
-  if (color_.IsTransparent()) {
-    return true;
-  }
-
   using VS = SolidFillPipeline::VertexShader;
 
   Command cmd;

impeller/entity/contents/solid_stroke_contents.cc

@@ -35,6 +35,10 @@ void SolidStrokeContents::SetPath(Path path) {
 
 std::optional<Rect> SolidStrokeContents::GetCoverage(
     const Entity& entity) const {
+  if (color_.IsTransparent()) {
+    return std::nullopt;
+  }
+
   auto path_bounds = path_.GetBoundingBox();
   if (!path_bounds.has_value()) {
     return std::nullopt;
@@ -173,7 +177,7 @@ static VertexBuffer CreateSolidStrokeVertices(
 bool SolidStrokeContents::Render(const ContentContext& renderer,
                                  const Entity& entity,
                                  RenderPass& pass) const {
-  if (color_.IsTransparent() || stroke_size_ <= 0.0) {
+  if (stroke_size_ <= 0.0) {
     return true;
   }
 

impeller/entity/contents/texture_contents.cc

@@ -4,6 +4,8 @@
 
 #include "texture_contents.h"
 
+#include <optional>
+
 #include "impeller/entity/contents/content_context.h"
 #include "impeller/entity/entity.h"
 #include "impeller/entity/texture_fill.frag.h"
@@ -35,6 +37,9 @@ void TextureContents::SetOpacity(Scalar opacity) {
 }
 
 std::optional<Rect> TextureContents::GetCoverage(const Entity& entity) const {
+  if (opacity_ == 0) {
+    return std::nullopt;
+  }
   return path_.GetTransformedBoundingBox(entity.GetTransformation());
 };