Hooks together Geolocation Feature in the Client and Engine.

blimp/client/session/blimp_client_session.h

Index: blimp/client/session/blimp_client_session.h
diff --git a/blimp/client/session/blimp_client_session.h b/blimp/client/session/blimp_client_session.h
index aaccdd832ecf85c380a4c75e5cf0f15ab8c05c8c..04746a6b2f757fad4ed084220e2abccb42fb6d8c 100644
--- a/blimp/client/session/blimp_client_session.h
+++ b/blimp/client/session/blimp_client_session.h
@@ -18,6 +18,7 @@
 #include "blimp/client/public/session/assignment.h"
 #include "blimp/common/proto/blimp_message.pb.h"
 #include "blimp/net/blimp_message_processor.h"
+#include "device/geolocation/geolocation_delegate.h"
 
 namespace net {
 class IPEndPoint;
@@ -36,6 +37,7 @@ class ThreadPipeManager;
 namespace client {
 
 class ClientNetworkComponents;
+class GeolocationFeature;
 class NavigationFeature;
 class ImeFeature;
 class RenderWidgetFeature;
@@ -103,6 +105,8 @@ class BlimpClientSession
   BlobImageSerializationProcessor blob_image_processor_;
 
   std::unique_ptr<BlobChannelReceiver> blob_receiver_;
+  std::unique_ptr<device::GeolocationDelegate> geolocation_delegate_;

[nyquist, 2016/08/31 23:29:11]

This class is going away veru shortly. Could you please add the feature and the
feature registration to BlimpClientContextImpl as well as here?

[CJ, 2016/08/31 23:49:37]

Done.

+  std::unique_ptr<GeolocationFeature> geolocation_feature_;
   std::unique_ptr<TabControlFeature> tab_control_feature_;
   std::unique_ptr<NavigationFeature> navigation_feature_;
   std::unique_ptr<ImeFeature> ime_feature_;