Media Remoting: Don't auto suspend the media pipeline.

media/base/media_observer.h

Index: media/base/media_observer.h
diff --git a/media/base/media_observer.h b/media/base/media_observer.h
index 55558de179952a4e1b8326654285b5ffcc5c2d85..f46208d31c020d353e6c2658ebe0acf593c019b5 100644
--- a/media/base/media_observer.h
+++ b/media/base/media_observer.h
@@ -11,6 +11,21 @@
 
 namespace media {
 
+class MEDIA_EXPORT MediaObserverClient {
+ public:
+  virtual ~MediaObserverClient() {}
+
+  // Requests to restart the media pipeline and create a new renderer as soon as
+  // possible.
+  virtual void SwitchRenderer() = 0;

[miu, 2017/02/13 23:36:08]

Observation: It seems both SwitchRenderer() and DisablePipelineAutoSuspend() are
always called together.

Suggestion: How about just have one SwitchRenderer() method that takes a "bool
disable_auto_suspend" argument, rather than these two separate methods? This
would simplify code in both RendererController and WMPI.

[xjz, 2017/02/14 01:46:10]

Done. I did observe this too. But I was not sure whether we should combine these
two in the interface design, since these two are not necessarily always called
together. After an offline discussion, I agree with this suggestion, since they
are now always called together and can both control the pipeline.

+
+  // Requests to activate monitoring changes on viewport intersection.
+  virtual void ActivateViewportIntersectionMonitoring(bool activate) = 0;
+
+  // Requests to disable pipeline auto suspend for optimization.
+  virtual void DisablePipelineAutoSuspend(bool disable) = 0;
+};
+
 // This class is an observer of media player events.
 class MEDIA_EXPORT MediaObserver {
  public:
@@ -45,6 +60,9 @@ class MEDIA_EXPORT MediaObserver {
   // Called when a poster image URL is set, which happens when media is loaded
   // or the poster attribute is changed.
   virtual void OnSetPoster(const GURL& poster) = 0;
+
+  // Set the MediaObserverClient.
+  virtual void SetClient(MediaObserverClient* client) = 0;
 };
 
 }  // namespace media