Refactor GeofencingManager to have one instance per StoragePartition.

content/browser/geofencing/geofencing_provider.h

Index: content/browser/geofencing/geofencing_provider.h
diff --git a/content/browser/geofencing/geofencing_provider.h b/content/browser/geofencing/geofencing_provider.h
index ab5faa079a105345dd0695890a9429ad5d5d98d4..42d367c230a7a387c81e2b1b40f886fc66c32941 100644
--- a/content/browser/geofencing/geofencing_provider.h
+++ b/content/browser/geofencing/geofencing_provider.h
@@ -5,6 +5,7 @@
 #ifndef CONTENT_BROWSER_GEOFENCING_GEOFENCING_PROVIDER_H_
 #define CONTENT_BROWSER_GEOFENCING_GEOFENCING_PROVIDER_H_
 
+#include "base/basictypes.h"
 #include "base/callback_forward.h"
 #include "content/common/geofencing_status.h"
 
@@ -16,12 +17,7 @@ namespace content {
 
 class GeofencingProvider {
  public:
-  // Callback that gets called on completion of registering a new region. The
-  // status indicates success or failure, and in case of success, an id to use
-  // to later unregister the region is passed as |registration_id|. If
-  // registration failed, providers should set |registration_id| to -1.
-  typedef base::Callback<void(GeofencingStatus, int registration_id)>
-      RegisterCallback;
+  typedef base::Callback<void(GeofencingStatus)> StatusCallback;
 
   virtual ~GeofencingProvider() {}
 
@@ -32,12 +28,15 @@ class GeofencingProvider {
   // that is registered with the platform specific provider does not exceed the
   // number of regions supported by the platform, although that isn't
   // implemented yet.
-  virtual void RegisterRegion(const blink::WebCircularGeofencingRegion& region,
-                              const RegisterCallback& callback) = 0;
+  // RegisterRegion should asynchronously call the |callback| to indicate
+  // success or failure.
+  virtual void RegisterRegion(int64 geofencing_registration_id,
+                              const blink::WebCircularGeofencingRegion& region,
+                              const StatusCallback& callback) = 0;
 
   // Called by |GeofencingManager| to unregister an existing registration. Will
   // only be called once for each registration.
-  virtual void UnregisterRegion(int registration_id) = 0;
+  virtual void UnregisterRegion(int64 geofencing_registration_id) = 0;
 };
 
 }  // namespace content