Implementing native chromium GeolocationPermissionContext

android_webview/native/aw_contents.h

Index: android_webview/native/aw_contents.h
diff --git a/android_webview/native/aw_contents.h b/android_webview/native/aw_contents.h
index 44da9f23ba0f0aaa35678a90a1c3097c6c2cbb65..0c91f3e8e9ec062727ab76cf83d787c7f079ba84 100644
--- a/android_webview/native/aw_contents.h
+++ b/android_webview/native/aw_contents.h
@@ -9,6 +9,7 @@
 #include <string>
 
 #include "android_webview/browser/find_helper.h"
+#include "android_webview/native/aw_geolocation_permission_context.h"

[boliu_use_chromium_pls, 2013/01/04 23:45:43]

doesn't appear to be used

[Kristian Monsen, 2013/01/05 02:44:58]

Done.

 #include "android_webview/public/browser/draw_gl.h"
 #include "base/android/scoped_java_ref.h"
 #include "base/android/jni_helper.h"
@@ -102,6 +103,11 @@ class AwContents : public FindHelper::Listener,
                            int scroll_x, int scroll_y);
   void FocusFirstNode(JNIEnv* env, jobject obj);
 
+  // Geolocation API support
+  void GeolocationShowPrompt(int render_process_id, int render_view_id,
+                             int bridge_id, const GURL& requesting_frame);

[boliu_use_chromium_pls, 2013/01/04 23:45:43]

nit: I think in chromium c++ style guide
(http://dev.chromium.org/developers/coding-style), you have all arguments in one
line or each on different lines (and lined up)

[Kristian Monsen, 2013/01/05 02:44:58]

You are right, done. Almost every other function around there breaks that style
rule.

[boliu, 2013/01/05 04:37:35]

True...attribute it to differences between java and c++ style guide

+  void GeolocationHidePrompt();
+
   // Find-in-page API and related methods.
   jint FindAllSync(JNIEnv* env, jobject obj, jstring search_string);
   void FindAllAsync(JNIEnv* env, jobject obj, jstring search_string);