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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: net/disk_cache/cache_creator.cc

Issue 14048014: Fix SimpleCache field trial creation. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: about:flags Created 7 years, 8 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 | « chrome/browser/chrome_browser_field_trials.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/disk_cache/cache_creator.cc
diff --git a/net/disk_cache/cache_creator.cc b/net/disk_cache/cache_creator.cc
index 412094205ec206c4b3c07e5805b6a5b557f83b9e..66ee48323ff14e65992b6d6688ec2268bd424c04 100644
--- a/net/disk_cache/cache_creator.cc
+++ b/net/disk_cache/cache_creator.cc
@@ -79,7 +79,9 @@ int CacheCreator::Run() {
// TODO(pasko): The two caches should never coexist on disk. Each individual
// cache backend should fail to initialize if it observes the index that does
// not belong to it.
- if (base::FieldTrialList::FindFullName("SimpleCacheTrial") == "Yes") {
+ const std::string experiment_name =
+ base::FieldTrialList::FindFullName("SimpleCacheTrial");
+ if (experiment_name == "ExplicitYes" || experiment_name == "ExperimentYes") {
// TODO(gavinp,pasko): While simple backend development proceeds, we're only
// testing it against net::DISK_CACHE. Turn it on for more cache types as
// appropriate.
« no previous file with comments | « chrome/browser/chrome_browser_field_trials.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698