Make ChromeGeolocationPermissionContextFactory a ProfileKeyedServiceFactory

chrome/browser/geolocation/chrome_geolocation_permission_context_factory.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 "chrome/browser/geolocation/chrome_geolocation_permission_context_factory.h"
 
+#include "chrome/browser/profiles/profile_dependency_manager.h"
 #include "chrome/common/pref_names.h"
 #if defined(OS_ANDROID)
 #include "chrome/browser/geolocation/chrome_geolocation_permission_context_android.h"
 #else
 #include "chrome/browser/geolocation/chrome_geolocation_permission_context.h"
 #endif
 
+namespace {
+
+class Service : public ProfileKeyedService {
+ public:
+  explicit Service(Profile* profile) {
+#if defined(OS_ANDROID)
+    context_ = new ChromeGeolocationPermissionContextAndroid(profile);
+#else
+    context_ =  new ChromeGeolocationPermissionContext(profile);

[Bernhard Bauer, 2013/01/02 14:59:53]

Nit: only one space

[John Knottenbelt, 2013/01/02 15:49:02]

Done.

+#endif
+  }
+
+  ChromeGeolocationPermissionContext* context() {
+    return static_cast<ChromeGeolocationPermissionContext*>(context_.get());
+  }
+
+ private:
+  scoped_refptr<content::GeolocationPermissionContext> context_;

[Bernhard Bauer, 2013/01/02 14:59:53]

If you make this a ChromeGeolocationPermissionContext, I think you could get rid
of the static_cast above?

[John Knottenbelt, 2013/01/02 15:49:02]

Yes. Thanks, that's much neater!

On 2013/01/02 14:59:53, Bernhard Bauer wrote:
Done.

+};
+
+}  // namespace
+
+// static
 ChromeGeolocationPermissionContext*
-    ChromeGeolocationPermissionContextFactory::Create(Profile* profile) {
-#if defined(OS_ANDROID)
-  return new ChromeGeolocationPermissionContextAndroid(profile);
-#else
-  return new ChromeGeolocationPermissionContext(profile);
-#endif
+ChromeGeolocationPermissionContextFactory::GetForProfile(Profile* profile) {
+  return static_cast<Service*>(
+      GetInstance()->GetServiceForProfile(profile, true))->context();
+}
+
+// static
+ChromeGeolocationPermissionContextFactory*
+ChromeGeolocationPermissionContextFactory::GetInstance() {
+  return Singleton<ChromeGeolocationPermissionContextFactory>::get();
+}
+
+ChromeGeolocationPermissionContextFactory::
+ChromeGeolocationPermissionContextFactory()
+  : ProfileKeyedServiceFactory(

[Bernhard Bauer, 2013/01/02 14:59:53]

Nit: Indent four spaces (in total)

[John Knottenbelt, 2013/01/02 15:49:02]

Done.

+      "ChromeGeolocationPermissionContext",

[Bernhard Bauer, 2013/01/02 14:59:53]

...and then this line four spaces w/r/t to the ProfileKeyedService in the
previous line.

[John Knottenbelt, 2013/01/02 15:49:02]

Done.

+      ProfileDependencyManager::GetInstance()) {
+}
+
+ChromeGeolocationPermissionContextFactory::
+~ChromeGeolocationPermissionContextFactory() {
+}
+
+ProfileKeyedService*
+ChromeGeolocationPermissionContextFactory::BuildServiceInstanceFor(
+    Profile* profile) const {
+  return new Service(profile);
 }
 
 void ChromeGeolocationPermissionContextFactory::RegisterUserPrefs(
     PrefServiceSyncable* user_prefs) {
 #if defined(OS_ANDROID)
   user_prefs->RegisterBooleanPref(prefs::kGeolocationEnabled,
                                   true,
                                   PrefServiceSyncable::UNSYNCABLE_PREF);
 #endif
 }
+
+bool ChromeGeolocationPermissionContextFactory::
+ServiceRedirectedInIncognito() const {
+  return true;
+}