Hooks together Geolocation Feature in the Client and Engine.

blimp/client/core/geolocation/blimp_location_provider.cc

+// Copyright 2016 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.
+
+#include "blimp/client/core/geolocation/blimp_location_provider.h"
+
+#include "device/geolocation/geoposition.h"
+#include "device/geolocation/location_arbitrator_impl.h"
+
+namespace blimp {
+namespace client {
+namespace {
+class BlimpGeolocationDelegate : public device::GeolocationDelegate {
+ public:
+  BlimpGeolocationDelegate() = default;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(BlimpGeolocationDelegate);
+};
+
+}  // namespace
+
+BlimpLocationProvider::BlimpLocationProvider()
+    : location_provider_(new device::LocationArbitratorImpl(
+          base::Bind(&BlimpLocationProvider::OnLocationUpdate,
+                     base::Unretained(this)),
+          new BlimpGeolocationDelegate())) {}

[Wez, 2016/08/19 23:12:10]

Why not pass in a default GeolocationDelegate here?

[CJ, 2016/08/22 20:28:08]

I thought there would be some issue with there not being a AccessTokenStore on
that one, but I looked at it again, and I guess there really isn't much of a
reason to worry about that, since for android it doesnt seem to be used. 

+
+BlimpLocationProvider::~BlimpLocationProvider() {}
+
+void BlimpLocationProvider::SetUpdateCallback(
+    const LocationProviderUpdateCallback& callback) {
+  DCHECK(!callback.is_null());
+  feature_update_callback_ = callback;
+}
+
+bool BlimpLocationProvider::StartProvider(bool enable_high_accuracy) {
+  return location_provider_->StartProvider(enable_high_accuracy);
+}
+
+void BlimpLocationProvider::StopProvider() {
+  location_provider_->StopProvider();
+}
+
+const device::Geoposition& BlimpLocationProvider::GetPosition() {
+  return location_provider_->GetPosition();
+}
+
+void BlimpLocationProvider::OnPermissionGranted() {
+  location_provider_->OnPermissionGranted();
+}
+
+void BlimpLocationProvider::OnLocationUpdate(
+    const device::Geoposition& new_position) {
+  feature_update_callback_.Run(this, new_position);
+}
+
+}  // namespace client
+}  // namespace blimp