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

Unified Diff: chrome/browser/ui/ash/launcher/chrome_launcher_controller.cc

Issue 2894743002: Make launching apps from shelf more intuitive (Closed)
Patch Set: Affected by new revision, gclient sync and re-upload. Created 3 years, 7 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/ui/ash/launcher/chrome_launcher_controller.cc
diff --git a/chrome/browser/ui/ash/launcher/chrome_launcher_controller.cc b/chrome/browser/ui/ash/launcher/chrome_launcher_controller.cc
index d0fda1728fc6a7a2a6264da3aee0fd1e5a69bf0d..465ef44ed504299fef3decf335f1ca363171376d 100644
--- a/chrome/browser/ui/ash/launcher/chrome_launcher_controller.cc
+++ b/chrome/browser/ui/ash/launcher/chrome_launcher_controller.cc
@@ -367,10 +367,6 @@ void ChromeLauncherController::SetV1AppStatus(const std::string& app_id,
}
}
-void ChromeLauncherController::Launch(const ash::ShelfID& id, int event_flags) {
- LaunchApp(id, ash::LAUNCH_FROM_UNKNOWN, event_flags);
-}
-
void ChromeLauncherController::Close(const ash::ShelfID& id) {
ash::ShelfItemDelegate* delegate = model_->GetShelfItemDelegate(id);
if (!delegate)
@@ -393,8 +389,9 @@ bool ChromeLauncherController::IsPlatformApp(const ash::ShelfID& id) {
void ChromeLauncherController::LaunchApp(const ash::ShelfID& id,
ash::ShelfLaunchSource source,
- int event_flags) {
- launcher_controller_helper_->LaunchApp(id, source, event_flags);
+ int event_flags,
+ int64_t display_id) {
+ launcher_controller_helper_->LaunchApp(id, source, event_flags, display_id);
}
void ChromeLauncherController::ActivateApp(const std::string& app_id,
@@ -414,7 +411,7 @@ void ChromeLauncherController::ActivateApp(const std::string& app_id,
if (!item_delegate->GetRunningApplications().empty())
SelectItemWithSource(item_delegate.get(), source);
else
- LaunchApp(shelf_id, source, event_flags);
+ LaunchApp(shelf_id, source, event_flags, display::kInvalidDisplayId);
}
void ChromeLauncherController::SetLauncherItemImage(

Powered by Google App Engine
This is Rietveld 408576698