Correct net::EmbeddedTestServer usage in some SSL related tests.

components/ssl_errors/error_classification_unittest.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 "components/ssl_errors/error_classification.h"
 
 #include "base/files/file_path.h"
 #include "base/memory/ptr_util.h"
 #include "base/message_loop/message_loop.h"
 #include "base/strings/string_split.h"
@@ skipping 320 matching lines @@
 
 // Tests that all possible NetworkClockState histogram values are recorded
 // appropriately.
 TEST_F(SSLErrorClassificationTest, NetworkClockStateHistogram) {
   base::Thread io_thread("IO thread");
   base::Thread::Options thread_options;
   thread_options.message_loop_type = base::MessageLoop::TYPE_IO;
   EXPECT_TRUE(io_thread.StartWithOptions(thread_options));
 
   net::EmbeddedTestServer test_server;
-  ASSERT_TRUE(test_server.Start());
+  ASSERT_TRUE(test_server.InitializeAndListen());
 
   base::HistogramTester histograms;
   histograms.ExpectTotalCount(kNetworkTimeHistogram, 0);
   TestingPrefServiceSimple pref_service;
   network_time::NetworkTimeTracker::RegisterPrefs(pref_service.registry());
   base::SimpleTestTickClock* tick_clock = new base::SimpleTestTickClock;
   base::SimpleTestClock* clock = new base::SimpleTestClock;
   // Do this to be sure that |is_null| returns false.
   clock->Advance(base::TimeDelta::FromDays(111));
   tick_clock->Advance(base::TimeDelta::FromDays(222));
@@ skipping 12 matching lines @@
   EXPECT_EQ(
       ssl_errors::ClockState::CLOCK_STATE_UNKNOWN,
       ssl_errors::GetClockState(base::Time::Now(), &network_time_tracker));
   histograms.ExpectTotalCount(kNetworkTimeHistogram, 1);
   histograms.ExpectBucketCount(
       kNetworkTimeHistogram,
       ssl_errors::NETWORK_CLOCK_STATE_UNKNOWN_NO_SYNC_ATTEMPT, 1);
 
   // First sync attempt is pending.
   test_server.RegisterRequestHandler(base::Bind(&NetworkErrorResponseHandler));
+  test_server.StartAcceptingConnections();
   EXPECT_TRUE(network_time_tracker.QueryTimeServiceForTesting());
   EXPECT_EQ(
       ssl_errors::ClockState::CLOCK_STATE_UNKNOWN,
       ssl_errors::GetClockState(base::Time::Now(), &network_time_tracker));
   histograms.ExpectTotalCount(kNetworkTimeHistogram, 2);
   histograms.ExpectBucketCount(
       kNetworkTimeHistogram,
       ssl_errors::NETWORK_CLOCK_STATE_UNKNOWN_FIRST_SYNC_PENDING, 1);
   network_time_tracker.WaitForFetchForTesting(123123123);
 
@@ skipping 59 matching lines @@
   clock->Advance(base::TimeDelta::FromDays(1));
   // GetClockState() will fall back to the build time heuristic.
   ssl_errors::GetClockState(clock->Now(), &network_time_tracker);
   histograms.ExpectTotalCount(kNetworkTimeHistogram, 8);
   histograms.ExpectBucketCount(
       kNetworkTimeHistogram, ssl_errors::NETWORK_CLOCK_STATE_UNKNOWN_SYNC_LOST,
       1);
 
   io_thread.Stop();
 }