Hooks together Geolocation Feature in the Client and Engine.

device/geolocation/location_arbitrator.h

Index: device/geolocation/location_arbitrator.h
diff --git a/device/geolocation/location_arbitrator_impl.h b/device/geolocation/location_arbitrator.h
similarity index 91%
rename from device/geolocation/location_arbitrator_impl.h
rename to device/geolocation/location_arbitrator.h
index aa8e92ebd0dec2f9d9efb23d9bacb7cebdf73651..e1d6d8dff8a3a216e38d23a7762be9596b557585 100644
--- a/device/geolocation/location_arbitrator_impl.h
+++ b/device/geolocation/location_arbitrator.h
@@ -2,8 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef DEVICE_GEOLOCATION_LOCATION_ARBITRATOR_IMPL_H_
-#define DEVICE_GEOLOCATION_LOCATION_ARBITRATOR_IMPL_H_
+#ifndef DEVICE_GEOLOCATION_LOCATION_ARBITRATOR_H_
+#define DEVICE_GEOLOCATION_LOCATION_ARBITRATOR_H_
 
 #include <stdint.h>
 #include <memory>
@@ -34,16 +34,15 @@ class LocationProvider;
 // This class is responsible for handling updates from multiple underlying
 // providers and resolving them to a single 'best' location fix at any given
 // moment.
-class DEVICE_GEOLOCATION_EXPORT LocationArbitratorImpl
-    : public LocationProvider {
+class DEVICE_GEOLOCATION_EXPORT LocationArbitrator : public LocationProvider {
  public:
   // Number of milliseconds newer a location provider has to be that it's worth
   // switching to this location provider on the basis of it being fresher
   // (regardles of relative accuracy). Public for tests.
   static const int64_t kFixStaleTimeoutMilliseconds;
 
-  explicit LocationArbitratorImpl(GeolocationDelegate* delegate);
-  ~LocationArbitratorImpl() override;
+  explicit LocationArbitrator(GeolocationDelegate* delegate);

[Kevin M, 2016/08/22 22:09:34]

Take a std::unique_ptr here

[CJ, 2016/08/23 18:07:13]

I don't think we want to do that. GeolocationProviderImpl owns the object and
passes the delegate more as an interface.  

+  ~LocationArbitrator() override;
 
   static GURL DefaultNetworkProviderURL();
   bool HasPermissionBeenGrantedForTest() const;
@@ -116,7 +115,7 @@ class DEVICE_GEOLOCATION_EXPORT LocationArbitratorImpl
   // Tracks whether providers should be running.
   bool is_running_;
 
-  DISALLOW_COPY_AND_ASSIGN(LocationArbitratorImpl);
+  DISALLOW_COPY_AND_ASSIGN(LocationArbitrator);
 };
 
 // Factory functions for the various types of location provider to abstract
@@ -125,4 +124,4 @@ std::unique_ptr<LocationProvider> NewSystemLocationProvider();
 
 }  // namespace device
 
-#endif  // DEVICE_GEOLOCATION_LOCATION_ARBITRATOR_IMPL_H_
+#endif  // DEVICE_GEOLOCATION_LOCATION_ARBITRATOR_H_