Adding new media controls to MediaNotification

content/public/browser/web_contents.h

 // Copyright (c) 2012 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 CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_
 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_
 
 #include <stdint.h>
 
 #include <set>
@@ skipping 24 matching lines @@
 
 namespace base {
 class DictionaryValue;
 class TimeTicks;
 }
 
 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;
 }
 
 namespace service_manager {
 class InterfaceProvider;
 }
 
 namespace content {
 
@@ skipping 641 matching lines @@
   // browser fullscreen.
   virtual void ExitFullscreen(bool will_cause_resize) = 0;
 
   // Unblocks requests from renderer for a newly created window. This is
   // used in showCreatedWindow() or sometimes later in cases where
   // delegate->ShouldResumeRequestsForCreatedWindow() indicated the requests
   // should not yet be resumed. Then the client is responsible for calling this
   // 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
   // SSLStatus to record the sensitive input field, so that embedders
   // can adjust the UI if desired.
   virtual void OnPasswordInputShownOnHttp() = 0;
 
   // Called when the WebContents has displayed a credit card field on an
   // HTTP page. This method modifies the appropriate NavigationEntry's
   // SSLStatus to record the sensitive input field, so that embedders
@@ skipping 37 matching lines @@
 
  private:
   // This interface should only be implemented inside content.
   friend class WebContentsImpl;
   WebContents() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_