Index: net/quic/test_tools/crypto_test_utils_chromium.cc |
diff --git a/net/quic/test_tools/crypto_test_utils_chromium.cc b/net/quic/test_tools/crypto_test_utils_chromium.cc |
index 6abc1c117f9b65f20e96b4a8a6617b3e03fcd3cc..366dd5e2dd01725f436b9931189dd7230c60b9ab 100644 |
--- a/net/quic/test_tools/crypto_test_utils_chromium.cc |
+++ b/net/quic/test_tools/crypto_test_utils_chromium.cc |
@@ -25,13 +25,15 @@ namespace { |
class TestProofVerifierChromium : public ProofVerifierChromium { |
public: |
- // TODO(rch): |transport_security_state| should be a scoped_ptr. |
- TestProofVerifierChromium(CertVerifier* cert_verifier, |
- TransportSecurityState* transport_security_state, |
- const std::string& cert_file) |
- : ProofVerifierChromium(cert_verifier, nullptr, transport_security_state), |
- cert_verifier_(cert_verifier), |
- transport_security_state_(transport_security_state) { |
+ TestProofVerifierChromium( |
+ scoped_ptr<CertVerifier> cert_verifier, |
+ scoped_ptr<TransportSecurityState> transport_security_state, |
+ const std::string& cert_file) |
+ : ProofVerifierChromium(cert_verifier.get(), |
+ nullptr, |
+ transport_security_state.get()), |
+ cert_verifier_(cert_verifier.Pass()), |
+ transport_security_state_(transport_security_state.Pass()) { |
// Load and install the root for the validated chain. |
scoped_refptr<X509Certificate> root_cert = |
ImportCertFromFile(GetTestCertsDirectory(), cert_file); |
@@ -122,7 +124,7 @@ ProofSource* CryptoTestUtils::ProofSourceForTesting() { |
// static |
ProofVerifier* CryptoTestUtils::ProofVerifierForTesting() { |
// TODO(rch): use a real cert verifier? |
- MockCertVerifier* cert_verifier = new MockCertVerifier(); |
+ scoped_ptr<MockCertVerifier> cert_verifier(new MockCertVerifier()); |
net::CertVerifyResult verify_result; |
verify_result.verified_cert = |
ImportCertFromFile(GetTestCertsDirectory(), "quic_test.example.com.crt"); |
@@ -133,7 +135,8 @@ ProofVerifier* CryptoTestUtils::ProofVerifierForTesting() { |
cert_verifier->AddResultForCertAndHost(verify_result.verified_cert.get(), |
"test.example.com", verify_result, OK); |
return new TestProofVerifierChromium( |
- cert_verifier, new TransportSecurityState, "quic_root.crt"); |
+ cert_verifier.Pass(), make_scoped_ptr(new TransportSecurityState), |
+ "quic_root.crt"); |
} |
// static |