Implement HasPermission() method in PermissionService.

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 576 matching lines @@
 
   rules->script_rules.push_back(
       ContentSettingPatternSource(ContentSettingsPattern::Wildcard(),
                                   ContentSettingsPattern::Wildcard(),
                                   CONTENT_SETTING_ALLOW,
                                   std::string(),
                                   incognito));
 }
 #endif  // defined(ENALBE_EXTENSIONS)
 
+content::PermissionStatus
+ContentSettingToPermissionStatus(ContentSetting setting) {
+  switch (setting) {
+    case CONTENT_SETTING_ALLOW:
+      return content::PERMISSION_STATUS_GRANTED;

[mlamouri (slow - plz ping), 2014/11/24 19:06:01]

nit: CONTENT_SETTING_SESSION_ONLY should also link to _GRANTED.

[timvolodine, 2014/11/27 17:53:42]

Done.

+    case CONTENT_SETTING_BLOCK:
+      return content::PERMISSION_STATUS_DENIED;
+    case CONTENT_SETTING_ASK:
+      return content::PERMISSION_STATUS_ASK;
+    default:
+      return content::PERMISSION_STATUS_ASK;

[mlamouri (slow - plz ping), 2014/11/24 19:06:01]

I think it would be better to not have a |default|. Instead, you should have a
case for _DEFAULT and _NUM_SETTINGS and call NOTREACHED() if you end up there.

[timvolodine, 2014/11/27 17:53:42]

Done.

+  }
+}
+
 }  // namespace
 
 namespace chrome {
 
 ChromeContentBrowserClient::ChromeContentBrowserClient()
     : prerender_tracker_(NULL),
       weak_factory_(this) {
 #if defined(ENABLE_PLUGINS)
   for (size_t i = 0; i < arraysize(kPredefinedAllowedDevChannelOrigins); ++i)
     allowed_dev_channel_origins_.insert(kPredefinedAllowedDevChannelOrigins[i]);
@@ skipping 1357 matching lines @@
                               requesting_frame.GetOrigin(),
                               user_gesture,
                               result_callback);
       break;
     case content::PERMISSION_NUM:
       NOTREACHED() << "Invalid RequestPermission for " << permission;
       break;
   }
 }
 
+void ChromeContentBrowserClient::HasPermission(
+    content::PermissionType permission,
+    content::BrowserContext* browser_context,
+    const GURL& requesting_origin,
+    const GURL& embedding_origin,
+    const content::PermissionStatusCallback& result_callback) {
+  DCHECK(browser_context);
+  Profile* profile = Profile::FromBrowserContext(browser_context);
+
+  PermissionContextBase* context = nullptr;
+  switch (permission) {
+    case content::PERMISSION_MIDI_SYSEX:
+      context = MidiPermissionContextFactory::GetForProfile(profile);
+      break;
+    case content::PERMISSION_NOTIFICATIONS:
+#if defined(ENABLE_NOTIFICATIONS)
+      context = DesktopNotificationServiceFactory::GetForProfile(profile);
+#else
+      NOTIMPLEMENTED();
+#endif
+      break;
+    case content::PERMISSION_GEOLOCATION:
+      context = GeolocationPermissionContextFactory::GetForProfile(profile);
+      break;
+    case content::PERMISSION_PROTECTED_MEDIA:
+      NOTIMPLEMENTED();
+      break;
+    case content::PERMISSION_PUSH_MESSAGING:
+      context = gcm::PushMessagingPermissionContextFactory::GetForProfile(
+          profile);
+      break;
+    case content::PERMISSION_NUM:
+      NOTREACHED() << "Invalid RequestPermission for " << permission;
+      break;
+  }
+
+  ContentSetting result = context
+      ? context->GetPermissionStatus(requesting_origin.GetOrigin(),
+                                     embedding_origin.GetOrigin())
+      : CONTENT_SETTING_DEFAULT;
+
+  result_callback.Run(ContentSettingToPermissionStatus(result));

[mlamouri (slow - plz ping), 2014/11/24 19:06:01]

Why do you use a callback instead of returning directly the value? It seems that
the call is synchronous, right?

[timvolodine, 2014/11/27 17:53:42]

Done.

+}
+
 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,
@@ skipping 689 matching lines @@
       switches::kDisableWebRtcEncryption,
     };
     to_command_line->CopySwitchesFrom(from_command_line,
                                       kWebRtcDevSwitchNames,
                                       arraysize(kWebRtcDevSwitchNames));
   }
 }
 #endif  // defined(ENABLE_WEBRTC)
 
 }  // namespace chrome