Deprecate FLAGS_quic_reloadable_flag_enable_async_get_proof

net/quic/chromium/crypto/proof_test_chromium.cc

 // Copyright (c) 2013 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 <memory>
 
 #include "base/files/file_path.h"
 #include "net/base/ip_endpoint.h"
 #include "net/base/net_errors.h"
 #include "net/base/test_completion_callback.h"
@@ skipping 118 matching lines @@
   std::unique_ptr<ProofVerifier> verifier(
       crypto_test_utils::ProofVerifierForTesting());
 
   const string server_config = "server config bytes";
   const string hostname = "test.example.com";
   const uint16_t port = 8443;
   const string first_chlo_hash = "first chlo hash bytes";
   const string second_chlo_hash = "first chlo hash bytes";
   const QuicVersion quic_version = GetParam();
 
+  bool called = false;
+  bool first_called = false;
+  bool ok, first_ok;
   QuicReferenceCountedPointer<ProofSource::Chain> chain;
   QuicReferenceCountedPointer<ProofSource::Chain> first_chain;
   string error_details;
   QuicCryptoProof proof, first_proof;
   QuicSocketAddress server_addr;
 
-  ASSERT_TRUE(source->GetProof(server_addr, hostname, server_config,
-                               quic_version, first_chlo_hash, QuicTagVector(),
-                               &first_chain, &first_proof));
-  ASSERT_TRUE(source->GetProof(server_addr, hostname, server_config,
-                               quic_version, second_chlo_hash, QuicTagVector(),
-                               &chain, &proof));
+  std::unique_ptr<ProofSource::Callback> cb(
+      new TestCallback(&called, &ok, &chain, &proof));
+  std::unique_ptr<ProofSource::Callback> first_cb(
+      new TestCallback(&first_called, &first_ok, &first_chain, &first_proof));
+
+  // GetProof here expects the async method to invoke the callback
+  // synchronously.
+  source->GetProof(server_addr, hostname, server_config, quic_version,
+                   first_chlo_hash, QuicTagVector(), std::move(first_cb));
+  source->GetProof(server_addr, hostname, server_config, quic_version,
+                   second_chlo_hash, QuicTagVector(), std::move(cb));
+  ASSERT_TRUE(called);
+  ASSERT_TRUE(first_called);
+  ASSERT_TRUE(ok);
+  ASSERT_TRUE(first_ok);
 
   // Check that the proof source is caching correctly:
   ASSERT_EQ(first_chain->certs, chain->certs);
   ASSERT_NE(proof.signature, first_proof.signature);
   ASSERT_EQ(first_proof.leaf_cert_scts, proof.leaf_cert_scts);
 
   RunVerification(verifier.get(), hostname, port, server_config, quic_version,
                   first_chlo_hash, chain->certs, proof.signature, true);
 
   RunVerification(verifier.get(), "foo.com", port, server_config, quic_version,
                   first_chlo_hash, chain->certs, proof.signature, false);
 
   RunVerification(verifier.get(), server_config.substr(1, string::npos), port,
                   server_config, quic_version, first_chlo_hash, chain->certs,
                   proof.signature, false);
 
   const string corrupt_signature = "1" + proof.signature;
   RunVerification(verifier.get(), hostname, port, server_config, quic_version,
                   first_chlo_hash, chain->certs, corrupt_signature, false);
 
   std::vector<string> wrong_certs;
   for (size_t i = 1; i < chain->certs.size(); i++) {
     wrong_certs.push_back(chain->certs[i]);
   }
 
   RunVerification(verifier.get(), "foo.com", port, server_config, quic_version,
                   first_chlo_hash, wrong_certs, corrupt_signature, false);
 }
 
-TEST_P(ProofTest, VerifySourceAsync) {
-  std::unique_ptr<ProofSource> source(
-      crypto_test_utils::ProofSourceForTesting());
-
-  const string server_config = "server config bytes";
-  const string hostname = "test.example.com";
-  const string first_chlo_hash = "first chlo hash bytes";
-  const string second_chlo_hash = "first chlo hash bytes";
-  const QuicVersion quic_version = GetParam();
-  QuicSocketAddress server_addr;
-
-  // Call synchronous version
-  QuicReferenceCountedPointer<ProofSource::Chain> expected_chain;
-  QuicCryptoProof expected_proof;
-  ASSERT_TRUE(source->GetProof(server_addr, hostname, server_config,
-                               quic_version, first_chlo_hash, QuicTagVector(),
-                               &expected_chain, &expected_proof));
-
-  // Call asynchronous version and compare results
-  bool called = false;
-  bool ok;
-  QuicReferenceCountedPointer<ProofSource::Chain> chain;
-  QuicCryptoProof proof;
-  std::unique_ptr<ProofSource::Callback> cb(
-      new TestCallback(&called, &ok, &chain, &proof));
-  source->GetProof(server_addr, hostname, server_config, quic_version,
-                   first_chlo_hash, QuicTagVector(), std::move(cb));
-  // TODO(gredner): whan GetProof really invokes the callback asynchronously,
-  // figure out what to do here.
-  ASSERT_TRUE(called);
-  ASSERT_TRUE(ok);
-  EXPECT_THAT(chain->certs, ::testing::ContainerEq(expected_chain->certs));
-  EXPECT_EQ(proof.leaf_cert_scts, expected_proof.leaf_cert_scts);
-}
-
 TEST_P(ProofTest, UseAfterFree) {
   std::unique_ptr<ProofSource> source(
       crypto_test_utils::ProofSourceForTesting());
 
   const string server_config = "server config bytes";
   const string hostname = "test.example.com";
   const string chlo_hash = "proof nonce bytes";
+  bool called = false;
+  bool ok;
   QuicReferenceCountedPointer<ProofSource::Chain> chain;
   string error_details;
   QuicCryptoProof proof;
   QuicSocketAddress server_addr;
+  std::unique_ptr<ProofSource::Callback> cb(
+      new TestCallback(&called, &ok, &chain, &proof));
 
-  ASSERT_TRUE(source->GetProof(server_addr, hostname, server_config, GetParam(),
-                               chlo_hash, QuicTagVector(), &chain, &proof));
+  // GetProof here expects the async method to invoke the callback
+  // synchronously.
+  source->GetProof(server_addr, hostname, server_config, GetParam(), chlo_hash,
+                   QuicTagVector(), std::move(cb));
+  ASSERT_TRUE(called);
+  ASSERT_TRUE(ok);
 
   // Make sure we can safely access results after deleting where they came from.
   EXPECT_FALSE(chain->HasOneRef());
   source = nullptr;
   EXPECT_TRUE(chain->HasOneRef());
 
   EXPECT_FALSE(chain->certs.empty());
   for (const string& cert : chain->certs) {
     EXPECT_FALSE(cert.empty());
   }
 }
 
 }  // namespace test
 }  // namespace net