Gets rid of the LocationArbitrator interface, in preference for LocationProvider.

device/geolocation/location_provider_base.h

Index: device/geolocation/location_provider_base.h
diff --git a/device/geolocation/location_provider_base.h b/device/geolocation/location_provider_base.h
index dc67a1045880d5c239c3ae9e8cc9f4567c4b6476..f8b5132c7ddca77a327408260c5e6425c8a6046f 100644
--- a/device/geolocation/location_provider_base.h
+++ b/device/geolocation/location_provider_base.h
@@ -20,10 +20,13 @@ class DEVICE_GEOLOCATION_EXPORT LocationProviderBase
  protected:
   void NotifyCallback(const Geoposition& position);
 
+  // Provides a hint to the provider that new location data is needed as soon
+  // as possible.
+  virtual void RequestRefresh();

[Wez, 2016/08/12 00:33:44]

Is there any point in keeping this at all?

Since it's no longer part of LocationProvider, just a protected method here,
nothing external can call it.

The only LocationProvider that does actually call it is the
NetworkLocationProvider - it's doing that internally, so there's no need for it
to be part of this base class - it can be a non-virtual in
NetworkLocationProvider itself.

[CJ, 2016/08/12 20:22:46]

Done.

+
   // Overridden from LocationProvider:
   void SetUpdateCallback(
       const LocationProviderUpdateCallback& callback) override;
-  void RequestRefresh() override;
 
  private:
   LocationProviderUpdateCallback callback_;