Media Remoting: Draw remoting interstitial on poster image.

media/remoting/remoting_interstitial_ui.h

 // 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.
 
 #ifndef MEDIA_REMOTING_REMOTING_INTERSTITIAL_UI_H_
 #define MEDIA_REMOTING_REMOTING_INTERSTITIAL_UI_H_
 
 #include "base/callback.h"
 #include "base/memory/ref_counted.h"
-#include "media/base/pipeline_metadata.h"
+#include "base/memory/weak_ptr.h"
 #include "third_party/skia/include/core/SkBitmap.h"
 
 namespace media {
 
 class VideoFrame;
 class VideoRendererSink;
+class RemotingRendererController;
 
 class RemotingInterstitialUI {
  public:
   RemotingInterstitialUI(VideoRendererSink* video_renderer_sink,
-                         const PipelineMetadata& pipeline_metadata);
+                         const base::WeakPtr<RemotingRendererController>&
+                             remoting_renderer_controller);
   ~RemotingInterstitialUI();
 
   void ShowInterstitial(bool is_remoting_successful);
 
  private:
   // Gets an 'interstitial' VideoFrame to paint on the media player when the
   // video is being played remotely.
   scoped_refptr<VideoFrame> GetInterstitial(const SkBitmap& background_image,
                                             bool is_remoting_successful);
 
+  // Called when poster image is downloaded.
+  void PosterDownloaded(bool is_remoting_successful,

[miu, 2016/11/29 22:56:08]

See comments in remoting_renderer_controller.h/.cc: The act of downloading and
deciding which content to show should be in the controller class. Then, the
controller should just always call the existing ShowInterstitial() method like
it did before, but now ShowInterstitial() should take two arguments: 1) whether
to show the success or fail screen; 2) the SkBitmap background.

[xjz, 2016/12/02 19:23:10]

Done.

+                        const SkBitmap& poster_image);
+
+  // Draw and display the interstitial.
+  void DisplayInterstitial(const SkBitmap& poster_image,
+                           bool is_remoting_successful);
+
   VideoRendererSink* const video_renderer_sink_;  // Outlives this class.
-  PipelineMetadata pipeline_metadata_;
+  const base::WeakPtr<RemotingRendererController> remoting_renderer_controller_;
+
+  base::WeakPtrFactory<RemotingInterstitialUI> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(RemotingInterstitialUI);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_REMOTING_REMOTING_INTERSTITIAL_UI_H_