Added ConnectionTimeObserver to calculate the times to authenticate and connect.

remoting/protocol/connection_to_host_impl.cc

 // Copyright 2015 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/protocol/connection_to_host_impl.h"
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/location.h"
 #include "remoting/base/constants.h"
 #include "remoting/protocol/audio_reader.h"
 #include "remoting/protocol/audio_stub.h"
 #include "remoting/protocol/auth_util.h"
 #include "remoting/protocol/authenticator.h"
 #include "remoting/protocol/client_control_dispatcher.h"
 #include "remoting/protocol/client_event_dispatcher.h"
 #include "remoting/protocol/client_stub.h"
 #include "remoting/protocol/client_video_dispatcher.h"
 #include "remoting/protocol/clipboard_stub.h"
 #include "remoting/protocol/errors.h"
 #include "remoting/protocol/jingle_session_manager.h"
 #include "remoting/protocol/transport.h"
 #include "remoting/protocol/video_stub.h"
 
+#define RETURN_STRING_LITERAL(x) \

[Sergey Ulanov, 2015/07/30 22:00:44]

nit: move this next to StateToString()

[tonychun, 2015/07/30 23:20:26]

Done.

+case x: \
+return #x;
+
 namespace remoting {
 namespace protocol {
 
 ConnectionToHostImpl::ConnectionToHostImpl()
     : event_callback_(nullptr),
       client_stub_(nullptr),
       clipboard_stub_(nullptr),
       audio_stub_(nullptr),
       signal_strategy_(nullptr),
       state_(INITIALIZING),
       error_(OK) {
 }
 
 ConnectionToHostImpl::~ConnectionToHostImpl() {
   CloseChannels();
 
   if (session_.get())
     session_.reset();
 
   if (session_manager_.get())
     session_manager_.reset();
 
   if (signal_strategy_)
     signal_strategy_->RemoveListener(this);
 }
 
+const char* ConnectionToHost::StateToString(State state) {
+  switch (state) {
+    RETURN_STRING_LITERAL(INITIALIZING);
+    RETURN_STRING_LITERAL(CONNECTING);
+    RETURN_STRING_LITERAL(AUTHENTICATED);
+    RETURN_STRING_LITERAL(CONNECTED);
+    RETURN_STRING_LITERAL(CLOSED);
+    RETURN_STRING_LITERAL(FAILED);
+  }
+  NOTREACHED();
+  return nullptr;
+}
+
 void ConnectionToHostImpl::Connect(
     SignalStrategy* signal_strategy,
     scoped_ptr<TransportFactory> transport_factory,
     scoped_ptr<Authenticator> authenticator,
     const std::string& host_jid,
     HostEventCallback* event_callback) {
   DCHECK(client_stub_);
   DCHECK(clipboard_stub_);
   DCHECK(monitored_video_stub_);
 
@@ skipping 233 matching lines @@
 
   if (state != state_) {
     state_ = state;
     error_ = error;
     event_callback_->OnConnectionState(state_, error_);
   }
 }
 
 }  // namespace protocol
 }  // namespace remoting