Separating the two tests into different executions.

remoting/test/chromoting_test_driver_tests.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/test/chromoting_test_fixture.h"
 #include "remoting/test/connection_time_observer.h"
 
 namespace {
 const base::TimeDelta kPinBasedMaxConnectionTimeInSeconds =
     base::TimeDelta::FromSeconds(4);
 const int kPinBasedMaxAuthenticationTimeMs = 2000;
 const int kMaxTimeToConnectMs = 2000;
 }
 
 namespace remoting {
 namespace test {
 
+// Note: Make sure to restart the host before running this test. Connecting to
+// a previously connected host will output time the time to reconnect versus the

[anandc, 2015/08/13 21:35:31]

nit: "output time the time to" -> "output the time to"

[tonychun, 2015/08/13 21:39:07]

Done.

+// time to connect.
 TEST_F(ChromotingTestFixture, TestMeasurePinBasedAuthentication) {
   bool connected = ConnectToHost(kPinBasedMaxConnectionTimeInSeconds);
   EXPECT_TRUE(connected);
 
   Disconnect();
   EXPECT_FALSE(connection_time_observer_->GetStateTransitionTime(
       protocol::ConnectionToHost::State::INITIALIZING,
       protocol::ConnectionToHost::State::CLOSED).is_max());
 
   int authentication_time = connection_time_observer_->GetStateTransitionTime(
       protocol::ConnectionToHost::State::INITIALIZING,
       protocol::ConnectionToHost::State::AUTHENTICATED).InMilliseconds();
   EXPECT_LE(authentication_time, kPinBasedMaxAuthenticationTimeMs);
 
   int authenticated_to_connected_time =
       connection_time_observer_->GetStateTransitionTime(
           protocol::ConnectionToHost::State::AUTHENTICATED,
           protocol::ConnectionToHost::State::CONNECTED).InMilliseconds();
   EXPECT_LE(authenticated_to_connected_time, kMaxTimeToConnectMs);
 
   connection_time_observer_->DisplayConnectionStats();
 }
 
+// Note: Make sure to restart the host before running this test. If the host
+// does not restart after a previous connection, then the first connection will

[anandc, 2015/08/13 21:35:31]

"does not restart" -> "is not restarted"

[tonychun, 2015/08/13 21:39:07]

Done.

+// be a reconnect and the second connection will be a second reconnect.
 TEST_F(ChromotingTestFixture, TestMeasureReconnectPerformance) {
   bool connected = ConnectToHost(kPinBasedMaxConnectionTimeInSeconds);
   EXPECT_TRUE(connected);
 
   Disconnect();
   EXPECT_FALSE(connection_time_observer_->GetStateTransitionTime(
       protocol::ConnectionToHost::State::INITIALIZING,
       protocol::ConnectionToHost::State::CLOSED).is_max());
 
   int authentication_time = connection_time_observer_->GetStateTransitionTime(
       protocol::ConnectionToHost::State::INITIALIZING,
       protocol::ConnectionToHost::State::AUTHENTICATED).InMilliseconds();
   EXPECT_LE(authentication_time, kPinBasedMaxAuthenticationTimeMs);
 
   int authenticated_to_connected_time =
       connection_time_observer_->GetStateTransitionTime(
           protocol::ConnectionToHost::State::AUTHENTICATED,
           protocol::ConnectionToHost::State::CONNECTED).InMilliseconds();
   EXPECT_LE(authenticated_to_connected_time, kMaxTimeToConnectMs);
 
-  connection_time_observer_->DisplayConnectionStats();
-
   // Begin reconnection to same host.
   connected = ConnectToHost(kPinBasedMaxConnectionTimeInSeconds);
   EXPECT_TRUE(connected);
 
   Disconnect();
   EXPECT_FALSE(connection_time_observer_->GetStateTransitionTime(
       protocol::ConnectionToHost::State::INITIALIZING,
       protocol::ConnectionToHost::State::CLOSED).is_max());
 
   authentication_time = connection_time_observer_->GetStateTransitionTime(
       protocol::ConnectionToHost::State::INITIALIZING,
       protocol::ConnectionToHost::State::AUTHENTICATED).InMilliseconds();
   EXPECT_LE(authentication_time, kPinBasedMaxAuthenticationTimeMs);
 
   authenticated_to_connected_time =
       connection_time_observer_->GetStateTransitionTime(
           protocol::ConnectionToHost::State::AUTHENTICATED,
           protocol::ConnectionToHost::State::CONNECTED).InMilliseconds();
   EXPECT_LE(authenticated_to_connected_time, kMaxTimeToConnectMs);
 
   connection_time_observer_->DisplayConnectionStats();
 }
 
 }  // namespace test
 }  // namespace remoting