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

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

Issue 2684723003: Remove LauncherItemController::Launch, cleanup. (Closed)
Patch Set: Use AppLauncherId Created 3 years, 10 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_impl_browsertest.cc
diff --git a/chrome/browser/ui/ash/launcher/chrome_launcher_controller_impl_browsertest.cc b/chrome/browser/ui/ash/launcher/chrome_launcher_controller_impl_browsertest.cc
index 2a6300f35ff2909974b20f59a2c526258abbc126..f4af721d8eaecc4a015102bf4a4e20b3cd8817bd 100644
--- a/chrome/browser/ui/ash/launcher/chrome_launcher_controller_impl_browsertest.cc
+++ b/chrome/browser/ui/ash/launcher/chrome_launcher_controller_impl_browsertest.cc
@@ -926,9 +926,9 @@ IN_PROC_BROWSER_TEST_F(ShelfAppBrowserTest, LaunchInBackground) {
LoadAndLaunchExtension("app1", extensions::LAUNCH_CONTAINER_TAB,
WindowOpenDisposition::NEW_BACKGROUND_TAB);
EXPECT_EQ(++tab_count, tab_strip->count());
- controller_->LaunchApp(last_loaded_extension_id(),
- ash::LAUNCH_FROM_UNKNOWN,
- 0);
+ controller_->LaunchApp(
+ ash::launcher::AppLauncherId(last_loaded_extension_id()),
+ ash::LAUNCH_FROM_UNKNOWN, 0);
}
// Confirm that clicking a icon for an app running in one of 2 maxmized windows
@@ -978,12 +978,11 @@ IN_PROC_BROWSER_TEST_F(ShelfAppBrowserTest, ActivateApp) {
IN_PROC_BROWSER_TEST_F(ShelfAppBrowserTest, LaunchApp) {
TabStripModel* tab_strip = browser()->tab_strip_model();
int tab_count = tab_strip->count();
- const Extension* extension =
- LoadExtension(test_data_dir_.AppendASCII("app1"));
-
- controller_->LaunchApp(extension->id(), ash::LAUNCH_FROM_UNKNOWN, 0);
+ ash::launcher::AppLauncherId id(
+ LoadExtension(test_data_dir_.AppendASCII("app1"))->id());
+ controller_->LaunchApp(id, ash::LAUNCH_FROM_UNKNOWN, 0);
EXPECT_EQ(++tab_count, tab_strip->count());
- controller_->LaunchApp(extension->id(), ash::LAUNCH_FROM_UNKNOWN, 0);
+ controller_->LaunchApp(id, ash::LAUNCH_FROM_UNKNOWN, 0);
EXPECT_EQ(++tab_count, tab_strip->count());
}

Powered by Google App Engine
This is Rietveld 408576698