commit ecd95aec6e98dc442b52fbd3edd321efd738445f
author jar@chromium.org <jar@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Wed Oct 20 23:58:17 2010
committer jar@chromium.org <jar@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Wed Oct 20 23:58:17 2010
tree 51f8f0c21b4b13ba53fb9d9c3146f7e0da3e41d9
parent aacbaac07bef57ef5ae79c2bb73cc493697523a3

A/B test of parelellism limits on DNS resolutions

Current default is a max of 50 parallel resolutions.

This test tries to set limits at 6, 8, 10, 14, and 20.
Stats gathered include the number of successful
resolutions, and well as the number of failed
resolutions.  We're looking to see if the ratio
changes as a function of the parallelism limit.

BUG=3041
r=eroman
Review URL: http://codereview.chromium.org/4010002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63297 0039d316-1c4b-4281-b951-d872f2087c98

net/base/host_resolver_impl.cc

diff --git a/net/base/host_resolver_impl.cc b/net/base/host_resolver_impl.cc
index 1d5377a..4dd8f5b 100644
--- a/net/base/host_resolver_impl.cc
+++ b/net/base/host_resolver_impl.cc
@@ -558,10 +558,10 @@
 
     UMA_HISTOGRAM_ENUMERATION("DNS.ResolveCategory", category, RESOLVE_MAX);
 
-    static bool show_experiment_histograms =
+    static bool show_speculative_experiment_histograms =
         base::FieldTrialList::Find("DnsImpact") &&
         !base::FieldTrialList::Find("DnsImpact")->group_name().empty();
-    if (show_experiment_histograms) {
+    if (show_speculative_experiment_histograms) {
       UMA_HISTOGRAM_ENUMERATION(
           base::FieldTrial::MakeName("DNS.ResolveCategory", "DnsImpact"),
           category, RESOLVE_MAX);
@@ -570,6 +570,18 @@
                                                  "DnsImpact"), duration);
       }
     }
+    static bool show_parallelism_experiment_histograms =
+        base::FieldTrialList::Find("DnsParallelism") &&
+        !base::FieldTrialList::Find("DnsParallelism")->group_name().empty();
+    if (show_parallelism_experiment_histograms) {
+      UMA_HISTOGRAM_ENUMERATION(
+          base::FieldTrial::MakeName("DNS.ResolveCategory", "DnsParallelism"),
+          category, RESOLVE_MAX);
+      if (RESOLVE_SUCCESS == category) {
+        DNS_HISTOGRAM(base::FieldTrial::MakeName("DNS.ResolveSuccess",
+                                                 "DnsParallelism"), duration);
+      }
+    }
   }