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

device/geolocation/mock_location_provider.h

Index: device/geolocation/mock_location_provider.h
diff --git a/device/geolocation/mock_location_provider.h b/device/geolocation/mock_location_provider.h
index c3357103f03ebc35c7c8430d2c259af9f23594c6..d78b4ae2c1e83064125626f85a854375e190b955 100644
--- a/device/geolocation/mock_location_provider.h
+++ b/device/geolocation/mock_location_provider.h
@@ -23,13 +23,15 @@ class MockLocationProvider : public LocationProviderBase {
   MockLocationProvider();
   ~MockLocationProvider() override;
 
+  bool IsProviderStarted() const;

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

nit: This can be an in-line "trivial getter", i.e:

bool is_started() const { return state_ != STOPPED; }

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

Done.

+
   // Updates listeners with the new position.
   void HandlePositionChanged(const Geoposition& position);
 
   // LocationProvider implementation.
   bool StartProvider(bool high_accuracy) override;
   void StopProvider() override;
-  void GetPosition(Geoposition* position) override;
+  const Geoposition& GetPosition() override;
   void OnPermissionGranted() override;
 
   bool is_permission_granted_;