Fix bug found in the location arbitrator whilst doing trial integration: need to make callback when new observer registers. + add a test for it.

BUG=none
TEST=GeolocationLocationArbitratorTest.RegistrationAfterFixArrives

Review URL: http://codereview.chromium.org/652136

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39884 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 8 insertions, 2 deletions

diff --git a/chrome/browser/geolocation/location_arbitrator.h b/chrome/browser/geolocation/location_arbitrator.h
index df2e517..86b6d99 100644
--- a/chrome/browser/geolocation/location_arbitrator.h
+++ b/chrome/browser/geolocation/location_arbitrator.h
@@ -19,8 +19,12 @@ struct Geoposition;
 // moment.
 class GeolocationArbitrator {
  public:
-  // Creates and returns a new instance of the location arbitrator.
-  static GeolocationArbitrator* New(
+  // Creates and returns a new instance of the location arbitrator. Will use
+  // the default access token store implementation bound to Chrome Prefs.
+  static GeolocationArbitrator* Create(URLRequestContextGetter* context_getter);
+  // Creates and returns a new instance of the location arbitrator. As above
+  // but allows injectino of the access token store, for testing.
+  static GeolocationArbitrator* Create(
       AccessTokenStore* access_token_store,
       URLRequestContextGetter* context_getter);
 
@@ -36,6 +40,8 @@ class GeolocationArbitrator {
   };
   struct UpdateOptions {
     UpdateOptions() : use_high_accuracy(false) {}
+    explicit UpdateOptions(bool high_accuracy)
+        : use_high_accuracy(high_accuracy) {}
     bool use_high_accuracy;
   };