Refactor content client to include a single Request/CancelPermission method.

chrome/browser/chrome_content_browser_client.cc

 // 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.
 
 #include "chrome/browser/chrome_content_browser_client.h"
 
 #include <set>
 #include <utility>
 #include <vector>
 
@@ skipping 1904 matching lines @@
   service->ShowDesktopNotification(
       params, render_frame_host, delegate.Pass(), cancel_callback);
 
   profile->GetHostContentSettingsMap()->UpdateLastUsage(
       params.origin, params.origin, CONTENT_SETTINGS_TYPE_NOTIFICATIONS);
 #else
   NOTIMPLEMENTED();
 #endif
 }
 
-void ChromeContentBrowserClient::RequestGeolocationPermission(
+void ChromeContentBrowserClient::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) {
   int render_process_id = web_contents->GetRenderProcessHost()->GetID();
   int render_view_id = web_contents->GetRenderViewHost()->GetRoutingID();
 
   const PermissionRequestID request_id(render_process_id,
                                        render_view_id,
                                        bridge_id,
                                        requesting_frame);
-  GeolocationPermissionContextFactory::GetForProfile(
-      Profile::FromBrowserContext(web_contents->GetBrowserContext()))->
-          RequestPermission(web_contents, request_id,
-                            requesting_frame.GetOrigin(), user_gesture,
-                            result_callback);
+
+  switch (permission) {
+    case content::PERMISSION_MIDI_SYSEX:
+      MidiPermissionContextFactory::GetForProfile(
+          Profile::FromBrowserContext(web_contents->GetBrowserContext()))
+      ->RequestPermission(web_contents, request_id, requesting_frame,
+                          user_gesture, result_callback);
+      break;
+    case content::PERMISSION_NOTIFICATIONS:
+      // TODO(miguelg) implement notifications once the platform refactor
+      // is finished.
+      LOG(WARNING) << "Permission request not implemented for " << permission;
+      break;
+    case content::PERMISSION_GEOLOCATION:
+      GeolocationPermissionContextFactory::GetForProfile(
+          Profile::FromBrowserContext(web_contents->GetBrowserContext()))
+      ->RequestPermission(web_contents, request_id,
+                          requesting_frame.GetOrigin(), user_gesture,
+                          result_callback);
+      break;
+#if defined(OS_ANDROID)
+    case content::PERMISSION_PROTECTED_MEDIA:
+      ProtectedMediaIdentifierPermissionContextFactory::GetForProfile(
+          Profile::FromBrowserContext(web_contents->GetBrowserContext()))
+      ->RequestProtectedMediaIdentifierPermission(web_contents,
+                                                  requesting_frame,
+                                                  result_callback);
+      break;
+#endif
+    default:
+      LOG(WARNING) << "Permission not available " << permission;
+  }
+
 }
 
-void ChromeContentBrowserClient::CancelGeolocationPermissionRequest(
+void ChromeContentBrowserClient::CancelPermissionRequest(
+    content::PermissionType permission,
     content::WebContents* web_contents,
     int bridge_id,
     const GURL& requesting_frame) {
   int render_process_id = web_contents->GetRenderProcessHost()->GetID();
   int render_view_id = web_contents->GetRenderViewHost()->GetRoutingID();
 
   const PermissionRequestID request_id(render_process_id,
                                        render_view_id,
                                        bridge_id,
                                        requesting_frame);
-  GeolocationPermissionContextFactory::GetForProfile(
-        Profile::FromBrowserContext(web_contents->GetBrowserContext()))->
-            CancelPermissionRequest(web_contents, request_id);
+  switch (permission) {
+    case content::PERMISSION_MIDI_SYSEX:
+          MidiPermissionContextFactory::GetForProfile(
+              Profile::FromBrowserContext(web_contents->GetBrowserContext()))
+          ->CancelPermissionRequest(web_contents, request_id);
+          break;
+        case content::PERMISSION_NOTIFICATIONS:
+          // TODO(miguelg) implement notifications once the platform refactor
+          // is finished.
+          LOG(WARNING) << "Cancel Permission request not implemented for "
+              << permission;
+          break;
+        case content::PERMISSION_GEOLOCATION:
+          GeolocationPermissionContextFactory::GetForProfile(
+                Profile::FromBrowserContext(
+                    web_contents->GetBrowserContext()))
+          ->CancelPermissionRequest(web_contents, request_id);
+          break;
+    #if defined(OS_ANDROID)
+        case content::PERMISSION_PROTECTED_MEDIA:
+          ProtectedMediaIdentifierPermissionContextFactory::GetForProfile(
+              Profile::FromBrowserContext(web_contents->GetBrowserContext()))
+          ->CancelProtectedMediaIdentifierPermissionRequests(
+              render_process_id,
+              render_view_id,
+              requesting_frame);
+          break;
+    #endif
+        default:
+          LOG(WARNING) << "Permission not available " << permission;
+  }
 }
 
-void ChromeContentBrowserClient::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) {
-  MidiPermissionContext* context =
-      MidiPermissionContextFactory::GetForProfile(
-          Profile::FromBrowserContext(web_contents->GetBrowserContext()));
-  int renderer_id = web_contents->GetRenderProcessHost()->GetID();
-  int render_view_id = web_contents->GetRenderViewHost()->GetRoutingID();
-  const PermissionRequestID id(renderer_id, render_view_id, bridge_id, GURL());
-
-  context->RequestPermission(web_contents, id, requesting_frame,
-                             user_gesture, result_callback);
-}
 
 void ChromeContentBrowserClient::DidUseGeolocationPermission(
     content::WebContents* web_contents,
     const GURL& frame_url,
     const GURL& main_frame_url) {
   Profile::FromBrowserContext(web_contents->GetBrowserContext())
       ->GetHostContentSettingsMap()
       ->UpdateLastUsage(
           frame_url, main_frame_url, CONTENT_SETTINGS_TYPE_GEOLOCATION);
 }
 
-void ChromeContentBrowserClient::RequestProtectedMediaIdentifierPermission(
-    content::WebContents* web_contents,
-    const GURL& origin,
-    base::Callback<void(bool)> result_callback,
-    base::Closure* cancel_callback) {
-#if defined(OS_ANDROID)
-  ProtectedMediaIdentifierPermissionContext* context =
-      ProtectedMediaIdentifierPermissionContextFactory::GetForProfile(
-          Profile::FromBrowserContext(web_contents->GetBrowserContext()));
-  context->RequestProtectedMediaIdentifierPermission(web_contents,
-                                                     origin,
-                                                     result_callback,
-                                                     cancel_callback);
-#else
-  NOTIMPLEMENTED();
-  result_callback.Run(false);
-#endif  // defined(OS_ANDROID)
-}
-
 bool ChromeContentBrowserClient::CanCreateWindow(
     const GURL& opener_url,
     const GURL& opener_top_level_frame_url,
     const GURL& source_origin,
     WindowContainerType container_type,
     const GURL& target_url,
     const content::Referrer& referrer,
     WindowOpenDisposition disposition,
     const WebWindowFeatures& features,
     bool user_gesture,
@@ skipping 595 matching lines @@
       switches::kDisableWebRtcEncryption,
     };
     to_command_line->CopySwitchesFrom(from_command_line,
                                       kWebRtcDevSwitchNames,
                                       arraysize(kWebRtcDevSwitchNames));
   }
 }
 #endif  // defined(ENABLE_WEBRTC)
 
 }  // namespace chrome