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

device/geolocation/location_provider_android.h

Index: device/geolocation/location_provider_android.h
diff --git a/device/geolocation/location_provider_android.h b/device/geolocation/location_provider_android.h
index bae4bb1b20b73241416a566dd35da59fa6d9737f..7b5734cb35be4c2b087eeeb379f0148c7e8f6a41 100644
--- a/device/geolocation/location_provider_android.h
+++ b/device/geolocation/location_provider_android.h
@@ -25,11 +25,14 @@ class LocationProviderAndroid : public LocationProviderBase {
   // LocationProvider.
   bool StartProvider(bool high_accuracy) override;
   void StopProvider() override;
-  void GetPosition(Geoposition* position) override;
-  void RequestRefresh() override;
+  const Geoposition& GetPosition() override;
   void OnPermissionGranted() override;
 
  private:
+  // Provides a hint to the provider that new location data is needed as soon
+  // as possible.
+  void RequestRefresh();

[Wez, 2016/08/19 01:33:12]

Not sure what you intended here but:
(1) This is private, so no code outside this class can call it to provide a
hint.
(2) Nothing inside the class calls it.
(3) It doesn't do anything, even if they did - pretty sure the Android impl only
override'd it because it had to!

[CJ, 2016/08/22 17:41:31]

Done.

+
   Geoposition last_position_;
 };