<webview>: Move Geolocation permission to chrome layer.

content/public/browser/browser_plugin_guest_delegate.cc

Index: content/public/browser/browser_plugin_guest_delegate.cc
diff --git a/content/public/browser/browser_plugin_guest_delegate.cc b/content/public/browser/browser_plugin_guest_delegate.cc
index 52577612aa1f32a2f9c751789d07b1e8ac2dbd6c..fb971c3109d9432eb711d46746a134de0ca65c34 100644
--- a/content/public/browser/browser_plugin_guest_delegate.cc
+++ b/content/public/browser/browser_plugin_guest_delegate.cc
@@ -4,6 +4,9 @@
 
 #include "content/public/browser/browser_plugin_guest_delegate.h"
 
+#include "base/bind.h"
+#include "base/message_loop/message_loop.h"
+
 namespace content {
 
 bool BrowserPluginGuestDelegate::HandleKeyboardEvent(
@@ -19,12 +22,18 @@ bool BrowserPluginGuestDelegate::IsOverridingUserAgent() const {
   return false;
 }
 
-bool BrowserPluginGuestDelegate::RequestPermission(
+int BrowserPluginGuestDelegate::RequestPermission(
     BrowserPluginPermissionType permission_type,
     const base::DictionaryValue& request_info,
     const PermissionResponseCallback& callback,
     bool allowed_by_default) {
-  return false;
+  base::MessageLoop::current()->PostTask(
+      FROM_HERE,
+      base::Bind(&PermissionResponseCallback::Run,
+                 base::Owned(new PermissionResponseCallback(callback)),
+                 allowed_by_default,
+                 std::string()));

[jam, 2014/04/14 18:11:55]

why not just an empty method? in general we try to keep all the delegate methods
empty in content/public

[Fady Samuel, 2014/04/14 18:51:14]

I've made it empty. This code is temporary. We will shortly be getting rid of
this entirely once all permissions are moved into the chrome layer.

+  return 0;
 }
 
 GURL BrowserPluginGuestDelegate::ResolveURL(const std::string& src) {