Fixes to Cast use of overlays for video frames

ui/ozone/platform/cast/overlay_manager_cast.cc

 // Copyright 2015 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 "ui/ozone/platform/cast/overlay_manager_cast.h"
 
+#include "base/bind.h"
+#include "base/location.h"
+#include "base/memory/ref_counted.h"
+#include "base/single_thread_task_runner.h"
+#include "chromecast/browser/media/cma_message_loop.h"
 #include "chromecast/public/cast_media_shlib.h"
 #include "chromecast/public/graphics_types.h"
 #include "chromecast/public/video_plane.h"
 #include "ui/gfx/geometry/rect_conversions.h"
 #include "ui/ozone/public/overlay_candidates_ozone.h"
 
 namespace ui {
 namespace {
 
+void SetVideoPlaneGeometry(
+    const chromecast::RectF& display_rect,
+    chromecast::media::VideoPlane::CoordinateType coordinate_type,
+    chromecast::media::VideoPlane::Transform transform) {
+  chromecast::media::VideoPlane* video_plane =
+      chromecast::media::CastMediaShlib::GetVideoPlane();
+  CHECK(video_plane);
+  video_plane->SetGeometry(display_rect, coordinate_type, transform);
+}
+
 // Translates a gfx::OverlayTransform into a VideoPlane::Transform.
 // Could be just a lookup table once we have unit tests for this code
 // to ensure it stays in sync with OverlayTransform.
 chromecast::media::VideoPlane::Transform ConvertTransform(
     gfx::OverlayTransform transform) {
   switch (transform) {
     case gfx::OVERLAY_TRANSFORM_NONE:
       return chromecast::media::VideoPlane::TRANSFORM_NONE;
     case gfx::OVERLAY_TRANSFORM_FLIP_HORIZONTAL:
       return chromecast::media::VideoPlane::FLIP_HORIZONTAL;
     case gfx::OVERLAY_TRANSFORM_FLIP_VERTICAL:
       return chromecast::media::VideoPlane::FLIP_VERTICAL;
     case gfx::OVERLAY_TRANSFORM_ROTATE_90:
       return chromecast::media::VideoPlane::ROTATE_90;
     case gfx::OVERLAY_TRANSFORM_ROTATE_180:
       return chromecast::media::VideoPlane::ROTATE_180;
     case gfx::OVERLAY_TRANSFORM_ROTATE_270:
       return chromecast::media::VideoPlane::ROTATE_270;
     default:
       NOTREACHED();
       return chromecast::media::VideoPlane::TRANSFORM_NONE;
   }
 }
 
+bool ExactlyEqual(const chromecast::RectF& r1, const chromecast::RectF& r2) {
+  return r1.x == r2.x && r1.y == r2.y && r1.width == r2.width &&
+         r1.height == r2.height;
+}
+
 class OverlayCandidatesCast : public OverlayCandidatesOzone {
  public:
+  OverlayCandidatesCast()
+      : cma_task_runner_(chromecast::media::CmaMessageLoop::GetTaskRunner()),
+        transform_(gfx::OVERLAY_TRANSFORM_INVALID),
+        display_rect_(0, 0, 0, 0) {}
+
   void CheckOverlaySupport(OverlaySurfaceCandidateList* surfaces) override {
     for (auto& candidate : *surfaces) {
       if (candidate.plane_z_order == -1) {
         candidate.overlay_handled = true;
 
         // Compositor requires all overlay rectangles to have integer coords
         candidate.display_rect = gfx::ToEnclosedRect(candidate.display_rect);
 
-        chromecast::media::VideoPlane* video_plane =
-            chromecast::media::CastMediaShlib::GetVideoPlane();
-
         chromecast::RectF display_rect(
             candidate.display_rect.x(), candidate.display_rect.y(),
             candidate.display_rect.width(), candidate.display_rect.height());
-        video_plane->SetGeometry(
-            display_rect,
-            chromecast::media::VideoPlane::COORDINATE_TYPE_GRAPHICS_PLANE,
-            ConvertTransform(candidate.transform));
+
+        // Update video plane geometry + transform to match compositor quad.
+        // This must be done on CMA thread - and no point doing if it hasn't
+        // changed.
+        if (candidate.transform != transform_ ||
+            !ExactlyEqual(display_rect, display_rect_)) {
+          transform_ = candidate.transform;
+          display_rect_ = display_rect;
+
+          cma_task_runner_->PostTask(
+              FROM_HERE,
+              base::Bind(
+                  &SetVideoPlaneGeometry, display_rect,
+                  chromecast::media::VideoPlane::COORDINATE_TYPE_GRAPHICS_PLANE,
+                  ConvertTransform(candidate.transform)));
+        }
         return;
       }
     }
   }
+
+ private:
+  scoped_refptr<base::SingleThreadTaskRunner> cma_task_runner_;
+  gfx::OverlayTransform transform_;
+  chromecast::RectF display_rect_;
 };
 
 }  // namespace
 
 OverlayManagerCast::OverlayManagerCast() {
 }
 
 OverlayManagerCast::~OverlayManagerCast() {
 }
 
 scoped_ptr<OverlayCandidatesOzone> OverlayManagerCast::CreateOverlayCandidates(
     gfx::AcceleratedWidget w) {
   return make_scoped_ptr(new OverlayCandidatesCast());
 }
 
 bool OverlayManagerCast::CanShowPrimaryPlaneAsOverlay() {
   return false;
 }
 
 }  // namespace ui