Blimp: relocate thread-unsafe member getter in DropConnection code.

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 f739f6df4f49f940a2770f5b82e53fbb6272cd9d..182119cc4d39c9c1cafae9e3ddeb37b988a95b7a 100644
--- a/blimp/client/session/blimp_client_session.h
+++ b/blimp/client/session/blimp_client_session.h
@@ -86,6 +86,10 @@ class BlimpClientSession
  private:
   void RegisterFeatures();
 
+  // Terminates the active connection held by |net_connections_|.
+  // May be called on any thread.

[Wez, 2016/09/17 01:56:31]

Eek, let's not leave inviting comments like that in the codebase :P

I think what we really mean here is just that this should be called on the main
thread, and will trigger the connection-drop on the IO thread, right?

[Kevin M, 2016/09/19 17:00:23]

Done.

+  void DropConnection();
+
   // NetworkEventObserver implementation.
   void OnConnected() override;
   void OnDisconnected(int result) override;