Use TaskScheduler instead of WorkerPool in multi_threaded_cert_verifier.cc.

net/quic/chromium/quic_stream_factory_test.cc

 // Copyright (c) 2012 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 "net/quic/chromium/quic_stream_factory.h"
 
 #include <memory>
 #include <ostream>
 #include <utility>
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/memory/ptr_util.h"
+#include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
 #include "base/strings/string_util.h"
+#include "base/test/scoped_task_scheduler.h"
 #include "net/base/test_proxy_delegate.h"
 #include "net/cert/cert_verifier.h"
 #include "net/cert/ct_policy_enforcer.h"
 #include "net/cert/multi_log_ct_verifier.h"
 #include "net/dns/mock_host_resolver.h"
 #include "net/http/http_response_headers.h"
 #include "net/http/http_response_info.h"
 #include "net/http/http_server_properties_impl.h"
 #include "net/http/http_util.h"
 #include "net/http/transport_security_state.h"
@@ skipping 4938 matching lines @@
 }
 
 // Verifies that the alternative proxy server provided by the proxy delegate
 // is added to the list of supported QUIC proxy servers, and the QUIC stream
 // factory is initialized correctly.
 TEST_P(QuicStreamFactoryTest, MaybeInitializeAlternativeProxyServer) {
   VerifyInitialization(true);
 }
 
 TEST_P(QuicStreamFactoryTest, StartCertVerifyJob) {
+  base::test::ScopedTaskScheduler scoped_task_scheduler(

[eroman, 2017/02/07 22:45:45]

What is the recipe for including ScopedTaskScheduler throughout this changelist?

It is flimsy to have this everywhere that transitively depends on the verifier
code.

Is this something that can be added generically to say NetTestSuite?

[gab, 2017/02/08 16:01:42]

We could (although we'd need a way to ask for a ScopedAsyncTaskScheduler for the
odd cases where that's required)

+      base::MessageLoop::current());
+
   Initialize();
 
   MockQuicData socket_data;
   socket_data.AddRead(SYNCHRONOUS, ERR_IO_PENDING);
   socket_data.AddWrite(
       ConstructSettingsPacket(1, SETTINGS_MAX_HEADER_LIST_SIZE,
                               kDefaultMaxUncompressedHeaderSize, nullptr));
   socket_data.AddSocketDataToFactory(&socket_factory_);
 
   // Save current state of |race_cert_verification|.
@@ skipping 701 matching lines @@
   // Clear all cached states.
   factory_->ClearCachedStatesInCryptoConfig(
       base::Callback<bool(const GURL&)>());
   EXPECT_TRUE(test_cases[0].state->certs().empty());
   EXPECT_TRUE(test_cases[1].state->certs().empty());
   EXPECT_TRUE(test_cases[2].state->certs().empty());
 }
 
 }  // namespace test
 }  // namespace net