Added ConnectionTimeObserver to calculate the times to authenticate and connect.

remoting/test/chromoting_test_driver.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 <string>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/files/file_path.h"
 #include "base/logging.h"
 #include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
 #include "base/strings/stringprintf.h"
 #include "base/test/test_suite.h"
 #include "base/test/test_switches.h"
 #include "google_apis/google_api_keys.h"
 #include "net/base/escape.h"
 #include "remoting/test/access_token_fetcher.h"
+#include "remoting/test/connection_time_observer.h"
 #include "remoting/test/host_info.h"
 #include "remoting/test/host_list_fetcher.h"
 #include "remoting/test/refresh_token_store.h"
 #include "remoting/test/test_chromoting_client.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace switches {
 const char kAuthCodeSwitchName[] = "authcode";
 const char kPinSwitchName[] = "pin";
 const char kHelpSwitchName[] = "help";
@@ skipping 160 matching lines @@
   // To avoid shared resource contention, tests will be run one at a time.
   command_line->AppendSwitch(switches::kSingleProcessTestsSwitchName);
 
   if (command_line->HasSwitch(switches::kHelpSwitchName)) {
     PrintUsage();
     PrintJsonFileInfo();
     PrintAuthCodeInfo();
     return 0;
   }
 
-  // Update the logging verbosity level is user specified one.
+  // Update the logging verbosity level if user specified one.
   std::string verbosity_level(
       command_line->GetSwitchValueASCII(switches::kLoggingLevelSwitchName));
   if (!verbosity_level.empty()) {
     // Turn on logging for the test_driver and remoting components.
     // This switch is parsed during logging::InitLogging.
     command_line->AppendSwitchASCII("vmodule",
                                     "*/remoting/*=" + verbosity_level);
     logging::LoggingSettings logging_settings;
     logging::InitLogging(logging_settings);
   }
@@ skipping 87 matching lines @@
 
   // Check if requested host is online and ready to receive connections.
   auto it = std::find_if(hostlist.begin(), hostlist.end(),
                         [&hostname](const remoting::test::HostInfo& host_info) {
                           return host_info.host_name == hostname &&
                                  host_info.IsReadyForConnection();
                         });
   if (it != hostlist.end()) {
     // Host is online and ready, initiate a remote session.
     base::Timer timer(true, false);
+    remoting::test::ConnectionTimeObserver observer;
+    test_chromoting_client.AddRemoteConnectionObserver(&observer);

[joedow, 2015/07/27 21:45:44]

Please send an update once your change to add the environment class to main()
lands so we can review the merge of the two CLs.

[tonychun, 2015/07/28 17:53:44]

The changes have landed and are now together with this CL.

+
+    // The timer will forcefully end the chromoting connection process in
+    // |kConnectionTimeoutSeconds| seconds. The set limit on time to connect
+    // stops the client from waiting indefinitely for a connection.
     timer.Start(FROM_HERE,
                 base::TimeDelta::FromSeconds(kConnectionTimeoutSeconds),
                 run_loop->QuitClosure());
+    observer.Initialize();
     test_chromoting_client.StartConnection(
         it->GenerateConnectionSetupInfo(access_token, username, pin));
     run_loop->Run();
+
+    observer.DisplayConnectionStats();
+    test_chromoting_client.RemoveRemoteConnectionObserver(&observer);
     test_chromoting_client.EndConnection();
   } else {
     LOG(ERROR) << "Requested host not found or not ready to connect";
   }
 
   return 0;
 }