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() {
 }
 
 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 =
+
+  const content::RenderViewHost* rvh =

[boliu, 2013/02/06 20:29:54]

Is this id->AwContents code duplicated with Selim's patch? I think I suggested
to him to move it into AwContents. (I'm not sure if he's landed yet)

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

Coordinating with Selim, will use that before landing this patch.

       content::RenderViewHost::FromID(render_process_id, render_view_id);
+  if (!rvh) {
+    callback.Run(false);
+    return;
+  }
+
   content::WebContents* web_contents =
-      content::WebContents::FromRenderViewHost(host);
+      content::WebContents::FromRenderViewHost(rvh);
+  if (!web_contents) {
+    callback.Run(false);
+    return;
+  }
+
   AwContents* aw_contents = AwContents::FromWebContents(web_contents);
-  aw_contents->OnGeolocationShowPrompt(
-      render_process_id,
-      render_view_id,
-      bridge_id,
-      requesting_frame);
+  if (!aw_contents) {
+    callback.Run(false);
+    return;
+  }

[joth, 2013/02/06 22:09:51]

yeah, this will all be cleaner with a method factored out (locally or
otherwise).

AwContents* aw_contetnts == FoBar..GetAwContentsFromID(process_id, view_id);
if (!aw_contetnts) {
  callback.Run(false);
  return;
}

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

Done.

+
+  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
+
+  const content::RenderViewHost* rvh =

[boliu, 2013/02/06 20:29:54]

Certainly don't duplicate this logic twice in the same file.

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

Done.

+    content::RenderViewHost::FromID(render_process_id, render_view_id);
+  if (!rvh) {
+    return;
+  }
+
+  content::WebContents* web_contents =
+    content::WebContents::FromRenderViewHost(rvh);
+  if (!web_contents) {
+    return;
+  }
+
+  AwContents* aw_contents = AwContents::FromWebContents(web_contents);
+  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