Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2241)

Unified Diff: chrome/browser/prefs/browser_prefs.cc

Issue 1432033003: Delete the auto-launch trial. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: +line break Created 5 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/prefs/browser_prefs.cc
diff --git a/chrome/browser/prefs/browser_prefs.cc b/chrome/browser/prefs/browser_prefs.cc
index 7e0866942f45979c80d3b387082165ce230c0061..32950be71848f27bb52501f4c7c53be507eeec47 100644
--- a/chrome/browser/prefs/browser_prefs.cc
+++ b/chrome/browser/prefs/browser_prefs.cc
@@ -60,7 +60,6 @@
#include "chrome/browser/ui/network_profile_bubble.h"
#include "chrome/browser/ui/prefs/prefs_tab_helper.h"
#include "chrome/browser/ui/search_engines/keyword_editor_controller.h"
-#include "chrome/browser/ui/startup/autolaunch_prompt.h"
#include "chrome/browser/ui/tabs/pinned_tab_codec.h"
#include "chrome/browser/ui/webui/flags_ui.h"
#include "chrome/browser/ui/webui/instant_ui.h"
@@ -227,14 +226,11 @@
namespace {
-#if !defined(OS_ANDROID)
-// The AutomaticProfileResetter service used this preference to save that the
-// profile reset prompt had already been shown, however, the preference has been
-// renamed in Local State. We keep the name here for now so that we can clear
-// out legacy values.
-// TODO(engedy): Remove this and usages in M42 or later. See crbug.com/398813.
-const char kLegacyProfileResetPromptMemento[] = "profile.reset_prompt_memento";
-#endif
+#if defined(OS_WIN)
+// Deprecated 11/2015 (M48). TODO(gab): delete in M52+.
+const char kShownAutoLaunchInfobarDeprecated[] =
+ "browser.shown_autolaunch_infobar";
+#endif // defined(OS_WIN)
} // namespace
@@ -357,12 +353,6 @@ void RegisterLocalState(PrefRegistrySimple* registry) {
#if defined(TOOLKIT_VIEWS)
RegisterBrowserViewLocalPrefs(registry);
#endif
-
- // Preferences registered only for migration (clearing or moving to a new key)
- // go here.
-#if !defined(OS_ANDROID)
- registry->RegisterDictionaryPref(kLegacyProfileResetPromptMemento);
-#endif // !defined(OS_ANDROID)
}
// Register prefs applicable to all profiles.
@@ -481,7 +471,6 @@ void RegisterProfilePrefs(user_prefs::PrefRegistrySyncable* registry) {
NewTabUI::RegisterProfilePrefs(registry);
PepperFlashSettingsManager::RegisterProfilePrefs(registry);
PinnedTabCodec::RegisterProfilePrefs(registry);
- RegisterAutolaunchUserPrefs(registry);
signin::RegisterProfilePrefs(registry);
#endif
@@ -521,6 +510,13 @@ void RegisterProfilePrefs(user_prefs::PrefRegistrySyncable* registry) {
#if !defined(OS_ANDROID) && !defined(OS_IOS)
browser_sync::ForeignSessionHandler::RegisterProfilePrefs(registry);
#endif
+
+ // Preferences registered only for migration (clearing or moving to a new key)
+ // go here.
+
+#if defined(OS_WIN)
+ registry->RegisterIntegerPref(kShownAutoLaunchInfobarDeprecated, 0);
+#endif // defined(OS_WIN)
}
void RegisterUserProfilePrefs(user_prefs::PrefRegistrySyncable* registry) {
@@ -549,11 +545,6 @@ void MigrateObsoleteBrowserPrefs(Profile* profile, PrefService* local_state) {
// Added 05/2014.
MigrateBrowserTabStripPrefs(local_state);
#endif
-
-#if !defined(OS_ANDROID)
- // Added 08/2014.
- local_state->ClearPref(kLegacyProfileResetPromptMemento);
-#endif
}
// This method should be periodically pruned of year+ old migrations.
@@ -568,6 +559,11 @@ void MigrateObsoleteProfilePrefs(Profile* profile) {
// Added 02/2015.
MigrateGoogleNowPrefs(profile);
#endif
+
+#if defined(OS_WIN)
+ // Added 11/2015.
+ profile_prefs->ClearPref(kShownAutoLaunchInfobarDeprecated);
+#endif
}
} // namespace chrome
« no previous file with comments | « chrome/browser/chrome_browser_field_trials_desktop.cc ('k') | chrome/browser/resources/options/browser_options.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698