WIP - WebMediaPlayer switch media renderer.

media/blink/webmediaplayer_impl.h

 // Copyright 2013 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_BLINK_WEBMEDIAPLAYER_IMPL_H_
 #define MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 211 matching lines @@
   };
 
   // Playback state variables computed together in UpdatePlayState().
   // (Public for testing.)
   struct PlayState {
     DelegateState delegate_state;
     bool is_memory_reporting_enabled;
     bool is_suspended;
   };
 
+  // Notify that video entered/exited full screen playing back. We may switch
+  // the media renderer in these calls.
+  void videoEnteredFullscreen() override;
+  void videoExitedFullscreen() override;

[liberato (no reviews please), 2016/08/29 16:49:12]

these names are a little confusing with respect to entered/exitedFullscreen. 
'Video' doesn't really add anything to differentiate them.

maybe elementOrContainerEnteredFullscreen()?  then again, that's not terribly
descriptive of what's going on, either.  becamePrimaryContent?

the particular check used in blink right now is heuristic, and might change
later.  better to choose a name that fits the intention, than the current
implementation.

also, please make the documentation very clear about what these do.  include a
contrast with enteredFullscreen, so that somebody i the future will be able to
pick which one they care about.  probably better in WMP rather than here.

[xjz, 2016/09/09 23:13:06]

Changed the names to ancestorEntered/exitedFullscreen().

+
  private:
   friend class WebMediaPlayerImplTest;
 
   void EnableOverlay();
   void DisableOverlay();
 
   void OnPipelineSuspended();
   void OnDemuxerOpened();
 
   // Pipeline::Client overrides.
@@ skipping 100 matching lines @@
   // using for video playback.  Called by |memory_usage_reporting_timer_|.
   // Memory usage reporting is done in two steps, because |demuxer_| must be
   // accessed on the media thread.
   void ReportMemoryUsage();
   void FinishMemoryUsageReport(int64_t demuxer_memory_usage);
 
   // Called during OnHidden() when we want a suspended player to enter the
   // paused state after some idle timeout.
   void ScheduleIdlePauseTimer();
 
+  // Remoter IPC messages handler to indicate mirroring session starts/ends.
+  void OnSinkAvailable();

[liberato (no reviews please), 2016/08/29 16:49:12]

does anybody call these?

[xjz, 2016/09/09 23:13:06]

This is called by Remoter, which is not implemented in this CL.

+  void OnSinkGone();
+
+  // Switch between the media renderer for local playing back and the remoting
+  // media renderer. |is_media_remoting_| should be set accordingly before
+  // calling this method.
+  void SwitchMediaRenderer();
+
   blink::WebLocalFrame* frame_;
 
   // The playback state last reported to |delegate_|, to avoid setting duplicate
   // states. (Which can have undesired effects like resetting the idle timer.)
   DelegateState delegate_state_;
 
   // Set when OnSuspendRequested() is called with |must_suspend| unset.
   bool is_idle_;
 
   // Set when OnSuspendRequested() is called with |must_suspend| set.
@@ skipping 166 matching lines @@
   // OnSuspendRequested() is called. If the state is UNKNOWN, the current frame
   // from the compositor will be queried to see if suspend is supported; the
   // state will be set to YES or NO respectively if a frame is available.
   enum class CanSuspendState { UNKNOWN, YES, NO };
   CanSuspendState can_suspend_state_;
 
   // Called some-time after OnHidden() if the media was suspended in a playing
   // state as part of the call to OnHidden().
   base::OneShotTimer background_pause_timer_;
 
+  // Indicates whether is in media remoting.
+  // TODO(erickung): Use this as an indicator for the RendererType when creating
+  // media renderer.
+  bool is_media_remoting_;
+
+  // Indicates whether is in mirroring or remoting. This is set through the
+  // Remoter IPC messages.
+  bool is_remote_sink_available_;
+
+  // Set this true when demuxer encounters encrypted stream.
+  bool is_encrypted_content_;
+
+  // Callback used to check whether the remoting CdmFactory is used.
+  WebMediaPlayerParams::IsRemoteCdmCB is_remote_cdm_cb_;
+
   DISALLOW_COPY_AND_ASSIGN(WebMediaPlayerImpl);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_