Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(157)

Unified Diff: net/quic/chromium/crypto/proof_source_chromium.cc

Issue 2516033003: Landing Recent QUIC changes until Mon Nov 14 04:43:50 2016 +0000 (Closed)
Patch Set: Remove unused UpdatePacketGapSentHistogram() function. Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « net/quic/chromium/crypto/proof_source_chromium.h ('k') | net/quic/chromium/crypto/proof_test_chromium.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/chromium/crypto/proof_source_chromium.cc
diff --git a/net/quic/chromium/crypto/proof_source_chromium.cc b/net/quic/chromium/crypto/proof_source_chromium.cc
index c03305d655c97b3c2a52edd26ef7a2c6f626af9e..f73d3e5a9589fa89e872e8d969c4c379328cbf93 100644
--- a/net/quic/chromium/crypto/proof_source_chromium.cc
+++ b/net/quic/chromium/crypto/proof_source_chromium.cc
@@ -85,8 +85,8 @@ bool ProofSourceChromium::GetProof(
base::StringPiece chlo_hash,
const QuicTagVector& /* connection_options */,
scoped_refptr<ProofSource::Chain>* out_chain,
- string* out_signature,
- string* out_leaf_cert_sct) {
+ QuicCryptoProof* proof) {
+ DCHECK(proof != nullptr);
DCHECK(private_key_.get()) << " this: " << this;
crypto::OpenSSLErrStackTracer err_tracer(FROM_HERE);
@@ -125,12 +125,12 @@ bool ProofSourceChromium::GetProof(
return false;
}
signature.resize(len);
- out_signature->assign(reinterpret_cast<const char*>(signature.data()),
- signature.size());
+ proof->signature.assign(reinterpret_cast<const char*>(signature.data()),
+ signature.size());
*out_chain = chain_;
VLOG(1) << "signature: "
- << base::HexEncode(out_signature->data(), out_signature->size());
- *out_leaf_cert_sct = signed_certificate_timestamp_;
+ << base::HexEncode(proof->signature.data(), proof->signature.size());
+ proof->leaf_cert_scts = signed_certificate_timestamp_;
return true;
}
@@ -146,10 +146,10 @@ void ProofSourceChromium::GetProof(const IPAddress& server_ip,
scoped_refptr<ProofSource::Chain> chain;
string signature;
string leaf_cert_sct;
- const bool ok =
- GetProof(server_ip, hostname, server_config, quic_version, chlo_hash,
- connection_options, &chain, &signature, &leaf_cert_sct);
- callback->Run(ok, chain, signature, leaf_cert_sct, nullptr /* details */);
+ QuicCryptoProof out_proof;
+ const bool ok = GetProof(server_ip, hostname, server_config, quic_version,
+ chlo_hash, connection_options, &chain, &out_proof);
+ callback->Run(ok, chain, out_proof, nullptr /* details */);
}
} // namespace net
« no previous file with comments | « net/quic/chromium/crypto/proof_source_chromium.h ('k') | net/quic/chromium/crypto/proof_test_chromium.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698