Implementing native chromium GeolocationPermissionContext

android_webview/browser/aw_content_browser_client.cc

Index: android_webview/browser/aw_content_browser_client.cc
diff --git a/android_webview/browser/aw_content_browser_client.cc b/android_webview/browser/aw_content_browser_client.cc
index 8b1b827dd423d0e29fc44a499d1f26f3bdc27c4e..1b5851da7c653eae9c188f26a56e362a20fb044a 100644
--- a/android_webview/browser/aw_content_browser_client.cc
+++ b/android_webview/browser/aw_content_browser_client.cc
@@ -44,13 +44,15 @@ class DummyAccessTokenStore : public content::AccessTokenStore {
 namespace android_webview {
 
 AwContentBrowserClient::AwContentBrowserClient(
-    ViewDelegateFactoryFn* view_delegate_factory)
+    ViewDelegateFactoryFn* view_delegate_factory,
+    content::GeolocationPermissionContext* geolocation_permission_context)

[benm (inactive), 2013/01/04 11:35:52]

Why does this need to come in as a param here? Couldn't we just new one up
inside AwBrowserContext the first time the getter is called?

[boliu, 2013/01/04 16:42:05]

That would lead to a DEPS violation since AwBrowserContext is in browser/ and
cannot include AwGeolocationPermisssionContext which is in native/.

Although I still vote for passing in a "FactoryFn" so it can be created lazily.
We can refactor into a true factory interface when we collect enough of these.

[Kristian Monsen, 2013/01/04 17:13:46]

Right now lazy instantiation would not get anything except increased code
complexity as the factory would have to be created at the same time.

Unless you have something else that you want to use it for right now I think it
is best to wait until it is needed. I don't know which other kinds of objects
the factory should create.

[boliu, 2013/01/04 17:17:30]

I was suggesting FactoryFn (ie function pointer) in this CL, basically just copy
ViewDelegateFactoryFn. Refactoring to a real factory class comes later.

[Kristian Monsen, 2013/01/04 22:54:31]

Updated to do this, take a look and see if this was what you were thinking.

     : view_delegate_factory_(view_delegate_factory) {
   FilePath user_data_dir;
   if (!PathService::Get(base::DIR_ANDROID_APP_DATA, &user_data_dir)) {
     NOTREACHED() << "Failed to get app data directory for Android WebView";
   }
-  browser_context_.reset(new AwBrowserContext(user_data_dir));
+  browser_context_.reset(
+      new AwBrowserContext(user_data_dir, geolocation_permission_context));
 }
 
 AwContentBrowserClient::~AwContentBrowserClient() {