diff options
| author | mpearson@chromium.org <mpearson@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-10-27 18:20:26 +0000 |
|---|---|---|
| committer | mpearson@chromium.org <mpearson@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-10-27 18:20:26 +0000 |
| commit | f89aa175f3b3a8b741131805502dbba7af6b3b1a (patch) | |
| tree | a7519ebf19deb37c7a61c55cc0c0d7839be9bd35 /chrome/browser/autocomplete | |
| parent | 7fd5e0e98ad663e794d691122786d1d0c70ca545 (diff) | |
| download | chromium_src-f89aa175f3b3a8b741131805502dbba7af6b3b1a.zip chromium_src-f89aa175f3b3a8b741131805502dbba7af6b3b1a.tar.gz chromium_src-f89aa175f3b3a8b741131805502dbba7af6b3b1a.tar.bz2 | |
Omnibox: Disable two field trials that are no longer needed
I set up these fields trial as trial runs for the pending field trial that moves all scoring code from HistoryURL provider to HistoryQuick provider. They were meant to serve as warning flags ot know if this move would be horrible. They didn't indicate horribleness. I'm disabling them. We probably won't need them anymore but I'm not sure, so I'm leaving the code in chromium for the moment. I'll soon be enabling the bigger-impact field trial. If that succeeds, that's probably the time I'll remove the code for these two field trials entirely.
BUG=
Review URL: https://chromiumcodereview.appspot.com/11273043
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@164548 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/autocomplete')
| -rw-r--r-- | chrome/browser/autocomplete/autocomplete_field_trial.cc | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/chrome/browser/autocomplete/autocomplete_field_trial.cc b/chrome/browser/autocomplete/autocomplete_field_trial.cc index c6dedb4..76e8fdf 100644 --- a/chrome/browser/autocomplete/autocomplete_field_trial.cc +++ b/chrome/browser/autocomplete/autocomplete_field_trial.cc @@ -45,19 +45,23 @@ const base::FieldTrial::Probability kHQPNewScoringFieldTrialDivisor = 100; const base::FieldTrial::Probability kHQPNewScoringFieldTrialExperimentFraction = 0; -// For HistoryURL provider cull redirects field trial, put 25% ( = 25/100 ) +// For HistoryURL provider cull redirects field trial, put 0% ( = 0/100 ) // of the users in the don't-cull-redirects experiment group. +// TODO(mpearson): Remove this field trial and the code it uses once I'm +// sure it's no longer needed. const base::FieldTrial::Probability kHUPCullRedirectsFieldTrialDivisor = 100; const base::FieldTrial::Probability - kHUPCullRedirectsFieldTrialExperimentFraction = 25; + kHUPCullRedirectsFieldTrialExperimentFraction = 0; -// For HistoryURL provider create shorter match field trial, put 25% +// For HistoryURL provider create shorter match field trial, put 0% // ( = 25/100 ) of the users in the don't-create-a-shorter-match // experiment group. +// TODO(mpearson): Remove this field trial and the code it uses once I'm +// sure it's no longer needed. const base::FieldTrial::Probability kHUPCreateShorterMatchFieldTrialDivisor = 100; const base::FieldTrial::Probability - kHUPCreateShorterMatchFieldTrialExperimentFraction = 25; + kHUPCreateShorterMatchFieldTrialExperimentFraction = 0; // Field trial IDs. // Though they are not literally "const", they are set only once, in |