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

device/geolocation/geolocation_provider_impl.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef DEVICE_GEOLOCATION_GEOLOCATION_PROVIDER_IMPL_H_
 #define DEVICE_GEOLOCATION_GEOLOCATION_PROVIDER_IMPL_H_
 
 #include <list>
 #include <memory>
 #include <vector>
 
 #include "base/callback_forward.h"
 #include "base/compiler_specific.h"
 #include "base/macros.h"
 #include "base/threading/thread.h"
 #include "device/geolocation/geolocation_export.h"
 #include "device/geolocation/geolocation_provider.h"
 #include "device/geolocation/geoposition.h"
+#include "device/geolocation/location_provider.h"
 
 namespace base {
 template <typename Type>
 struct DefaultSingletonTraits;
 class SingleThreadTaskRunner;
 }
 
 namespace device {
 class LocationArbitrator;

[Kevin M, 2016/08/10 23:16:19]

Not used?

[CJ, 2016/08/11 22:16:53]

Done.

 
 class DEVICE_GEOLOCATION_EXPORT GeolocationProviderImpl
     : public NON_EXPORTED_BASE(GeolocationProvider),
       public base::Thread {
  public:
   // GeolocationProvider implementation:
   std::unique_ptr<GeolocationProvider::Subscription> AddLocationUpdateCallback(
       const LocationUpdateCallback& callback,
       bool enable_high_accuracy) override;
   void UserDidOptIntoLocationServices() override;
@@ skipping 11 matching lines @@
   bool user_did_opt_into_location_services_for_testing() {
     return user_did_opt_into_location_services_;
   }
 
  protected:
   friend struct base::DefaultSingletonTraits<GeolocationProviderImpl>;
   GeolocationProviderImpl();
   ~GeolocationProviderImpl() override;
 
   // Useful for injecting mock geolocation arbitrator in tests.
   // TODO(mvanouwerkerk): Use something like SetArbitratorForTesting instead.

[Wez, 2016/08/09 01:17:04]

nit: Consider implementing this TODO() while you're here. :)

e.g. Create the LocationArbitrator in the Init() and allow it to be replaced
with a MockLocationProvider via the API as he suggests.

[CJ, 2016/08/11 22:06:43]

Done.

-  virtual std::unique_ptr<LocationArbitrator> CreateArbitrator();
+  virtual std::unique_ptr<LocationProvider> CreateArbitrator();
 
  private:
   bool OnGeolocationThread() const;
 
   // Start and stop providers as needed when clients are added or removed.
   void OnClientsChanged();
 
   // Stops the providers when there are no more registered clients. Note that
   // once the Geolocation thread is started, it will stay alive (but sitting
   // idle without any pending messages).
@@ skipping 16 matching lines @@
   base::CallbackList<void(const Geoposition&)> high_accuracy_callbacks_;
   base::CallbackList<void(const Geoposition&)> low_accuracy_callbacks_;
 
   bool user_did_opt_into_location_services_;
   Geoposition position_;
 
   // True only in testing, where we want to use a custom position.
   bool ignore_location_updates_;
 
   // Only to be used on the geolocation thread.
-  std::unique_ptr<LocationArbitrator> arbitrator_;
+  std::unique_ptr<LocationProvider> arbitrator_;
 
   // Used to PostTask()s from the geolocation thread to creation thread.
   const scoped_refptr<base::SingleThreadTaskRunner> main_task_runner_;
 
   DISALLOW_COPY_AND_ASSIGN(GeolocationProviderImpl);
 };
 
 }  // namespace device
 
 #endif  // DEVICE_GEOLOCATION_GEOLOCATION_PROVIDER_IMPL_H_