Added ConnectionTimeObserver to calculate the times to authenticate and connect.

remoting/client/plugin/chromoting_instance.cc

 // Copyright (c) 2012 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/plugin/chromoting_instance.h"
 
 #include <string>
 #include <vector>
 
 #if defined(OS_NACL)
@@ skipping 80 matching lines @@
 const int kBandwidthHistogramBuckets = 100;
 
 #if defined(USE_OPENSSL)
 // Size of the random seed blob used to initialize RNG in libjingle. Libjingle
 // uses the seed only for OpenSSL builds. OpenSSL needs at least 32 bytes of
 // entropy (see http://wiki.openssl.org/index.php/Random_Numbers), but stores
 // 1039 bytes of state, so we initialize it with 1k or random data.
 const int kRandomSeedSize = 1024;
 #endif  // defined(USE_OPENSSL)
 
-std::string ConnectionStateToString(protocol::ConnectionToHost::State state) {

[tonychun, 2015/07/29 18:30:33]

Since this functionality exists in remoting/protocol/connection_to_host.h, I
removed this.

-  // Values returned by this function must match the
-  // remoting.ClientSession.State enum in JS code.
-  switch (state) {
-    case protocol::ConnectionToHost::INITIALIZING:
-      return "INITIALIZING";
-    case protocol::ConnectionToHost::CONNECTING:
-      return "CONNECTING";
-    case protocol::ConnectionToHost::AUTHENTICATED:
-      return "AUTHENTICATED";
-    case protocol::ConnectionToHost::CONNECTED:
-      return "CONNECTED";
-    case protocol::ConnectionToHost::CLOSED:
-      return "CLOSED";
-    case protocol::ConnectionToHost::FAILED:
-      return "FAILED";
-  }
-  NOTREACHED();
-  return std::string();
-}
-
 // TODO(sergeyu): Ideally we should just pass ErrorCode to the webapp
 // and let it handle it, but it would be hard to fix it now because
 // client plugin and webapp versions may not be in sync. It should be
 // easy to do after we are finished moving the client plugin to NaCl.
 std::string ConnectionErrorToString(protocol::ErrorCode error) {
   // Values returned by this function must match the
   // remoting.ClientSession.Error enum in JS code.
   switch (error) {
     case protocol::OK:
       return "NONE";
@@ skipping 1100 matching lines @@
 
 #if !defined(OS_NACL)
   // Log messages are forwarded to the webapp only in PNaCl version of the
   // plugin, so ProcessLogToUI() needs to be called explicitly in the non-PNaCl
   // version.
   ProcessLogToUI(message);
 #endif  // !defined(OS_NACL)
 }
 
 }  // namespace remoting