QUIC - delete #ifdef USE_OPENSSL in unittests

net/quic/test_tools/crypto_test_utils.h

Index: net/quic/test_tools/crypto_test_utils.h
diff --git a/net/quic/test_tools/crypto_test_utils.h b/net/quic/test_tools/crypto_test_utils.h
index 3104bca5320499a09819d86696bb8047d88feb82..919649e12d8c461235ecbeac812ca37b78c6ec5e 100644
--- a/net/quic/test_tools/crypto_test_utils.h
+++ b/net/quic/test_tools/crypto_test_utils.h
@@ -127,19 +127,13 @@ class CryptoTestUtils {
   // Returns a |ProofVerifier| that uses the QUIC testing root CA.
   static ProofVerifier* ProofVerifierForTesting();
 
+  // Returns a real ProofVerifier (not a fake proof verifier) for testing.
+  static ProofVerifier* ProofVerifierRealForTesting();

[Ryan Hamilton, 2015/11/05 05:26:47]

nit: How about RealProofVerifierForTesting()?

[ramant (doing other things), 2015/11/05 17:36:43]

Done.

+
   // Returns a |ProofVerifyContext| that must be used with the verifier
   // returned by |ProofVerifierForTesting|.
   static ProofVerifyContext* ProofVerifyContextForTesting();
 
-  // These functions return a fake |ProofSource|, |ProofVerifier|, or
-  // |ProofVerifyContext| that works with each other. These are suitable for
-  // unit tests that aren't concerned with |ProofSource| and |ProofVerifier|.
-  // TODO(wtc): delete these when Chromium has a working
-  // ProofSourceForTesting().
-  static ProofSource* FakeProofSourceForTesting();
-  static ProofVerifier* FakeProofVerifierForTesting();
-  static ProofVerifyContext* FakeProofVerifyContextForTesting();
-
   // MockCommonCertSets returns a CommonCertSets that contains a single set with
   // hash |hash|, consisting of the certificate |cert| at index |index|.
   static CommonCertSets* MockCommonCertSets(base::StringPiece cert,