Fix bug of posting task inside constructor

remoting/client/jni/chromoting_jni_instance.cc

 // Copyright 2013 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/chromoting_jni_instance.h"
 
 #include <android/log.h>
 #include <stdint.h>
 
 #include <utility>
@@ skipping 61 matching lines @@
   xmpp_config_.auth_token = auth_token;
 
   client_auth_config_.host_id = host_id;
   client_auth_config_.pairing_client_id = pairing_id;
   client_auth_config_.pairing_secret = pairing_secret;
   client_auth_config_.fetch_secret_callback = base::Bind(
       &ChromotingJniInstance::FetchSecret, weak_factory_.GetWeakPtr());
   client_auth_config_.fetch_third_party_token_callback =
       base::Bind(&ChromotingJniInstance::FetchThirdPartyToken,
                  weak_factory_.GetWeakPtr(), host_pubkey);
-
-  // Post a task to start connection
-  jni_runtime_->network_task_runner()->PostTask(
-      FROM_HERE,
-      base::Bind(&ChromotingJniInstance::ConnectToHostOnNetworkThread, this));
 }
 
 ChromotingJniInstance::~ChromotingJniInstance() {
   // This object is ref-counted, so this dtor can execute on any thread.
   // Ensure that all these objects have been freed already, so they are not
   // destroyed on some random thread.
   DCHECK(!view_);
   DCHECK(!client_context_);
   DCHECK(!video_renderer_);
   DCHECK(!client_);
   DCHECK(!signaling_);
 }
 
+void ChromotingJniInstance::Connect() {
+  if (jni_runtime_->network_task_runner()->BelongsToCurrentThread()) {
+    ConnectToHostOnNetworkThread();
+  } else {
+    jni_runtime_->network_task_runner()->PostTask(
+        FROM_HERE,
+        base::Bind(&ChromotingJniInstance::ConnectToHostOnNetworkThread, this));
+  }
+}
+
 void ChromotingJniInstance::Disconnect() {
   if (!jni_runtime_->network_task_runner()->BelongsToCurrentThread()) {
     jni_runtime_->network_task_runner()->PostTask(
         FROM_HERE,
         base::Bind(&ChromotingJniInstance::Disconnect, this));
     return;
   }
 
   stats_logging_enabled_ = false;
 
@@ skipping 407 matching lines @@
       perf_tracker_->round_trip_ms().Max());
 
   jni_runtime_->logger()->LogStatistics(perf_tracker_.get());
 
   jni_runtime_->network_task_runner()->PostDelayedTask(
       FROM_HERE, base::Bind(&ChromotingJniInstance::LogPerfStats, this),
       base::TimeDelta::FromMilliseconds(kPerfStatsIntervalMs));
 }
 
 }  // namespace remoting