Adding new media controls to MediaNotification

content/public/browser/web_contents.h

Index: content/public/browser/web_contents.h
diff --git a/content/public/browser/web_contents.h b/content/public/browser/web_contents.h
index 931553d59112f690340a6905f7d42e3c16ab1fa2..57a21c8da1404ad107c8d15f1fb598068481899d 100644
--- a/content/public/browser/web_contents.h
+++ b/content/public/browser/web_contents.h
@@ -42,6 +42,12 @@ namespace blink {
 struct WebFindOptions;
 }
 
+namespace blink {
+namespace mojom {

[boliu, 2016/10/24 17:02:47]

put this in the namespace above?

[Zhiqiang Zhang (Slow), 2016/10/24 17:33:10]

No need for this since it's not in the public API.

+enum class MediaSessionAction;
+}
+}
+
 namespace net {
 struct LoadStateWithParam;
 }
@@ -703,12 +709,18 @@ class WebContents : public PageNavigator,
   // as soon as they are ready.
   virtual void ResumeLoadingCreatedWebContents() = 0;
 
+  // TODO(zqzhang): Decouple MediaSession messages from WebContents to avoid
+  // plumbing. See https://crbug.com/658678
+
   // Requests to resume the current media session.
   virtual void ResumeMediaSession() = 0;
   // Requests to suspend the current media session.
   virtual void SuspendMediaSession() = 0;
   // Requests to stop the current media session.
   virtual void StopMediaSession() = 0;
+  // Tell the media session a user action has performed.
+  virtual void DidReceiveMediaSessionAction(

[boliu, 2016/10/24 17:02:46]

Is this only going to be called from WebContentsAndroid? If so, it doesn't need
to be in the public API.

[Zhiqiang Zhang (Slow), 2016/10/24 17:33:10]

Thanks. Removed.

+      blink::mojom::MediaSessionAction action) = 0;
 
   // Called when the WebContents has displayed a password field on an
   // HTTP page. This method modifies the appropriate NavigationEntry's