Media Remoting: Draw remoting interstitial on poster image.

media/remoting/remoting_interstitial_ui.cc

 // Copyright 2016 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 "media/remoting/remoting_interstitial_ui.h"
 
 #include "media/base/media_resources.h"
 #include "media/base/video_frame.h"
 #include "media/base/video_renderer_sink.h"
 #include "media/base/video_util.h"
 #include "third_party/skia/include/core/SkCanvas.h"
 #include "third_party/skia/include/core/SkTypeface.h"
 #include "third_party/skia/include/effects/SkBlurImageFilter.h"
 #include "ui/gfx/color_palette.h"
-#include "ui/gfx/geometry/size.h"
 #include "ui/gfx/paint_vector_icon.h"
 #include "ui/gfx/skbitmap_operations.h"
 #include "ui/gfx/vector_icons_public.h"
 
 namespace media {
 
 namespace {
 
-gfx::Size GetRotatedVideoSize(VideoRotation rotation, gfx::Size natural_size) {
-  if (rotation == VIDEO_ROTATION_90 || rotation == VIDEO_ROTATION_270)
-    return gfx::Size(natural_size.height(), natural_size.width());
-  return natural_size;
-}
+scoped_refptr<VideoFrame> GetInterstitial(const SkBitmap& background_image,
+                                          const gfx::Size& canvas_size,
+                                          bool is_remoting_successful) {
+  SkBitmap modified_bitmap;
+  modified_bitmap.allocN32Pixels(canvas_size.width(), canvas_size.height());
+  modified_bitmap.eraseColor(SK_ColorBLACK);
+  SkCanvas canvas(modified_bitmap);
 
-}  // namespace
+  // Draw |background_image| on the canvas.
+  if (!background_image.drawsNothing()) {
+    color_utils::HSL shift = {-1, 0, 0.2};  // Make monochromatic.
+    SkBitmap monochromatic_image =
+        SkBitmapOperations::CreateHSLShiftedBitmap(background_image, shift);
 
-RemotingInterstitialUI::RemotingInterstitialUI(
-    VideoRendererSink* video_renderer_sink,
-    const PipelineMetadata& pipeline_metadata)
-    : video_renderer_sink_(video_renderer_sink),
-      pipeline_metadata_(pipeline_metadata) {
-  DCHECK(pipeline_metadata_.has_video);
-  pipeline_metadata_.natural_size = GetRotatedVideoSize(
-      pipeline_metadata_.video_rotation, pipeline_metadata_.natural_size);
-}
-
-RemotingInterstitialUI::~RemotingInterstitialUI() {}
-
-scoped_refptr<VideoFrame> RemotingInterstitialUI::GetInterstitial(
-    const SkBitmap& background_image,
-    bool is_remoting_successful) {
-  const gfx::Size canvas_size =
-      gfx::Size(background_image.width(), background_image.height());
-  DCHECK(canvas_size == pipeline_metadata_.natural_size);
-
-  color_utils::HSL shift = {-1, 0, 0.2};  // Make monochromatic.
-  SkBitmap modified_bitmap =
-      SkBitmapOperations::CreateHSLShiftedBitmap(background_image, shift);
-
-  SkCanvas canvas(modified_bitmap);
+    const gfx::Size image_size =
+        gfx::Size(background_image.width(), background_image.height());
+    if (image_size != canvas_size) {
+      // Centered the background image.
+      gfx::Rect centered_rect =
+          ComputeLetterboxRegion(gfx::Rect(canvas_size), image_size);
+      canvas.writePixels(monochromatic_image, centered_rect.x(),
+                         centered_rect.y());
+    } else {
+      canvas.writePixels(monochromatic_image, 0, 0);
+    }
+  }
 
   // Blur the background image.
   SkScalar sigma = SkDoubleToScalar(10);
   SkPaint paint_blur;
   paint_blur.setImageFilter(SkBlurImageFilter::Make(sigma, sigma, nullptr));
   canvas.saveLayer(0, &paint_blur);
   canvas.restore();
 
   // Create SkPaint for text and icon bitmap.
   // After |paint| draws, the new canvas should look like this:
@@ skipping 47 matching lines @@
       canvas_size, base::TimeDelta());
   modified_bitmap.lockPixels();
   media::CopyRGBToVideoFrame(
       reinterpret_cast<uint8_t*>(modified_bitmap.getPixels()),
       modified_bitmap.rowBytes(),
       gfx::Rect(canvas_size.width(), canvas_size.height()), video_frame.get());
   modified_bitmap.unlockPixels();
   return video_frame;
 }
 
-void RemotingInterstitialUI::ShowInterstitial(bool is_remoting_successful) {
-  if (!video_renderer_sink_)
-    return;
+}  // namespace
 
-  // TODO(xjz): Provide poster image, if available, rather than a blank, black
-  // image.
-  SkBitmap background_image;
-  const gfx::Size size = pipeline_metadata_.natural_size;
-  background_image.allocN32Pixels(size.width(), size.height());
-  background_image.eraseColor(SK_ColorBLACK);
+void ShowInterstitialOnSink(VideoRendererSink* const video_renderer_sink,
+                            const SkBitmap& background_image,
+                            const gfx::Size& canvas_size,
+                            bool is_remoting_successful) {
+  DCHECK(!canvas_size.IsEmpty());

[miu, 2016/12/03 01:05:31]

I'm trying to remember if this can happen. Maybe a web page could, legally, set
the video element's width/height to 0x0. So, should this DCHECK be replaced
with:

  if (canvas_size.IsEmpty())
    return;

[xjz, 2016/12/06 19:50:55]

Add the check before this function is called.

+  DCHECK(video_renderer_sink);
 
   const scoped_refptr<VideoFrame> interstitial =
-      GetInterstitial(background_image, is_remoting_successful);
+      GetInterstitial(background_image, canvas_size, is_remoting_successful);
   if (!interstitial)
     return;
 
-  video_renderer_sink_->PaintSingleFrame(interstitial);
+  video_renderer_sink->PaintSingleFrame(interstitial);
 }
 
 }  // namespace media