Implementing native chromium GeolocationPermissionContext

android_webview/browser/aw_browser_context.h

Index: android_webview/browser/aw_browser_context.h
diff --git a/android_webview/browser/aw_browser_context.h b/android_webview/browser/aw_browser_context.h
index d184cdeecd3fe6adfad7d7305cf79201c3a770a3..715cb48c7859204538af9bbac1f53f1a9761a4b6 100644
--- a/android_webview/browser/aw_browser_context.h
+++ b/android_webview/browser/aw_browser_context.h
@@ -9,14 +9,19 @@
 #include "base/file_path.h"
 #include "base/memory/ref_counted.h"
 #include "content/public/browser/browser_context.h"
+#include "content/public/browser/geolocation_permission_context.h"
 
 namespace android_webview {
 
 class AwURLRequestContextGetter;
 
+typedef content::GeolocationPermissionContext* GeolocationPermissionGetterFn();

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

Name it FactoryFn, since the function is also passing ownership of the returned
object.

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

Done, also added _fn to the end of the variable names so it is clear it is a
function and not an object.

+
 class AwBrowserContext : public content::BrowserContext {
  public:
-  AwBrowserContext(const FilePath path);
+  AwBrowserContext(
+      const FilePath path,
+      GeolocationPermissionGetterFn* geolocation_permission_getter);
   virtual ~AwBrowserContext();
 
   // Called before BrowserThreads are created.
@@ -51,6 +56,9 @@ class AwBrowserContext : public content::BrowserContext {
   FilePath context_storage_path_;
 
   scoped_refptr<AwURLRequestContextGetter> url_request_context_getter_;
+  GeolocationPermissionGetterFn* geolocation_permission_getter_;
+  scoped_refptr<content::GeolocationPermissionContext>
+      geolocation_permission_context_;
 
   AwDownloadManagerDelegate download_manager_delegate_;