Adds EngineGeolocationFeature for Blimp Geolocation project.

blimp/engine/feature/geolocation/blimp_location_provider.h

 // Copyright (c) 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.
 
 #ifndef BLIMP_ENGINE_FEATURE_GEOLOCATION_BLIMP_LOCATION_PROVIDER_H_
 #define BLIMP_ENGINE_FEATURE_GEOLOCATION_BLIMP_LOCATION_PROVIDER_H_
 
+#include "base/memory/weak_ptr.h"
+#include "blimp/common/proto/geolocation.pb.h"
 #include "content/public/browser/location_provider.h"
 #include "content/public/common/geoposition.h"
 
 namespace blimp {
 namespace engine {
 
 // Location provider for Blimp using the device's provider over the network.
 class BlimpLocationProvider : public content::LocationProvider {
  public:
-  BlimpLocationProvider();
+  // A delegate that implements a subset of LocationProvider's functions.
+  class Delegate {
+   public:
+    using GeopositionReceivedCallback =
+        base::Callback<void(const content::Geoposition&)>;
+
+    virtual ~Delegate() {}
+
+    virtual void RequestAccuracy(
+        GeolocationSetInterestLevelMessage::Level level) = 0;
+    virtual void RequestRefresh() = 0;
+

[Kevin M, 2016/07/21 17:37:05]

remove newline

[CJ, 2016/07/21 22:04:46]

Done.

+    virtual void SetUpdateCallback(
+        const GeopositionReceivedCallback& callback) = 0;
+  };
+
+  explicit BlimpLocationProvider(base::WeakPtr<Delegate> delegate);
   ~BlimpLocationProvider() override;
 
   // content::LocationProvider implementation.
   bool StartProvider(bool high_accuracy) override;
   void StopProvider() override;
   void GetPosition(content::Geoposition* position) override;
   void RequestRefresh() override;
   void OnPermissionGranted() override;
-
- private:
-  void NotifyCallback(const content::Geoposition& position);
-  void OnLocationResponse(const content::Geoposition& position);
   void SetUpdateCallback(
       const LocationProviderUpdateCallback& callback) override;
 
-  LocationProviderUpdateCallback callback_;
+ private:
+  // This delegate handles a subset of the LocationProvider functionality.
+  base::WeakPtr<Delegate> delegate_;
 
-  content::Geoposition position_;
+  content::Geoposition cached_position_;
+
+  // Indicates whether a successful StartProvider call has occured.

[Wez, 2016/07/22 01:18:58]

nit: Suggest "True if a ..."

[CJ, 2016/07/22 20:03:30]

Done.

+  bool is_started_;
 
   DISALLOW_COPY_AND_ASSIGN(BlimpLocationProvider);
 };
 
 }  // namespace engine
 }  // namespace blimp
 
 #endif  // BLIMP_ENGINE_FEATURE_GEOLOCATION_BLIMP_LOCATION_PROVIDER_H_