WIP - Add EME

media/remoting/remoting_renderer_factory.cc

Index: media/remoting/remoting_renderer_factory.cc
diff --git a/media/remoting/remoting_renderer_factory.cc b/media/remoting/remoting_renderer_factory.cc
index 52a5db8859da5c087c7f4931e1f5b903e5afde1c..c6e7a742b4d6b2a87cc5da86fc4f67f2b86b5a29 100644
--- a/media/remoting/remoting_renderer_factory.cc
+++ b/media/remoting/remoting_renderer_factory.cc
@@ -10,7 +10,7 @@ namespace media {
 
 RemotingRendererFactory::RemotingRendererFactory(
     std::unique_ptr<RendererFactory> default_renderer_factory,
-    std::unique_ptr<RemotingController> remoting_controller)
+    std::unique_ptr<RemotingRendererController> remoting_controller)
     : default_renderer_factory_(std::move(default_renderer_factory)),
       remoting_controller_(std::move(remoting_controller)) {}
 
@@ -22,7 +22,8 @@ std::unique_ptr<Renderer> RemotingRendererFactory::CreateRenderer(
     AudioRendererSink* audio_renderer_sink,
     VideoRendererSink* video_renderer_sink,
     const RequestSurfaceCB& request_surface_cb) {
-  if (remoting_controller_ && remoting_controller_->is_remoting()) {
+  if (remoting_controller_ && (remoting_controller_->IsRemoting() ||
+                               remoting_controller_->IsTerminated())) {

[miu, 2016/10/25 04:21:26]

Seems like this should be:

  if (remoting_controller_ &&
      remoting_controller_->IsRemoting() &&
      !remoting_controller_->IsTerminated()) {

Rather than having two separate IsRemoting() and IsTerminated() methods, you
might just combine them and rename to something like IsRenderingRemotely().

[xjz, 2016/10/26 22:00:27]

Renamed IsRemoting() to IsRenderingRemotely(). I originally thought that we
should switch to remoting renderer if remoting session is started or is
permanently terminated. On a second thought, I think for the latter, we may just
create a separate renderer to only render that failure page (with apacible's
change). So now I separated the two cases. WDYT?

     VLOG(1) << "Create Remoting renderer.";
     // TODO(xjz): Merge this with Eric's implementation.
     NOTIMPLEMENTED();