Implementing native chromium GeolocationPermissionContext

android_webview/native/aw_geolocation_permission_context.h

+// 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.
+
+#ifndef ANDROID_WEBVIEW_BROWSER_AW_GEOLOCATION_PERMISSION_CONTEXT_H_
+#define ANDROID_WEBVIEW_BROWSER_AW_GEOLOCATION_PERMISSION_CONTEXT_H_
+
+#include "base/callback_forward.h"
+#include "content/public/browser/geolocation_permission_context.h"
+
+class GURL;
+
+namespace android_webview {
+
+class AwGeolocationPermissionContext :
+    public content::GeolocationPermissionContext {
+ public:
+  virtual ~AwGeolocationPermissionContext();
+
+  static content::GeolocationPermissionContext* Create();
+
+  // content::GeolocationPermissionContext implementation
+  virtual void RequestGeolocationPermission(
+      int render_process_id,
+      int render_view_id,
+      int bridge_id,
+      const GURL& requesting_frame,
+      base::Callback<void(bool)> callback) OVERRIDE;
+
+  // content::GeolocationPermissionContext implementation

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

nit: usually one of this comment is enough, and group the overriden methods
together without space between them, but since there are only two methods,
doesn't matter that much

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

Done.

+  virtual void CancelGeolocationPermissionRequest(
+      int render_process_id,
+      int render_view_id,
+      int bridge_id,
+      const GURL& requesting_frame) OVERRIDE;
+
+  void InvokeCallback(
+      int render_process_id,
+      int render_view_id,
+      int bridge_id,
+      const GURL& requesting_frame,
+      bool value
+      );

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

pretty sure the end parenthesis should be on end of previous line.

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

Done.

+};
+
+}  // anemspace android_webview

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

typo

+
+#endif  // ANDROID_WEBVIEW_BROWSER_AW_GEOLOCATION_PERMISSION_CONTEXT_H_