<webview>: Move Geolocation permission to chrome layer.

content/public/browser/browser_plugin_guest_delegate.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_BROWSER_PLUGIN_GUEST_DELEGATE_H_
 #define CONTENT_PUBLIC_BROWSER_BROWSER_PLUGIN_GUEST_DELEGATE_H_
 
 #include "base/callback_forward.h"
 #include "base/process/kill.h"
 #include "base/strings/string16.h"
@@ skipping 69 matching lines @@
   // Notification that the guest is hung.
   virtual void RendererUnresponsive() {}
 
   typedef base::Callback<void(bool /* allow */,
                               const std::string& /* user_input */)>
       PermissionResponseCallback;
 
   // Request permission from the delegate to perform an action of the provided
   // |permission_type|. Details of the permission request are found in
   // |request_info|. A |callback| is provided to make the decision.
-  // Returns whether the delegate has, or will handle the permission request.
-  virtual bool RequestPermission(
+  // Returns an unique identifier for the request.

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

this return value isn't used in content (which i also agree with), so take it
out of here.

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

Done.

+  virtual int RequestPermission(
       BrowserPluginPermissionType permission_type,
       const base::DictionaryValue& request_info,
       const PermissionResponseCallback& callback,
       bool allowed_by_default);
 
   // Requests resolution of a potentially relative URL.
   virtual GURL ResolveURL(const std::string& src);
 
   // Notifies that the content size of the guest has changed in autosize mode.
   virtual void SizeChanged(const gfx::Size& old_size,
                            const gfx::Size& new_size) {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_BROWSER_PLUGIN_GUEST_DELEGATE_H_