ContentBrowserClient::RequestPermission replies with PermissionStatus instead of bool.

content/public/browser/content_browser_client.cc

Index: content/public/browser/content_browser_client.cc
diff --git a/content/public/browser/content_browser_client.cc b/content/public/browser/content_browser_client.cc
index 67d2df7c1bf839247279e9a134aa3e7a52900f22..96c076a602de161ea5e0fb2a3a789d3ab8d9e315 100644
--- a/content/public/browser/content_browser_client.cc
+++ b/content/public/browser/content_browser_client.cc
@@ -230,8 +230,8 @@ void ContentBrowserClient::RequestPermission(
     int bridge_id,
     const GURL& requesting_frame,
     bool user_gesture,
-    const base::Callback<void(bool)>& result_callback) {
-  result_callback.Run(true);
+    const base::Callback<void(PermissionStatus)>& callback) {
+  callback.Run(PERMISSION_STATUS_DENIED);

[gunsch, 2015/03/26 22:36:55]

I'm a little late to the party, but I just noticed that among the changes in
this CL it changed the behavior of all content embedders from defaulting to
returning "true" when permissions were requested to "denied". I wasn't able to
find a discussion about this change on the thread or the linked bug. What was
the reasoning here?

[mlamouri (slow - plz ping), 2015/03/30 09:32:41]

If you look at ContentBrowserClient, this is how it behave: everything usually
fails as a default implementation.

FWIW, the permission management has been moved out of ContentBrowserClient (to
content::PermissionManager) so hopefully, content embedders will catch that
change in default behaviour.

 }
 
 PermissionStatus ContentBrowserClient::GetPermissionStatus(