Propagate media session id into MediaPlayerAndroid

media/base/android/media_player_android.h

Index: media/base/android/media_player_android.h
diff --git a/media/base/android/media_player_android.h b/media/base/android/media_player_android.h
index 16982d69c8cd8f73716f1c6ca76e9d2b12d526d6..2038ba8e35653b59a0c8ab1d96113b916ce2570a 100644
--- a/media/base/android/media_player_android.h
+++ b/media/base/android/media_player_android.h
@@ -23,6 +23,11 @@ namespace media {
 class MediaKeys;
 class MediaPlayerManager;
 
+// The media session for media elements that don't have an explicit
+// user created media session set. Must be in-sync with
+// WebMediaSession::DefaultID in blink.
+enum { kDefaultMediaSessionID = 0 };

[whywhat, 2016/03/10 22:34:56]

nit: I believe Id is preferred to ID but I don't feel very strongly, the
codebase seems to prefer Id over ID but there's not that many instances:

https://code.google.com/p/chromium/codesearch#search/&q=case:yes%20file:%5C.h...

https://code.google.com/p/chromium/codesearch#search/&q=case:yes%20file:%5C.h...

[davve, 2016/03/11 13:20:29]

Done.

+
 // This class serves as the base class for different media player
 // implementations on Android. Subclasses need to provide their own
 // MediaPlayerAndroid::Create() implementation.
@@ -108,6 +113,8 @@ class MEDIA_EXPORT MediaPlayerAndroid {
 
   GURL frame_url() { return frame_url_; }
 
+  int media_session_id() { return media_session_id_; }
+
   // Attach/Detaches |listener_| for listening to all the media events. If
   // |j_media_player| is NULL, |listener_| only listens to the system media
   // events. Otherwise, it also listens to the events from |j_media_player|.
@@ -119,7 +126,8 @@ class MEDIA_EXPORT MediaPlayerAndroid {
       int player_id,
       MediaPlayerManager* manager,
       const OnDecoderResourcesReleasedCB& on_decoder_resources_released_cb,
-      const GURL& frame_url);
+      const GURL& frame_url,
+      int media_session_id);
 
   // TODO(qinmin): Simplify the MediaPlayerListener class to only listen to
   // media interrupt events. And have a separate child class to listen to all
@@ -173,6 +181,9 @@ class MEDIA_EXPORT MediaPlayerAndroid {
   // Listener object that listens to all the media player events.
   scoped_ptr<MediaPlayerListener> listener_;
 
+  // Media session ID assigned to this player.
+  int media_session_id_;
+
   // Weak pointer passed to |listener_| for callbacks.
   // NOTE: Weak pointers must be invalidated before all other member variables.
   base::WeakPtrFactory<MediaPlayerAndroid> weak_factory_;