Simplify geolocation permission request in the Content API.

android_webview/browser/aw_content_browser_client.cc

Index: android_webview/browser/aw_content_browser_client.cc
diff --git a/android_webview/browser/aw_content_browser_client.cc b/android_webview/browser/aw_content_browser_client.cc
index 7c3dc6590991f180d19518ec90060fffd9ba3880..d5e8390de86601050375bb4aea37a91316795baf 100644
--- a/android_webview/browser/aw_content_browser_client.cc
+++ b/android_webview/browser/aw_content_browser_client.cc
@@ -386,6 +386,24 @@ void AwContentBrowserClient::ShowDesktopNotification(
   NOTREACHED() << "Android WebView does not support desktop notifications.";
 }
 
+void AwContentBrowserClient::RequestGeolocationPermission(
+    content::WebContents* web_contents,
+    int bridge_id,
+    const GURL& requesting_frame,
+    bool user_gesture,
+    base::Callback<void(bool)> result_callback,
+    base::Closure* cancel_callback) {
+  AwContentsClientBridgeBase* client =
+      AwContentsClientBridgeBase::FromWebContents(web_contents);
+  if (client) {
+    client->RequestGeolocationPermission(
+        web_contents, requesting_frame, result_callback, cancel_callback);
+  } else {
+    LOG(WARNING) << "Failed to find the associated bridge for geolocation "
+                 << "permission request.";

[mkosiba (inactive), 2014/06/13 08:17:03]

do we need to result_callback.Run(false); in this case?

[jam, 2014/06/13 14:56:08]

Done.

+  }
+}
+
 bool AwContentBrowserClient::CanCreateWindow(
     const GURL& opener_url,
     const GURL& opener_top_level_frame_url,