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

Unified Diff: chrome/browser/ui/webui/ntp/app_launcher_handler.cc

Issue 93883004: Sync the launch type pref for apps. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: add test for invalid value, make tests actually check things Created 7 years 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/ui/webui/ntp/app_launcher_handler.cc
diff --git a/chrome/browser/ui/webui/ntp/app_launcher_handler.cc b/chrome/browser/ui/webui/ntp/app_launcher_handler.cc
index 06599e8ca32dfb6b91243757cfcd1e6229cc06f7..ae7e3454b7c396e601a800f4f3a24b769a1f30d3 100644
--- a/chrome/browser/ui/webui/ntp/app_launcher_handler.cc
+++ b/chrome/browser/ui/webui/ntp/app_launcher_handler.cc
@@ -562,10 +562,9 @@ void AppLauncherHandler::HandleSetLaunchType(const ListValue* args) {
base::AutoReset<bool> auto_reset(&ignore_changes_, true);
extensions::SetLaunchType(
- extension_service_->extension_prefs(),
+ extension_service_,
extension_id,
- static_cast<extensions::LaunchType>(
- static_cast<int>(launch_type)));
+ static_cast<extensions::LaunchType>(static_cast<int>(launch_type)));
}
void AppLauncherHandler::HandleUninstallApp(const ListValue* args) {

Powered by Google App Engine
This is Rietveld 408576698