Hooks together Geolocation Feature in the Client and Engine.

blimp/engine/feature/geolocation/engine_geolocation_feature.h

Index: blimp/engine/feature/geolocation/engine_geolocation_feature.h
diff --git a/blimp/engine/feature/geolocation/engine_geolocation_feature.h b/blimp/engine/feature/geolocation/engine_geolocation_feature.h
index 4219db03ebd54581a03516b8a9dca50b4ac97bec..e1bbb447d1c8d4ed54740d81dd5d2b5bb7478af5 100644
--- a/blimp/engine/feature/geolocation/engine_geolocation_feature.h
+++ b/blimp/engine/feature/geolocation/engine_geolocation_feature.h
@@ -44,9 +44,11 @@ class EngineGeolocationFeature : public BlimpMessageProcessor,
       GeolocationSetInterestLevelMessage::Level level) override;
   void OnPermissionGranted() override;
   void SetUpdateCallback(const GeopositionReceivedCallback& callback) override;
+  void SetTaskRunner(scoped_refptr<base::SingleThreadTaskRunner> runner);

[Wez, 2016/09/01 02:45:43]

This method isn't part of the Delegate interface, so shouldn't be grouped with
the other methods.

Why do we need it at all, though?  Can't we just set the TaskRunner directly, in
SetUpdateCallback()?

 
   std::unique_ptr<BlimpMessageProcessor> outgoing_message_processor_;
   GeopositionReceivedCallback geoposition_received_callback_;
+  scoped_refptr<base::SingleThreadTaskRunner> task_runner_;
   base::WeakPtrFactory<EngineGeolocationFeature> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(EngineGeolocationFeature);