<webview>: Move Geolocation permission to chrome layer.

content/public/browser/browser_plugin_guest_delegate.cc

 // Copyright 2013 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/public/browser/browser_plugin_guest_delegate.h"
 
+#include "base/bind.h"
+#include "base/message_loop/message_loop.h"
+
 namespace content {
 
 bool BrowserPluginGuestDelegate::HandleKeyboardEvent(
     const NativeWebKeyboardEvent& event) {
   return false;
 }
 
 bool BrowserPluginGuestDelegate::IsDragAndDropEnabled() {
   return false;
 }
 
 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) {
   return GURL(src);
 }
 
 }  // namespace content