Implement HasPermission() method in PermissionService.

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 4ea44b3fb3c4cf201d36398217eeb0dcb79629a8..91024d7842b8f86ee556b97984fa56cc54444015 100644
--- a/chrome/browser/chrome_content_browser_client.h
+++ b/chrome/browser/chrome_content_browser_client.h
@@ -197,6 +197,11 @@ class ChromeContentBrowserClient : public content::ContentBrowserClient {
       const GURL& requesting_frame,
       bool user_gesture,
       const base::Callback<void(bool)>& result_callback) override;
+  virtual void HasPermission(
+      content::PermissionType permission,
+      content::WebContents* web_contents,

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

We want to pass a BrowserContext here so we don't depend on WebContents.

[timvolodine, 2014/11/24 18:54:21]

Done.

+      const GURL& requesting_frame,

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

s/requesting_frame/requesting_origin/

Also, you might want to pass the embedding_origin like in
PermissionContextBase::GetPermissionStatus().

[timvolodine, 2014/11/24 18:54:22]

Done.

+      const base::Callback<void(bool)>& result_callback) override;

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

The callback shouldn't pass a |bool| back. We want to have more information, at
least ASK/GRANTED/DENIED.

[timvolodine, 2014/11/24 18:54:21]

done.
To do that I extracted permission_status enum into content/public/common to be
able to include in the content/public/browser/content_browser_client.h

   void CancelPermissionRequest(content::PermissionType permission,
                                content::WebContents* web_contents,
                                int bridge_id,