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

Unified Diff: chrome/browser/net/async_dns_field_trial.cc

Issue 10970017: Merge 157744 - [net] Enable async DNS field trial on CrOS (canary+dev channels). (Closed) Base URL: svn://svn.chromium.org/chrome/branches/1271/src/
Patch Set: Created 8 years, 3 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/net/async_dns_field_trial.cc
===================================================================
--- chrome/browser/net/async_dns_field_trial.cc (revision 157790)
+++ chrome/browser/net/async_dns_field_trial.cc (working copy)
@@ -11,20 +11,21 @@
namespace chrome_browser_net {
bool ConfigureAsyncDnsFieldTrial() {
-#if defined(OS_ANDROID) || defined(OS_IOS) || defined(OS_CHROMEOS)
+#if defined(OS_ANDROID) || defined(OS_IOS)
// There is no DnsConfigService on those platforms so disable the field trial.
return false;
#endif
const base::FieldTrial::Probability kAsyncDnsDivisor = 100;
base::FieldTrial::Probability enabled_probability = 0;
- // TODO(szym): expand to DEV channel after fixing http://crbug.com/121085
- if (chrome::VersionInfo::GetChannel() <= chrome::VersionInfo::CHANNEL_CANARY)
- enabled_probability = 0;
+#if defined(OS_CHROMEOS)
+ if (chrome::VersionInfo::GetChannel() <= chrome::VersionInfo::CHANNEL_DEV)
+ enabled_probability = 50;
+#endif
scoped_refptr<base::FieldTrial> trial(
base::FieldTrialList::FactoryGetFieldTrial(
- "AsyncDns", kAsyncDnsDivisor, "disabled", 2012, 9, 30, NULL));
+ "AsyncDns", kAsyncDnsDivisor, "disabled", 2012, 10, 30, NULL));
int enabled_group = trial->AppendGroup("enabled", enabled_probability);
return trial->group() == enabled_group;
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698