Added ConnectionTimeObserver to calculate the times to authenticate and connect.

remoting/client/plugin/chromoting_instance.cc

Index: remoting/client/plugin/chromoting_instance.cc
diff --git a/remoting/client/plugin/chromoting_instance.cc b/remoting/client/plugin/chromoting_instance.cc
index 9e8c291fe6f3dbc5fb8312bfcb497b1dd15a97d9..9e891ba03f75ca102e7620c72f0899f3a9331603 100644
--- a/remoting/client/plugin/chromoting_instance.cc
+++ b/remoting/client/plugin/chromoting_instance.cc
@@ -98,27 +98,6 @@ const int kBandwidthHistogramBuckets = 100;
 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