Move Renderer permanent callbacks into RendererClient interface.

media/mojo/services/mojo_renderer_service.cc

Index: media/mojo/services/mojo_renderer_service.cc
diff --git a/media/mojo/services/mojo_renderer_service.cc b/media/mojo/services/mojo_renderer_service.cc
index ce94a77a5a3150d66c628c0ef74caebdca8779b2..ce49ddddb0f9f5a70df1745d3a916d599a8db98d 100644
--- a/media/mojo/services/mojo_renderer_service.cc
+++ b/media/mojo/services/mojo_renderer_service.cc
@@ -103,19 +103,36 @@ void MojoRendererService::SetCdm(int32_t cdm_id,
                                             weak_this_, cdm, callback));
 }
 
+void MojoRendererService::OnError(PipelineStatus error) {
+  DVLOG(1) << __FUNCTION__ << "(" << error << ")";
+  state_ = STATE_ERROR;
+  client_->OnError();
+}
+
+void MojoRendererService::OnEnded() {
+  DVLOG(1) << __FUNCTION__;
+  CancelPeriodicMediaTimeUpdates();
+  client_->OnEnded();
+}
+
+void MojoRendererService::OnStatisticsUpdate(const PipelineStatistics& stats) {}
+
+void MojoRendererService::OnBufferingStateChange(BufferingState state) {
+  DVLOG(2) << __FUNCTION__ << "(" << state << ")";
+  client_->OnBufferingStateChange(
+      static_cast<interfaces::BufferingState>(state));
+}
+
+void MojoRendererService::OnWaitingForDecryptionKey() {}

[xhwang, 2016/05/09 18:13:22]

This CL makes it super clear that we should plumb OnStatisticsUpdate() and
OnWaitingForDecryptionKey() over the renderer mojom client interface :) Could
you please add a TODO somewhere to make sure we don't forget?

[alokp, 2016/05/09 21:31:43]

Done.

+
 void MojoRendererService::OnStreamReady(
     const mojo::Callback<void(bool)>& callback) {
   DCHECK_EQ(state_, STATE_INITIALIZING);
 
   renderer_->Initialize(
-      stream_provider_.get(),
-      base::Bind(
-          &MojoRendererService::OnRendererInitializeDone, weak_this_, callback),
-      base::Bind(&MojoRendererService::OnUpdateStatistics, weak_this_),
-      base::Bind(&MojoRendererService::OnBufferingStateChanged, weak_this_),
-      base::Bind(&MojoRendererService::OnRendererEnded, weak_this_),
-      base::Bind(&MojoRendererService::OnError, weak_this_),
-      base::Bind(base::DoNothing));
+      this, stream_provider_.get(),
+      base::Bind(&MojoRendererService::OnRendererInitializeDone, weak_this_,
+                 callback));
 }
 
 void MojoRendererService::OnRendererInitializeDone(
@@ -134,9 +151,6 @@ void MojoRendererService::OnRendererInitializeDone(
   callback.Run(true);
 }
 
-void MojoRendererService::OnUpdateStatistics(const PipelineStatistics& stats) {
-}
-
 void MojoRendererService::UpdateMediaTime(bool force) {
   const uint64_t media_time = renderer_->GetMediaTime().InMicroseconds();
   if (!force && media_time == last_media_time_usec_)
@@ -161,25 +175,6 @@ void MojoRendererService::SchedulePeriodicMediaTimeUpdates() {
       base::Bind(&MojoRendererService::UpdateMediaTime, weak_this_, false));
 }
 
-void MojoRendererService::OnBufferingStateChanged(
-    BufferingState new_buffering_state) {
-  DVLOG(2) << __FUNCTION__ << "(" << new_buffering_state << ")";
-  client_->OnBufferingStateChange(
-      static_cast<interfaces::BufferingState>(new_buffering_state));
-}
-
-void MojoRendererService::OnRendererEnded() {
-  DVLOG(1) << __FUNCTION__;
-  CancelPeriodicMediaTimeUpdates();
-  client_->OnEnded();
-}
-
-void MojoRendererService::OnError(PipelineStatus error) {
-  DVLOG(1) << __FUNCTION__ << "(" << error << ")";
-  state_ = STATE_ERROR;
-  client_->OnError();
-}
-
 void MojoRendererService::OnFlushCompleted(const mojo::Closure& callback) {
   DVLOG(1) << __FUNCTION__;
   DCHECK_EQ(state_, STATE_FLUSHING);