diff options
author | rtenneti@chromium.org <rtenneti@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-01-06 21:18:10 +0000 |
---|---|---|
committer | rtenneti@chromium.org <rtenneti@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-01-06 21:18:10 +0000 |
commit | dbbf3aff4122b1e80093e6a8842525b51b5f9073 (patch) | |
tree | d1c5fa6bdec1ab39a7e052a7e620d9a5ce933ff7 /chrome/browser/net | |
parent | 5419ff930cd0fd9e6afeb82189d1a34729d6b55c (diff) | |
download | chromium_src-dbbf3aff4122b1e80093e6a8842525b51b5f9073.zip chromium_src-dbbf3aff4122b1e80093e6a8842525b51b5f9073.tar.gz chromium_src-dbbf3aff4122b1e80093e6a8842525b51b5f9073.tar.bz2 |
Fix for collecting network connectivity stats. Enabled the disabled
FieldTrial.
In Canary collect stats for all users to get more data.
R=jar
TEST=metrics service testing.
Review URL: http://codereview.chromium.org/9108027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116742 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/net')
-rw-r--r-- | chrome/browser/net/network_stats.cc | 15 |
1 files changed, 10 insertions, 5 deletions
diff --git a/chrome/browser/net/network_stats.cc b/chrome/browser/net/network_stats.cc index 075d272..919cf69 100644 --- a/chrome/browser/net/network_stats.cc +++ b/chrome/browser/net/network_stats.cc @@ -14,6 +14,7 @@ #include "base/threading/platform_thread.h" #include "base/time.h" #include "base/tuple.h" +#include "chrome/common/chrome_version_info.h" #include "content/public/browser/browser_thread.h" #include "net/base/net_errors.h" #include "net/base/net_util.h" @@ -540,19 +541,23 @@ void CollectNetworkStats(const std::string& network_stats_server, if (!trial.get()) { // Set up a field trial to collect network stats for UDP and TCP. - base::FieldTrial::Probability kDivisor = 1000; + const base::FieldTrial::Probability kDivisor = 1000; // Enable the connectivity testing for 0.5% of the users. - base::FieldTrial::Probability kProbabilityPerGroup = 5; + base::FieldTrial::Probability probability_per_group = 5; - // After October 30, 2011 builds, it will always be in default group + chrome::VersionInfo::Channel channel = chrome::VersionInfo::GetChannel(); + if (channel == chrome::VersionInfo::CHANNEL_CANARY) + probability_per_group = kDivisor; + + // After October 30, 2012 builds, it will always be in default group // (disable_network_stats). trial = new base::FieldTrial("NetworkConnectivity", kDivisor, - "disable_network_stats", 2011, 10, 30); + "disable_network_stats", 2012, 10, 30); // Add option to collect_stats for NetworkConnectivity. int collect_stats_group = trial->AppendGroup("collect_stats", - kProbabilityPerGroup); + probability_per_group); if (trial->group() == collect_stats_group) collect_stats = true; } |