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

Unified Diff: net/quic/quic_utils.cc

Issue 1726113002: Deprecate --gfe2_reloadable_flag_quic_utils_use_fast_incremental_hash. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@114856985
Patch Set: fix mistake made hand merging Created 4 years, 10 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/quic_flags.cc ('k') | net/quic/quic_utils_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/quic_utils.cc
diff --git a/net/quic/quic_utils.cc b/net/quic/quic_utils.cc
index ee0455ba93179157523db34fe194709b055ae07c..7d9dfeea13b37bae9a7024436ecca9213cfbbea8 100644
--- a/net/quic/quic_utils.cc
+++ b/net/quic/quic_utils.cc
@@ -56,6 +56,8 @@ uint128 IncrementalHashFast(uint128 uhash, const char* data, size_t len) {
}
#endif
+#ifndef QUIC_UTIL_HAS_UINT128
+// Slow implementation of IncrementalHash. In practice, only used by Chromium.
uint128 IncrementalHashSlow(uint128 hash, const char* data, size_t len) {
// kPrime = 309485009821345068724781371
static const uint128 kPrime(16777216, 315);
@@ -66,12 +68,11 @@ uint128 IncrementalHashSlow(uint128 hash, const char* data, size_t len) {
}
return hash;
}
+#endif
uint128 IncrementalHash(uint128 hash, const char* data, size_t len) {
#ifdef QUIC_UTIL_HAS_UINT128
- return FLAGS_quic_utils_use_fast_incremental_hash
- ? IncrementalHashFast(hash, data, len)
- : IncrementalHashSlow(hash, data, len);
+ return IncrementalHashFast(hash, data, len);
#else
return IncrementalHashSlow(hash, data, len);
#endif
« no previous file with comments | « net/quic/quic_flags.cc ('k') | net/quic/quic_utils_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698