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

Unified Diff: net/http/http_cache_transaction.cc

Issue 11045008: Fix bug in percentage increase computation for cache sensitivity analysis, (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 2 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
Index: net/http/http_cache_transaction.cc
===================================================================
--- net/http/http_cache_transaction.cc (revision 159725)
+++ net/http/http_cache_transaction.cc (working copy)
@@ -150,11 +150,9 @@
COMPILE_ASSERT(HttpCache::Transaction::kNumValidationHeaders ==
arraysize(kValidationHeaders),
Invalid_number_of_validation_headers);
- if (!base::StringToInt(
- base::FieldTrialList::FindFullName("CacheSensitivityAnalysis"),
- &sensitivity_analysis_percent_increase_)) {
- sensitivity_analysis_percent_increase_ = 0;
- }
+ base::StringToInt(
rvargas (doing something else) 2012/10/03 02:01:09 Sorry, I'm confused now. What was the bug?. Why tw
tburkard 2012/10/03 16:47:21 This change here is to make the two groups work fo
rvargas (doing something else) 2012/10/03 22:19:05 I thought the original behavior is what you wanted
+ base::FieldTrialList::FindFullName("CacheSensitivityAnalysis"),
+ &sensitivity_analysis_percent_increase_);
}
HttpCache::Transaction::~Transaction() {
@@ -2285,7 +2283,7 @@
base::TimeDelta cache_time = base::TimeTicks::Now() - cache_io_start_;
cache_io_start_ = base::TimeTicks();
if (sensitivity_analysis_percent_increase_ > 0) {
- cache_time *= 100 + sensitivity_analysis_percent_increase_;
+ cache_time *= sensitivity_analysis_percent_increase_;
cache_time /= 100;
MessageLoop::current()->PostDelayedTask(
FROM_HERE,

Powered by Google App Engine
This is Rietveld 408576698