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..95a3e2bed237bbdbb4968ca99421befd28eb0094 100644
--- a/device/geolocation/mock_location_provider.h
+++ b/device/geolocation/mock_location_provider.h
@@ -23,6 +23,8 @@ class MockLocationProvider : public LocationProviderBase {
   MockLocationProvider();
   ~MockLocationProvider() override;
 
+  bool providers_started() const { return state_ != STOPPED; }

[Kevin M, 2016/08/15 19:12:46]

This style of inline getter is only really allowed for just getting the value of
a field, not for computed results. This should be something like
IsProviderStarted().


nit: This is a LocationProvider (singular), but this getter is plural.

[CJ, 2016/08/15 20:11:51]

Done.

+
   // Updates listeners with the new position.
   void HandlePositionChanged(const Geoposition& position);