Implement HasPermission() method in PermissionService.

content/browser/permissions/permission_service_impl.cc

 // Copyright 2014 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 "content/browser/permissions/permission_service_impl.h"
 
 #include "content/browser/geolocation/geolocation_provider_impl.h"
 #include "content/public/browser/content_browser_client.h"
 #include "content/public/browser/render_process_host.h"
+#include "content/public/browser/web_contents.h"
 
 namespace content {
 
 namespace {
 
 PermissionType PermissionNameToPermissionType(PermissionName name) {
   switch(name) {
     case PERMISSION_NAME_GEOLOCATION:
       return PERMISSION_GEOLOCATION;
   }
@@ skipping 87 matching lines @@
         it.GetCurrentValue()->origin);
   }
 
   pending_requests_.Clear();
 }
 
 void PermissionServiceImpl::HasPermission(
     PermissionName permission,
     const mojo::String& origin,
     const mojo::Callback<void(PermissionStatus)>& callback) {
-  NOTIMPLEMENTED();
+  DCHECK(context_->GetBrowserContext());
+
+  // If there is no web_contents() we use |origin| as the embedding_origin.
+  GURL embedding_origin = context_->web_contents()
+      ? context_->web_contents()->GetLastCommittedURL().GetOrigin()
+      : GURL(origin);

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

I think it would be better to have a
PermissionServiceContext::GetEmbeddingOrigin() instead of having the logic here.

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

done. but we still need the |origin| so cannot move the logic completely to
context.

+
+  GetContentClient()->browser()->HasPermission(
+      PermissionNameToPermissionType(permission),
+      context_->GetBrowserContext(),
+      GURL(origin),
+      embedding_origin,
+      base::Bind(&PermissionServiceImpl::OnHasPermissionResponse,
+                 weak_factory_.GetWeakPtr(),
+                 callback));
+}
+
+void PermissionServiceImpl::OnHasPermissionResponse(
+    const mojo::Callback<void(PermissionStatus)>& callback,
+    PermissionStatus content_setting) {
+  callback.Run(content_setting);

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

nit: s/content_setting/permission_status/

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

Done.

 }
 
 }  // namespace content