Move AsyncDns field trial to server-side control.

chrome/browser/net/async_dns_field_trial.cc

Index: chrome/browser/net/async_dns_field_trial.cc
diff --git a/chrome/browser/net/async_dns_field_trial.cc b/chrome/browser/net/async_dns_field_trial.cc
index 95ce472b48897f292f4db492b635c02d788dc9b0..1e10f64e4a1038057d076c157c61ad64d0ca3095 100644
--- a/chrome/browser/net/async_dns_field_trial.cc
+++ b/chrome/browser/net/async_dns_field_trial.cc
@@ -4,9 +4,9 @@
 
 #include "chrome/browser/net/async_dns_field_trial.h"
 
+#include "base/memory/ref_counted.h"

[Alexei Svitkine (slow), 2012/10/25 17:35:11]

This isn't needed anymore, I think.

[szym, 2012/10/25 17:45:15]

Done.

 #include "base/metrics/field_trial.h"
 #include "build/build_config.h"
-#include "chrome/common/chrome_version_info.h"
 
 namespace chrome_browser_net {
 
@@ -15,18 +15,10 @@ bool ConfigureAsyncDnsFieldTrial() {
   // 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;
-
-  if (chrome::VersionInfo::GetChannel() <= chrome::VersionInfo::CHANNEL_DEV)
-    enabled_probability = 50;
-
-  scoped_refptr<base::FieldTrial> trial(
-      base::FieldTrialList::FactoryGetFieldTrial(
-          "AsyncDns", kAsyncDnsDivisor, "disabled", 2012, 10, 30, NULL));
-
-  int enabled_group = trial->AppendGroup("enabled", enabled_probability);
-  return trial->group() == enabled_group;
+  // Match both "AsyncDnsA" and "AsyncDnsB" groups.
+  const std::string kAsyncDnsPrefix("AsyncDns");
+  return base::FieldTrialList::FindFullName("AsyncDns").compare(

[Alexei Svitkine (slow), 2012/10/25 17:34:41]

Nit: Use StartsWith() from base/string_util.h.

[szym, 2012/10/25 17:45:15]

Oh wow! Thanks!

+      0, kAsyncDnsPrefix.length(), kAsyncDnsPrefix) == 0;
 }
 
 }  // namespace chrome_browser_net