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

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

Issue 2870683002: ash: Remove ShelfModel id conversion functions. (Closed)
Patch Set: Address comments. 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
« no previous file with comments | « chrome/browser/ui/ash/launcher/launcher_context_menu.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/ash/launcher/launcher_context_menu_unittest.cc
diff --git a/chrome/browser/ui/ash/launcher/launcher_context_menu_unittest.cc b/chrome/browser/ui/ash/launcher/launcher_context_menu_unittest.cc
index 05042a66f5fc13227b73588d386406a477bb9ebb..73caaff1fbb2b1a586b212cb146a71289e8083b7 100644
--- a/chrome/browser/ui/ash/launcher/launcher_context_menu_unittest.cc
+++ b/chrome/browser/ui/ash/launcher/launcher_context_menu_unittest.cc
@@ -90,7 +90,7 @@ class LauncherContextMenuTest : public ash::test::AshTestBase {
ash::ShelfItemType shelf_item_type,
ash::WmShelf* wm_shelf) {
ash::ShelfItem item;
- item.id = ash::ShelfID("dummy id");
+ item.id = ash::ShelfID("idmockidmockidmockidmockidmockid");
item.type = shelf_item_type;
return LauncherContextMenu::Create(controller(), &item, wm_shelf);
}
@@ -220,8 +220,7 @@ TEST_F(LauncherContextMenuTest, ArcLauncherContextMenuItemCheck) {
controller()->PinAppWithID(app_id);
- const ash::ShelfID item_id = controller()->GetShelfIDForAppID(app_id);
- const ash::ShelfItem* item = controller()->GetItem(item_id);
+ const ash::ShelfItem* item = controller()->GetItem(ash::ShelfID(app_id));
ASSERT_TRUE(item);
int64_t primary_id = display::Screen::GetScreen()->GetPrimaryDisplay().id();
ash::WmShelf* wm_shelf = GetWmShelf(primary_id);
@@ -272,8 +271,7 @@ TEST_F(LauncherContextMenuTest, ArcLauncherContextMenuItemCheck) {
CreateArcWindow(window_app_id2);
arc_test().app_instance()->SendTaskCreated(2, arc_test().fake_apps()[1],
std::string());
- const ash::ShelfID item_id2 = controller()->GetShelfIDForAppID(app_id2);
- const ash::ShelfItem* item2 = controller()->GetItem(item_id2);
+ const ash::ShelfItem* item2 = controller()->GetItem(ash::ShelfID(app_id2));
ASSERT_TRUE(item2);
menu.reset(new ArcLauncherContextMenu(controller(), item2, wm_shelf));
@@ -296,8 +294,7 @@ TEST_F(LauncherContextMenuTest, ArcLauncherContextMenuItemCheck) {
CreateArcWindow(window_app_id3);
arc_test().app_instance()->SendTaskCreated(3, arc_test().fake_apps()[2],
shortcuts[0].intent_uri);
- const ash::ShelfID item_id3 = controller()->GetShelfIDForAppID(app_id3);
- const ash::ShelfItem* item3 = controller()->GetItem(item_id3);
+ const ash::ShelfItem* item3 = controller()->GetItem(ash::ShelfID(app_id3));
ASSERT_TRUE(item3);
menu.reset(new ArcLauncherContextMenu(controller(), item3, wm_shelf));
« no previous file with comments | « chrome/browser/ui/ash/launcher/launcher_context_menu.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698