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

device/geolocation/network_location_provider.h

Index: device/geolocation/network_location_provider.h
diff --git a/device/geolocation/network_location_provider.h b/device/geolocation/network_location_provider.h
index f47310425789ad83277d07f3a746df95302cbd51..3c0c67d403cb17376ca4f728c678106e2f004473 100644
--- a/device/geolocation/network_location_provider.h
+++ b/device/geolocation/network_location_provider.h
@@ -70,11 +70,13 @@ class NetworkLocationProvider : public base::NonThreadSafe,
       const base::string16& access_token);
   ~NetworkLocationProvider() override;
 
+  // LocationProviderBase implemenation.
+  void RequestRefresh() override;

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

There doesn't seem to be any need for this to be part of LocationProviderBase -
let's make it purely an internal detail of this provider?

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

Done.

+
   // LocationProvider implementation
   bool StartProvider(bool high_accuracy) override;
   void StopProvider() override;
   void GetPosition(Geoposition* position) override;
-  void RequestRefresh() override;
   void OnPermissionGranted() override;
 
  private: