Use TaskScheduler instead of WorkerPool in multi_threaded_cert_verifier.cc.

net/cert_net/nss_ocsp_unittest.cc

Index: net/cert_net/nss_ocsp_unittest.cc
diff --git a/net/cert_net/nss_ocsp_unittest.cc b/net/cert_net/nss_ocsp_unittest.cc
index d86e6a2e91e24fd345f15e2d4fe03a9f334bd102..c8cd63e452243b4274521ddd71344c4c7e87aac8 100644
--- a/net/cert_net/nss_ocsp_unittest.cc
+++ b/net/cert_net/nss_ocsp_unittest.cc
@@ -12,6 +12,7 @@
 #include "base/logging.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
+#include "base/test/scoped_async_task_scheduler.h"
 #include "net/base/net_errors.h"
 #include "net/base/test_completion_callback.h"
 #include "net/cert/cert_status_flags.h"
@@ -124,6 +125,11 @@ class NssHttpTest : public ::testing::Test {
   const CertificateList empty_cert_list_;
 
  private:
+  // Required by MultiThreadedCertVerifier. Needs to be a

[Ryan Sleevi, 2017/02/07 22:52:09]

This seems to be an undesirable coupling.

It also seems to suggest that TaskScheduler is having the dependency issues that
I highlighted (re: net threads and IO threads and worker threads).

Perhaps we should setup another GVC to figure out how to look at this
holistically?

+  // ScopedAsyncTaskScheduler rather than a ScopedTaskScheduler because the task
+  // posted by MultiThreadedCertVerifier synchronously waits for a task running
+  // on the main thread (OCSPRequestSession::Wait).
+  base::test::ScopedAsyncTaskScheduler scoped_async_task_scheduler_;
   TestURLRequestContext context_;
   AiaResponseHandler* handler_;
   scoped_refptr<CertVerifyProc> verify_proc_;