Access ChromotingHost::clients_ only on network thread.

remoting/client/chromoting_client.cc

Index: remoting/client/chromoting_client.cc
diff --git a/remoting/client/chromoting_client.cc b/remoting/client/chromoting_client.cc
index 327afa067dfa04bc9d65367bcc9acbc7e6b02725..e50d95d38456fdc2d877f53355effc5d0e5d9056 100644
--- a/remoting/client/chromoting_client.cc
+++ b/remoting/client/chromoting_client.cc
@@ -64,6 +64,12 @@ void ChromotingClient::Stop(const base::Closure& shutdown_task) {
     return;
   }
 
+  // Drop all pending packets.
+  while(!received_packets_.empty()) {
+    received_packets_.front().done.Run();
+    received_packets_.pop_front();

[Wez, 2011/11/09 02:32:22]

This looks leaked from another CL?

[Sergey Ulanov, 2011/11/09 21:24:00]

Yeah, it should be in a separate CL. I thought my change triggered DCHECK due to
this issue, but it is actually not related to my change.

+  }
+
   connection_->Disconnect(base::Bind(&ChromotingClient::OnDisconnected,
                                      base::Unretained(this), shutdown_task));
 }