Hooks together Geolocation Feature in the Client and Engine.

blimp/client/core/blimp_client_context_impl.h

Index: blimp/client/core/blimp_client_context_impl.h
diff --git a/blimp/client/core/blimp_client_context_impl.h b/blimp/client/core/blimp_client_context_impl.h
index 86c3cc8bee4ccca718081e240a4d305612f7e67f..9d8550b828af511ebc54e4c537f00f920d218d7f 100644
--- a/blimp/client/core/blimp_client_context_impl.h
+++ b/blimp/client/core/blimp_client_context_impl.h
@@ -6,6 +6,7 @@
 #define BLIMP_CLIENT_CORE_BLIMP_CLIENT_CONTEXT_IMPL_H_
 
 #include <memory>
+#include <string>
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
@@ -74,6 +75,9 @@ class BlimpClientContextImpl : public BlimpClientContext,
 
   std::unique_ptr<BlimpContentsManager> blimp_contents_manager_;
 
+  // TODO(lethalantidote): Add unique_ptr to GeolocationFeature and
+  // register it.

[Wez, 2016/08/24 23:07:18]

Not clear what this means - why not do that now? Or, if it can't happen now, why
do we need a TODO rather than just a bug filed for that work?

[CJ, 2016/08/25 19:08:22]

I can file a bug, sure. Kmarshall suggested that we wait until after this file
stabilizes more, since it seems that we are in transition from using
blimp_client_session to using blimp_client_context_impl. 

+
   // Container struct for network components.
   // Must be deleted on the IO thread.
   std::unique_ptr<ClientNetworkComponents> net_components_;