Media Remoting: Draw remoting interstitial on poster image.

media/remoting/remoting_renderer_controller.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_RENDERER_CONTROLLER_H_
 #define MEDIA_REMOTING_REMOTING_RENDERER_CONTROLLER_H_
 
 #include "base/callback.h"
 #include "base/memory/weak_ptr.h"
 #include "media/base/media_observer.h"
 #include "media/remoting/remoting_source_impl.h"
+#include "third_party/skia/include/core/SkBitmap.h"
 
 namespace media {
 
 namespace remoting {
 class RpcBroker;
 }
 
 // This class:
 // 1) Implements the RemotingSourceImpl::Client;
 // 2) Monitors player events as a MediaObserver;
 // 3) May trigger the switch of the media renderer between local playback
 // and remoting.
 class RemotingRendererController final : public RemotingSourceImpl::Client,
                                          public MediaObserver {
  public:
   explicit RemotingRendererController(
       scoped_refptr<RemotingSourceImpl> remoting_source);
   ~RemotingRendererController() override;
 
   // RemotingSourceImpl::Client implemenations.
   void OnStarted(bool success) override;
   void OnSessionStateChanged() override;
 
   // MediaObserver implementations.
   void OnEnteredFullscreen() override;
   void OnExitedFullscreen() override;
   void OnSetCdm(CdmContext* cdm_context) override;
   void OnMetadataChanged(const PipelineMetadata& metadata) override;
+  void OnSetPoster(const GURL& poster) override;
 
   void SetSwitchRendererCallback(const base::Closure& cb);
 
+  using PosterDownloadedCallback = base::Callback<void(const SkBitmap& bitmap)>;
+  using DownloadPosterCallback =
+      base::Callback<void(const GURL& url, const PosterDownloadedCallback& cb)>;
+  void SetDownloadPosterCallback(const DownloadPosterCallback& cb);
+
   base::WeakPtr<RemotingRendererController> GetWeakPtr() {
     return weak_factory_.GetWeakPtr();
   }
 
   // Used by RemotingRendererFactory to query whether to create Media Remoting
   // Renderer.
   bool remote_rendering_started() const {
     DCHECK(thread_checker_.CalledOnValidThread());
     return remote_rendering_started_;
   }
 
   void StartDataPipe(
       std::unique_ptr<mojo::DataPipe> audio_data_pipe,
       std::unique_ptr<mojo::DataPipe> video_data_pipe,
       const RemotingSourceImpl::DataPipeStartCallback& done_callback);
 
   // Used by RemotingRendererImpl to query the session state.
   RemotingSourceImpl* remoting_source() const {
     DCHECK(thread_checker_.CalledOnValidThread());
     return remoting_source_.get();
   }
 
   base::WeakPtr<remoting::RpcBroker> GetRpcBroker() const;
 
   PipelineMetadata pipeline_metadata() const {
     DCHECK(thread_checker_.CalledOnValidThread());
     return pipeline_metadata_;
   }
 
+  GURL poster() const {

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

Rather than exposing poster() and download_poster_cb(), let's have the
controller be responsible for handling all this, and just give the UI the
SkBitmap it needs whenever the poster URL changes.

Some quick reading:
https://en.wikipedia.org/wiki/Model%E2%80%93view%E2%80%93controller (In our
case, the model and controller are merged together as
RemotingRendererController, and that's okay at this scale.)

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

Done.

+    DCHECK(thread_checker_.CalledOnValidThread());
+    return poster_;
+  }
+
+  DownloadPosterCallback download_poster_cb() const {
+    DCHECK(thread_checker_.CalledOnValidThread());
+    return download_poster_cb_;
+  }
+
  private:
   bool has_audio() const {
     return pipeline_metadata_.has_audio &&
            pipeline_metadata_.audio_decoder_config.IsValidConfig();
   }
 
   bool has_video() const {
     return pipeline_metadata_.has_video &&
            pipeline_metadata_.video_decoder_config.IsValidConfig();
   }
@@ skipping 27 matching lines @@
   // This is initially the RemotingSourceImpl passed to the ctor, and might be
   // replaced with a different instance later if OnSetCdm() is called.
   scoped_refptr<RemotingSourceImpl> remoting_source_;
 
   // This is used to check all the methods are called on the current thread in
   // debug builds.
   base::ThreadChecker thread_checker_;
 
   PipelineMetadata pipeline_metadata_;
 
+  GURL poster_;
+
+  DownloadPosterCallback download_poster_cb_;
+
   base::WeakPtrFactory<RemotingRendererController> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(RemotingRendererController);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_REMOTING_REMOTING_RENDERER_CONTROLLER_H_