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

Unified Diff: net/quic/crypto/cert_compressor.h

Issue 14816006: Land Recent QUIC changes (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Added missing NET_PRIVATE_EXPORT to QuicWallTime Created 7 years, 7 months 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/crypto/aes_128_gcm_encrypter_test.cc ('k') | net/quic/crypto/cert_compressor.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/crypto/cert_compressor.h
diff --git a/net/quic/crypto/cert_compressor.h b/net/quic/crypto/cert_compressor.h
index cf8021bae1e9a9bb02d53de1183d1da1189395e6..587684d353be9e887afc579eb3202d6cfbc1bd7c 100644
--- a/net/quic/crypto/cert_compressor.h
+++ b/net/quic/crypto/cert_compressor.h
@@ -22,9 +22,9 @@ namespace net {
// that they already have. In the event that one of them is to be
// compressed, it can be replaced with just the hash.
// 2) The peer may provide a number of hashes that represent sets of
-// pre-shared certificates. If one of those certificates is to be
-// compressed, and it's known to the given CommonCertSet, then it can be
-// replaced with a set hash and index.
+// pre-shared certificates (CommonCertSets). If one of those certificates
+// is to be compressed, and it's known to the given CommonCertSets, then it
+// can be replaced with a set hash and certificate index.
// 3) Otherwise the certificates are compressed with zlib using a pre-shared
// dictionary that consists of the certificates handled with the above
// methods and a small chunk of common substrings.
@@ -37,8 +37,8 @@ class NET_EXPORT_PRIVATE CertCompressor {
// hashes of certificates that the peer already possesses.
static std::string CompressChain(const std::vector<std::string>& certs,
base::StringPiece client_common_set_hashes,
- base::StringPiece client_cached,
- CommonCertSet* common_set);
+ base::StringPiece client_cached_cert_hashes,
+ const CommonCertSets* common_set);
// DecompressChain decompresses the result of |CompressChain|, given in |in|,
// into a series of certificates that are written to |out_certs|.
@@ -46,7 +46,7 @@ class NET_EXPORT_PRIVATE CertCompressor {
// |common_set| contains the common certificate sets known locally.
static bool DecompressChain(base::StringPiece in,
const std::vector<std::string>& cached_certs,
- CommonCertSet* common_set,
+ const CommonCertSets* common_set,
std::vector<std::string>* out_certs);
};
« no previous file with comments | « net/quic/crypto/aes_128_gcm_encrypter_test.cc ('k') | net/quic/crypto/cert_compressor.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698