Make MultiThreadedCertVerifier receive a CertVerifyProc, rather than creating one

net/base/multi_threaded_cert_verifier_unittest.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/base/multi_threaded_cert_verifier.h"
 
 #include "base/bind.h"
 #include "base/file_path.h"
 #include "base/format_macros.h"
 #include "base/stringprintf.h"
@@ skipping 31 matching lines @@
     verify_result->verified_cert = cert;
     verify_result->cert_status = CERT_STATUS_COMMON_NAME_INVALID;
     return ERR_CERT_COMMON_NAME_INVALID;
   }
 };
 
 }  // namespace
 
 class MultiThreadedCertVerifierTest : public ::testing::Test {
  public:
-  MultiThreadedCertVerifierTest() {
-    verifier_.SetCertVerifyProc(new MockCertVerifyProc());
-  }
+  MultiThreadedCertVerifierTest() : verifier_(new MockCertVerifyProc()) {}

[wtc, 2012/09/18 02:51:55]

This is good. The original code required us to create a
default CertVerifyProc only to throw it away.

   virtual ~MultiThreadedCertVerifierTest() {}
 
  protected:
   MultiThreadedCertVerifier verifier_;
 };
 
 TEST_F(MultiThreadedCertVerifierTest, CacheHit) {
   FilePath certs_dir = GetTestCertsDirectory();
   scoped_refptr<X509Certificate> test_cert(
       ImportCertFromFile(certs_dir, "ok_cert.pem"));
@@ skipping 254 matching lines @@
         EXPECT_FALSE(key1 < key2);
         EXPECT_TRUE(key2 < key1);
         break;
       default:
         FAIL() << "Invalid expectation. Can be only -1, 0, 1";
     }
   }
 }
 
 }  // namespace net