Pass media session id over IPC

third_party/WebKit/public/platform/modules/mediasession/WebMediaSession.h

 // Copyright 2015 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 WebMediaSession_h
 #define WebMediaSession_h
 
 #include "public/platform/WebCallbacks.h"
 #include "public/platform/modules/mediasession/WebMediaSessionError.h"
 
 namespace blink {
 
+static const int kDefaultMediaSessionID = 0;

[DaleCurtis, 2016/02/26 19:14:30]

No static const in header files since each include gets its own copy. Instead
define it in the .cpp file and a prototype here or use an enum.

[DaleCurtis, 2016/02/26 19:16:44]

Discussion here
https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/hDqJ4KBVqog

[davve, 2016/02/29 09:04:01]

Thanks for looking! An enum looks like the appropriate choice here.

+
 using WebMediaSessionActivateCallback = WebCallbacks<void, const WebMediaSessionError&>;
 using WebMediaSessionDeactivateCallback = WebCallbacks<void, void>;
 
 struct WebMediaMetadata;
 
 class WebMediaSession {
 public:
     virtual ~WebMediaSession() = default;
 
     // Tries to activate the session by requesting audio focus from
@@ skipping 10 matching lines @@
 
     // Updates the metadata associated with the WebMediaSession. The metadata
     // can be a nullptr in which case the associated metadata should be reset.
     // The pointer is not owned by the WebMediaSession implementation.
     virtual void setMetadata(const WebMediaMetadata*) = 0;
 };
 
 } // namespace blink
 
 #endif // WebMediaSession_h