Implementing geolocation for the Android Webview

android_webview/native/aw_geolocation_permission_context.cc

 // Copyright (c) 2012 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 "android_webview/native/aw_geolocation_permission_context.h"
 
 #include "android_webview/native/aw_contents.h"
 #include "base/bind.h"
 #include "base/callback.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/web_contents.h"
-#include "googleurl/src/gurl.h"
 
 namespace android_webview {
 
 AwGeolocationPermissionContext::~AwGeolocationPermissionContext() {
 }
 
+namespace {
+
+// TODO(kristianm): Removed this when https://codereview.chromium.org/12207061/

[benm (inactive), 2013/02/07 17:22:39]

guess you can do this now :)

[Kristian Monsen, 2013/02/08 00:07:44]

Done. I didn't send out a review mail for this patch since I knew this one was
landing.

+// has landed
+AwContents* GetAwContents(int render_process_id,
+                          int render_view_id,
+                          int bridge_id) {
+  const content::RenderViewHost* rvh =
+      content::RenderViewHost::FromID(render_process_id, render_view_id);
+  if (!rvh) {
+    return NULL;
+  }
+
+  content::WebContents* web_contents =
+      content::WebContents::FromRenderViewHost(rvh);
+  if (!web_contents) {
+    return NULL;
+  }
+
+  return AwContents::FromWebContents(web_contents);
+}
+
+} // anonymous namespace
+
 void
 AwGeolocationPermissionContext::RequestGeolocationPermissionOnUIThread(
     int render_process_id,
     int render_view_id,
     int bridge_id,
     const GURL& requesting_frame,
     base::Callback<void(bool)> callback) {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
-  const content::RenderViewHost* host =
-      content::RenderViewHost::FromID(render_process_id, render_view_id);
-  content::WebContents* web_contents =
-      content::WebContents::FromRenderViewHost(host);
-  AwContents* aw_contents = AwContents::FromWebContents(web_contents);
-  aw_contents->OnGeolocationShowPrompt(
-      render_process_id,
-      render_view_id,
-      bridge_id,
-      requesting_frame);
+
+  AwContents* aw_contents =
+      GetAwContents(render_process_id, render_view_id, bridge_id);
+  if (!aw_contents) {
+    callback.Run(false);
+    return;
+  }
+  aw_contents->OnGeolocationShowPrompt(requesting_frame, callback);
 }
 
 void
 AwGeolocationPermissionContext::RequestGeolocationPermission(
     int render_process_id,
     int render_view_id,
     int bridge_id,
     const GURL& requesting_frame,
     base::Callback<void(bool)> callback) {
   content::BrowserThread::PostTask(
@@ skipping 15 matching lines @@
   return new AwGeolocationPermissionContext();
 }
 
 void
 AwGeolocationPermissionContext::CancelGeolocationPermissionRequestOnUIThread(
     int render_process_id,
     int render_view_id,
     int bridge_id,
     const GURL& requesting_frame) {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
-  // TODO(kristianm): Implement this
+
+  AwContents* aw_contents =
+      GetAwContents(render_process_id, render_view_id, bridge_id);
+  if (aw_contents) {
+    aw_contents->OnGeolocationHidePrompt(requesting_frame);
+  }
 }
 
 void
 AwGeolocationPermissionContext::CancelGeolocationPermissionRequest(
     int render_process_id,
     int render_view_id,
     int bridge_id,
     const GURL& requesting_frame) {
   content::BrowserThread::PostTask(
       content::BrowserThread::UI, FROM_HERE,
       base::Bind(
           &AwGeolocationPermissionContext::
               CancelGeolocationPermissionRequestOnUIThread,
           this,
           render_process_id,
           render_view_id,
           bridge_id,
           requesting_frame));
 }
 
-void InvokeCallback(
-    int render_process_id,
-    int render_view_id,
-    int bridge_id,
-    const GURL& requesting_frame,
-    bool value) {
-  // TODO(kristianm): Implement this
-}
-
 }  // namespace android_webview