diff options
author | sreeram@chromium.org <sreeram@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-04-02 22:11:37 +0000 |
---|---|---|
committer | sreeram@chromium.org <sreeram@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-04-02 22:11:37 +0000 |
commit | 0c9d6bb1242ef34989da59b8ce4d697247ba8310 (patch) | |
tree | b22660a46498c91161b7bea7ecf104e68b4deef1 /chrome/browser/instant/instant_controller.cc | |
parent | 4dfb8f00b69ec3dde0660c4790a469df5596a3ab (diff) | |
download | chromium_src-0c9d6bb1242ef34989da59b8ce4d697247ba8310.zip chromium_src-0c9d6bb1242ef34989da59b8ce4d697247ba8310.tar.gz chromium_src-0c9d6bb1242ef34989da59b8ce4d697247ba8310.tar.bz2 |
Remove kInstantEnabledTime.
This is not being used, and there are no plans to use it in the future.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/9958037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@130224 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/instant/instant_controller.cc')
-rw-r--r-- | chrome/browser/instant/instant_controller.cc | 32 |
1 files changed, 1 insertions, 31 deletions
diff --git a/chrome/browser/instant/instant_controller.cc b/chrome/browser/instant/instant_controller.cc index b1d6e3f..f6018d0 100644 --- a/chrome/browser/instant/instant_controller.cc +++ b/chrome/browser/instant/instant_controller.cc @@ -69,31 +69,12 @@ void InstantController::RegisterUserPrefs(PrefService* prefs) { prefs->RegisterBooleanPref(prefs::kInstantEnabledOnce, false, PrefService::SYNCABLE_PREF); - prefs->RegisterInt64Pref(prefs::kInstantEnabledTime, - false, - PrefService::SYNCABLE_PREF); PromoCounter::RegisterUserPrefs(prefs, prefs::kInstantPromo); } // static void InstantController::RecordMetrics(Profile* profile) { - if (!IsEnabled(profile)) - return; - - PrefService* service = profile->GetPrefs(); - if (service) { - int64 enable_time = service->GetInt64(prefs::kInstantEnabledTime); - if (!enable_time) { - service->SetInt64(prefs::kInstantEnabledTime, - base::Time::Now().ToInternalValue()); - } else { - base::TimeDelta delta = - base::Time::Now() - base::Time::FromInternalValue(enable_time); - // Histogram from 1 hour to 30 days. - UMA_HISTOGRAM_CUSTOM_COUNTS("Instant.EnabledTime.Predictive", - delta.InHours(), 1, 30 * 24, 50); - } - } + UMA_HISTOGRAM_ENUMERATION("Instant.Status", IsEnabled(profile), 2); } // static @@ -121,8 +102,6 @@ void InstantController::Enable(Profile* profile) { service->SetBoolean(prefs::kInstantEnabledOnce, true); service->SetBoolean(prefs::kInstantEnabled, true); service->SetBoolean(prefs::kInstantConfirmDialogShown, true); - service->SetInt64(prefs::kInstantEnabledTime, - base::Time::Now().ToInternalValue()); } // static @@ -131,15 +110,6 @@ void InstantController::Disable(Profile* profile) { if (!service || !IsEnabled(profile)) return; - int64 enable_time = service->GetInt64(prefs::kInstantEnabledTime); - if (enable_time) { - base::TimeDelta delta = - base::Time::Now() - base::Time::FromInternalValue(enable_time); - // Histogram from 1 minute to 10 days. - UMA_HISTOGRAM_CUSTOM_COUNTS("Instant.TimeToDisable.Predictive", - delta.InMinutes(), 1, 60 * 24 * 10, 50); - } - base::Histogram* histogram = base::LinearHistogram::FactoryGet( "Instant.Preference" + InstantFieldTrial::GetGroupName(profile), 1, 2, 3, base::Histogram::kUmaTargetedHistogramFlag); |