[Remoting Android] Move pairing secret fetcher to the network thread

remoting/client/jni/jni_client.cc

 // 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.
 
 #include "remoting/client/jni/jni_client.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
 #include "base/logging.h"
 #include "jni/Client_jni.h"
@@ skipping 42 matching lines @@
   session_.reset(new ChromotingJniInstance(
       runtime_, GetWeakPtr(), display_handler_,
       secret_fetcher_->GetWeakPtr(), username, auth_token, host_jid, host_id,
       host_pubkey, pairing_id, pairing_secret, capabilities, flags));
   session_->Connect();
 }
 
 void JniClient::DisconnectFromHost() {
   DCHECK(runtime_->ui_task_runner()->BelongsToCurrentThread());
   display_handler_ = nullptr;
-  secret_fetcher_.reset();
   if (session_) {
     session_->Disconnect();
     runtime_->network_task_runner()->DeleteSoon(FROM_HERE,
                                                  session_.release());
   }
+  if (secret_fetcher_) {
+    runtime_->network_task_runner()->DeleteSoon(FROM_HERE,
+                                                 secret_fetcher_.release());

[Lambros, 2016/06/07 01:21:12]

Indentation looks wrong here (and also on line 66). git cl format?

[Yuwei, 2016/06/07 18:01:03]

Done.

+  }
 }
 
 void JniClient::OnConnectionState(protocol::ConnectionToHost::State state,
                                   protocol::ErrorCode error) {
   DCHECK(runtime_->ui_task_runner()->BelongsToCurrentThread());
 
   JNIEnv* env = base::android::AttachCurrentThread();
   Java_Client_onConnectionState(env, java_client_.obj(), state, error);
 }
 
@@ skipping 205 matching lines @@
   return weak_factory_.GetWeakPtr();
 }
 
 static jlong Init(JNIEnv* env, const JavaParamRef<jobject>& caller) {
   return reinterpret_cast<intptr_t>(
       new JniClient(ChromotingJniRuntime::GetInstance(),
                     base::android::ScopedJavaGlobalRef<jobject>(env, caller)));
 }
 
 }  // namespace remoting