Mac Overlays: Allow SolidColor and Tile quads to be candidates for overlays.

cc/output/overlay_candidate.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/output/overlay_candidate.h"
 
 #include <algorithm>
 #include <limits>
 #include "base/logging.h"
 #include "cc/base/math_util.h"
 #include "cc/quads/io_surface_draw_quad.h"
 #include "cc/quads/solid_color_draw_quad.h"
 #include "cc/quads/stream_video_draw_quad.h"
 #include "cc/quads/texture_draw_quad.h"
+#include "cc/quads/tile_draw_quad.h"
 #include "ui/gfx/geometry/rect_conversions.h"
 #include "ui/gfx/geometry/vector3d_f.h"
 
 namespace cc {
 
 namespace {
 // Tolerance for considering axis vector elements to be zero.
 const SkMScalar kEpsilon = std::numeric_limits<float>::epsilon();
 
 enum Axis { NONE, AXIS_POS_X, AXIS_NEG_X, AXIS_POS_Y, AXIS_NEG_Y };
@@ skipping 140 matching lines @@
 
 }  // namespace
 
 OverlayCandidate::OverlayCandidate()
     : transform(gfx::OVERLAY_TRANSFORM_NONE),
       format(RGBA_8888),
       uv_rect(0.f, 0.f, 1.f, 1.f),
       is_clipped(false),
       use_output_surface_for_resource(false),
       resource_id(0),
+      solid_color(SK_ColorTRANSPARENT),
       plane_z_order(0),
       overlay_handled(false) {}
 
 OverlayCandidate::~OverlayCandidate() {}
 
 // static
+bool OverlayCandidate::IsAllowedQuad(const DrawQuad* quad) {
+  switch (quad->material) {
+    case DrawQuad::TEXTURE_CONTENT:
+      return TextureDrawQuad::MaterialCast(quad)->allow_overlay();
+    case DrawQuad::STREAM_VIDEO_CONTENT:
+      return StreamVideoDrawQuad::MaterialCast(quad)->allow_overlay();
+    case DrawQuad::IO_SURFACE_CONTENT:
+      return IOSurfaceDrawQuad::MaterialCast(quad)->allow_overlay;
+    default:
+      break;
+  }
+  return false;
+}
+
+// static
 bool OverlayCandidate::FromDrawQuad(const DrawQuad* quad,
                                     OverlayCandidate* candidate) {
   if (quad->needs_blending || quad->shared_quad_state->opacity != 1.f ||
       quad->shared_quad_state->blend_mode != SkXfermode::kSrcOver_Mode)
     return false;
 
   auto& transform = quad->shared_quad_state->quad_to_target_transform;
   candidate->display_rect = gfx::RectF(quad->rect);
   transform.TransformRect(&candidate->display_rect);
   candidate->quad_rect_in_target_space =
       MathUtil::MapEnclosingClippedRect(transform, quad->rect);
 
   candidate->format = RGBA_8888;
   candidate->clip_rect = quad->shared_quad_state->clip_rect;
   candidate->is_clipped = quad->shared_quad_state->is_clipped;
 
   switch (quad->material) {
     case DrawQuad::TEXTURE_CONTENT:
       return FromTextureQuad(TextureDrawQuad::MaterialCast(quad), candidate);
     case DrawQuad::STREAM_VIDEO_CONTENT:
       return FromStreamVideoQuad(StreamVideoDrawQuad::MaterialCast(quad),
                                  candidate);
     case DrawQuad::IO_SURFACE_CONTENT:
       return FromIOSurfaceQuad(IOSurfaceDrawQuad::MaterialCast(quad),
                                candidate);

[ccameron, 2015/09/30 18:52:09]

We're also going to want to make sure that things that derive from
OverlayStrategyCommon don't attempt these quad types (because if they do,
they'll hit the O(n^2) list modification, along with unknown performance
characteristics, etc).

[Andre, 2015/09/30 21:15:15]

It's currently using OverlayCandidate::IsAllowedQuad() to filter it out in
OverlayStrategCommon.
But will need a better way or name if we add allow_overlay to TileDrawQuad.

+    case DrawQuad::TILED_CONTENT:
+      return FromTileQuad(TileDrawQuad::MaterialCast(quad), candidate);
+    case DrawQuad::SOLID_COLOR:
+      return FromSolidColorQuad(SolidColorDrawQuad::MaterialCast(quad),
+                                candidate);
     default:
       break;
   }
 
   return false;
 }
 
 // static
 bool OverlayCandidate::FromTextureQuad(const TextureDrawQuad* quad,
                                        OverlayCandidate* candidate) {
-  if (!quad->allow_overlay())
-    return false;
   gfx::OverlayTransform overlay_transform = GetOverlayTransform(
       quad->shared_quad_state->quad_to_target_transform, quad->y_flipped);
   if (quad->background_color != SK_ColorTRANSPARENT ||
       quad->premultiplied_alpha ||
       overlay_transform == gfx::OVERLAY_TRANSFORM_INVALID)
     return false;
   candidate->resource_id = quad->resource_id();
   candidate->resource_size_in_pixels = quad->resource_size_in_pixels();
   candidate->transform = overlay_transform;
   candidate->uv_rect = BoundingRect(quad->uv_top_left, quad->uv_bottom_right);
   return true;
 }
 
 // static
 bool OverlayCandidate::FromStreamVideoQuad(const StreamVideoDrawQuad* quad,
                                            OverlayCandidate* candidate) {
-  if (!quad->allow_overlay())
-    return false;
   gfx::OverlayTransform overlay_transform = GetOverlayTransform(
       quad->shared_quad_state->quad_to_target_transform, false);
   if (overlay_transform == gfx::OVERLAY_TRANSFORM_INVALID)
     return false;
   if (!quad->matrix.IsScaleOrTranslation()) {
     // We cannot handle anything other than scaling & translation for texture
     // coordinates yet.
     return false;
   }
   candidate->resource_id = quad->resource_id();
@@ skipping 23 matching lines @@
     candidate->transform = ComposeTransforms(
         gfx::OVERLAY_TRANSFORM_FLIP_VERTICAL, candidate->transform);
     candidate->uv_rect = gfx::RectF(uv0.x(), uv0.y(), delta.x(), delta.y());
   }
   return true;
 }
 
 // static
 bool OverlayCandidate::FromIOSurfaceQuad(const IOSurfaceDrawQuad* quad,
                                          OverlayCandidate* candidate) {
-  if (!quad->allow_overlay)
-    return false;
   gfx::OverlayTransform overlay_transform = GetOverlayTransform(
       quad->shared_quad_state->quad_to_target_transform, false);
   if (overlay_transform != gfx::OVERLAY_TRANSFORM_NONE)
     return false;
   candidate->resource_id = quad->io_surface_resource_id();
   candidate->resource_size_in_pixels = quad->io_surface_size;
   candidate->transform = overlay_transform;
   candidate->uv_rect = gfx::RectF(1.f, 1.f);
   return true;
 }
 
