Restrict Instant field trial to UMA opt-in users.

Note that if the Instant experiment has already been enabled, and *then* the
user opts out of UMA, this won't disable the experiment immediately (since the
global field trial state doesn't change). Instead, it will be disabled at
browser restart. I think this is acceptable since, if the user is fiddling with
preferences after Instant has been enabled, they can already click the Instant
checkbox to turn it off.

In addition:
+ Fix a typo in the "Omnibox.QueryTime.*" histogram.
+ Update the comment in the .h file so it flows better.

BUG=91820
TEST=Disable UMA reporting (as is default in a Chromium build), and verify that the Instant experiment doesn't get enabled.


Review URL: http://codereview.chromium.org/7583012

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

1 files changed, 1 insertions, 1 deletions

diff --git a/chrome/browser/autocomplete/autocomplete.cc b/chrome/browser/autocomplete/autocomplete.cc
index b56c1bf..d728924 100644
--- a/chrome/browser/autocomplete/autocomplete.cc
+++ b/chrome/browser/autocomplete/autocomplete.cc
@@ -869,7 +869,7 @@ void AutocompleteController::Start(
       (text.length() < 6)) {
     base::TimeTicks end_time = base::TimeTicks::Now();
     std::string name = "Omnibox.QueryTime." +
-                       InstantFieldTrial::GetGroupName(profile_) +
+                       InstantFieldTrial::GetGroupName(profile_) + "." +
                        base::IntToString(text.length());
     base::Histogram* counter = base::Histogram::FactoryGet(
         name, 1, 1000, 50, base::Histogram::kUmaTargetedHistogramFlag);