diff options
author | asvitkine@chromium.org <asvitkine@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-11-29 18:17:55 +0000 |
---|---|---|
committer | asvitkine@chromium.org <asvitkine@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-11-29 18:17:55 +0000 |
commit | 57b531b405a961a7af36a54b9956c5514212ac53 (patch) | |
tree | 779b1bb62092c8b5b35b18e946fc0ab20bf1a297 /base/metrics | |
parent | 0b855e5294c87a53f732f6719ec1e56121c71d93 (diff) | |
download | chromium_src-57b531b405a961a7af36a54b9956c5514212ac53.zip chromium_src-57b531b405a961a7af36a54b9956c5514212ac53.tar.gz chromium_src-57b531b405a961a7af36a54b9956c5514212ac53.tar.bz2 |
Add temporary UMA histogram to understand how trials are getting disabled.
Only checks the uniformity 1 percent trial to avoid noise from other trials,
some of which should be legitimately expiring.
This temporary histogram can be removed once we understand the issue.
BUG=none
TEST=no visible changes
Review URL: https://chromiumcodereview.appspot.com/11412227
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170223 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/metrics')
-rw-r--r-- | base/metrics/field_trial.cc | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/base/metrics/field_trial.cc b/base/metrics/field_trial.cc index 0088103..de95c41 100644 --- a/base/metrics/field_trial.cc +++ b/base/metrics/field_trial.cc @@ -279,8 +279,12 @@ FieldTrial* FieldTrialList::FactoryGetFieldTrial( FieldTrial* field_trial = new FieldTrial(name, total_probability, default_group_name); - if (GetBuildTime() > CreateTimeFromParams(year, month, day_of_month)) + if (GetBuildTime() > CreateTimeFromParams(year, month, day_of_month)) { + // TODO(asvitkine): Temporary histogram. Remove this once it is not needed. + if (name == "UMA-Uniformity-Trial-1-Percent") + UMA_HISTOGRAM_BOOLEAN("Variations.UniformityTrialExpired", true); field_trial->Disable(); + } FieldTrialList::Register(field_trial); return field_trial; } |