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

Unified Diff: chrome/browser/dom_ui/shown_sections_handler.cc

Issue 3156049: First set of changes for M7 NTP. (Closed) Base URL: git://codf21.jail/chromium.git
Patch Set: pre-review cleanup Created 10 years, 4 months 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/dom_ui/shown_sections_handler.cc
diff --git a/chrome/browser/dom_ui/shown_sections_handler.cc b/chrome/browser/dom_ui/shown_sections_handler.cc
index 80c6229f1c578f1c43f77e976e6e316de484ce81..86319da2a172a200d98c57bb3d39e6b83a05361d 100644
--- a/chrome/browser/dom_ui/shown_sections_handler.cc
+++ b/chrome/browser/dom_ui/shown_sections_handler.cc
@@ -89,7 +89,7 @@ void ShownSectionsHandler::HandleSetShownSections(const ListValue* args) {
// static
void ShownSectionsHandler::RegisterUserPrefs(PrefService* pref_service) {
pref_service->RegisterIntegerPref(prefs::kNTPShownSections,
- THUMB | RECENT | TIPS | SYNC);
+ THUMB | RECENT | TIPS | SYNC | APPS);
}
// static
@@ -113,6 +113,12 @@ void ShownSectionsHandler::MigrateUserPrefs(PrefService* pref_service,
changed = true;
}
+ if (old_pref_version < 3) {
+ // We reset people to only see apps by default in v3.
Miranda Callahan 2010/08/26 02:57:08 Should this "only" be in this spot? Or should it b
+ shown_sections = APPS;
+ changed = true;
+ }
+
if (changed)
pref_service->SetInteger(prefs::kNTPShownSections, shown_sections);
}

Powered by Google App Engine
This is Rietveld 408576698