Refactor GeofencingManager to have one instance per StoragePartition.

content/browser/storage_partition_impl.h

Index: content/browser/storage_partition_impl.h
diff --git a/content/browser/storage_partition_impl.h b/content/browser/storage_partition_impl.h
index f0c12d420837021209023e885a39c1e9c3084d0c..385c6184bdfe011593632f7eef1aed8e470db763 100644
--- a/content/browser/storage_partition_impl.h
+++ b/content/browser/storage_partition_impl.h
@@ -43,6 +43,7 @@ class StoragePartitionImpl : public StoragePartition {
   DOMStorageContextWrapper* GetDOMStorageContext() override;
   IndexedDBContextImpl* GetIndexedDBContext() override;
   ServiceWorkerContextWrapper* GetServiceWorkerContext() override;
+  virtual GeofencingManager* GetGeofencingManager() override;

[Avi (use Gerrit), 2014/10/23 17:32:45]

no |virtual|; it's implied by |override|.

[Marijn Kruisselbrink, 2014/10/23 20:54:35]

Done.

 
   void ClearDataForOrigin(uint32 remove_mask,
                           uint32 quota_storage_remove_mask,
@@ -116,7 +117,8 @@ class StoragePartitionImpl : public StoragePartition {
       IndexedDBContextImpl* indexed_db_context,
       ServiceWorkerContextWrapper* service_worker_context,
       WebRTCIdentityStore* webrtc_identity_store,
-      storage::SpecialStoragePolicy* special_storage_policy);
+      storage::SpecialStoragePolicy* special_storage_policy,
+      GeofencingManager* geofencing_manager);
 
   void ClearDataImpl(uint32 remove_mask,
                      uint32 quota_storage_remove_mask,
@@ -156,6 +158,7 @@ class StoragePartitionImpl : public StoragePartition {
   scoped_refptr<ServiceWorkerContextWrapper> service_worker_context_;
   scoped_refptr<WebRTCIdentityStore> webrtc_identity_store_;
   scoped_refptr<storage::SpecialStoragePolicy> special_storage_policy_;
+  scoped_refptr<GeofencingManager> geofencing_manager_;
 
   DISALLOW_COPY_AND_ASSIGN(StoragePartitionImpl);
 };