Group the different permission related methods in the content api.

chrome/browser/chrome_content_browser_client.h

Index: chrome/browser/chrome_content_browser_client.h
diff --git a/chrome/browser/chrome_content_browser_client.h b/chrome/browser/chrome_content_browser_client.h
index cbd7382e5fe65914652fb4db217a299641969ac0..fde816d8b6a71cd83baa04b7aaeb5c1187e8cfad 100644
--- a/chrome/browser/chrome_content_browser_client.h
+++ b/chrome/browser/chrome_content_browser_client.h
@@ -169,7 +169,7 @@ class ChromeContentBrowserClient : public content::ContentBrowserClient {
       bool expired_previous_decision,
       const base::Callback<void(bool)>& callback,
       content::CertificateRequestResultType* request) override;
-  void SelectClientCertificate(
+  virtual void SelectClientCertificate(

[Peter Beverloo, 2014/10/24 14:37:28]

s/virtual//.

[Miguel Garcia, 2014/10/24 16:05:32]

Acknowledged.

       int render_process_id,
       int render_frame_id,
       net::SSLCertRequestInfo* cert_request_info,
@@ -180,11 +180,6 @@ class ChromeContentBrowserClient : public content::ContentBrowserClient {
                       int render_process_id,
                       int render_frame_id) override;
   content::MediaObserver* GetMediaObserver() override;
-  void RequestDesktopNotificationPermission(
-      const GURL& source_origin,
-      content::RenderFrameHost* render_frame_host,
-      const base::Callback<void(blink::WebNotificationPermission)>& callback)
-      override;
   blink::WebNotificationPermission CheckDesktopNotificationPermission(
       const GURL& source_origin,
       content::ResourceContext* context,
@@ -194,30 +189,21 @@ class ChromeContentBrowserClient : public content::ContentBrowserClient {
       content::RenderFrameHost* render_frame_host,
       scoped_ptr<content::DesktopNotificationDelegate> delegate,
       base::Closure* cancel_callback) override;
-  void RequestGeolocationPermission(
+  void RequestPermission(
+      content::PermissionType permission,
       content::WebContents* web_contents,
       int bridge_id,
       const GURL& requesting_frame,
       bool user_gesture,
       const base::Callback<void(bool)>& result_callback) override;
-  void CancelGeolocationPermissionRequest(
-      content::WebContents* web_contents,
-      int bridge_id,
-      const GURL& requesting_frame) override;
-  void RequestMidiSysExPermission(content::WebContents* web_contents,
-                                  int bridge_id,
-                                  const GURL& requesting_frame,
-                                  bool user_gesture,
-                                  base::Callback<void(bool)> result_callback,
-                                  base::Closure* cancel_callback) override;
-  void DidUseGeolocationPermission(content::WebContents* web_contents,
-                                   const GURL& frame_url,
-                                   const GURL& main_frame_url) override;
-  void RequestProtectedMediaIdentifierPermission(
-      content::WebContents* web_contents,
-      const GURL& origin,
-      base::Callback<void(bool)> result_callback,
-      base::Closure* cancel_callback) override;
+  void CancelPermissionRequest(content::PermissionType permission,
+                               content::WebContents* web_contents,
+                               int bridge_id,
+                               const GURL& requesting_frame) override;
+  void RegisterPermissionUsage(content::PermissionType permission,
+                               content::WebContents* web_contents,
+                               const GURL& frame_url,
+                               const GURL& main_frame_url) override;
   bool CanCreateWindow(const GURL& opener_url,
                        const GURL& opener_top_level_frame_url,
                        const GURL& source_origin,