+// static
+bool OverlayCandidate::FromTileQuad(const TileDrawQuad* quad,
+                                    OverlayCandidate* candidate) {

[ccameron, 2015/09/30 18:52:09]

We're going to need to plumb through a "allow_overlay" flag here, because not
all resources are overlay-able (e.g, for now, CPU raster is possible, but GPU
raster isn't). 

[Andre, 2015/09/30 20:57:17]

I'm not sure how to do this.
I think we want to TileDrawQuad to allow_overlay if its resource has a GLImage.
The tile quads are created in PictureLayerImpl::AppendQuads(). There it has
access to a TileDrawInfo and resource_id.
reveman, how do you think this should be done?

+  candidate->transform = GetOverlayTransform(
+      quad->shared_quad_state->quad_to_target_transform, false);
+  if (candidate->transform != gfx::OVERLAY_TRANSFORM_NONE)
+    return false;
+  candidate->resource_id = quad->resource_id();
+  candidate->resource_size_in_pixels = quad->texture_size;
+  candidate->uv_rect = quad->tex_coord_rect;
+  candidate->uv_rect.Scale(1.f / quad->texture_size.width(),
+                           1.f / quad->texture_size.height());
+  return true;
+}
+
+// static
+bool OverlayCandidate::FromSolidColorQuad(const SolidColorDrawQuad* quad,
+                                          OverlayCandidate* candidate) {
+  candidate->transform = GetOverlayTransform(
+      quad->shared_quad_state->quad_to_target_transform, false);
+  if (candidate->transform != gfx::OVERLAY_TRANSFORM_NONE)
+    return false;
+  candidate->solid_color = quad->color;
+  return true;
+}
+
 }  // namespace cc