Hooks together Geolocation Feature in the Client and Engine.

blimp/engine/feature/geolocation/engine_geolocation_feature.h

 // 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.
 
 #ifndef BLIMP_ENGINE_FEATURE_GEOLOCATION_ENGINE_GEOLOCATION_FEATURE_H_
 #define BLIMP_ENGINE_FEATURE_GEOLOCATION_ENGINE_GEOLOCATION_FEATURE_H_
 
 #include <memory>
 
 #include "base/memory/weak_ptr.h"
@@ skipping 26 matching lines @@
                       const net::CompletionCallback& callback) override;
 
  private:
   void NotifyCallback(const device::Geoposition& position);
 
   // BlimpLocationProvider::Delegate implementation.
   void RequestAccuracy(
       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);
 };
 
 }  // namespace engine
 }  // namespace blimp
 
 #endif  // BLIMP_ENGINE_FEATURE_GEOLOCATION_ENGINE_GEOLOCATION_FEATURE_H_