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

Unified Diff: ash/launcher/launcher_model_unittest.cc

Issue 10905201: Move app list (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: review issues Created 8 years, 3 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 | « ash/launcher/launcher_model.cc ('k') | ash/launcher/launcher_navigator_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/launcher/launcher_model_unittest.cc
diff --git a/ash/launcher/launcher_model_unittest.cc b/ash/launcher/launcher_model_unittest.cc
index 9a4fbcd88b983c9cb772bcd8f1fa281f97bcc1c2..cc286c6d294ca8c27a5b8c996c31c9969e8c14d4 100644
--- a/ash/launcher/launcher_model_unittest.cc
+++ b/ash/launcher/launcher_model_unittest.cc
@@ -140,65 +140,69 @@ TEST(LauncherModel, AddIndices) {
// Two initial items should have different ids.
EXPECT_NE(model.items()[0].id, model.items()[1].id);
+ // Items come after the browser.
+ int browser = 1;
+ ASSERT_EQ(ash::TYPE_BROWSER_SHORTCUT, model.items()[browser].type);
+
// Tabbed items should be after shortcut.
LauncherItem item;
int tabbed_index1 = model.Add(item);
- EXPECT_EQ(1, tabbed_index1);
+ EXPECT_EQ(browser + 1, tabbed_index1);
// Add another tabbed item, it should follow first.
int tabbed_index2 = model.Add(item);
- EXPECT_EQ(2, tabbed_index2);
+ EXPECT_EQ(browser + 2, tabbed_index2);
// APP_SHORTCUT preceed browsers.
item.type = TYPE_APP_SHORTCUT;
int app_shortcut_index1 = model.Add(item);
- EXPECT_EQ(1, app_shortcut_index1);
+ EXPECT_EQ(browser + 1, app_shortcut_index1);
item.type = TYPE_APP_SHORTCUT;
int app_shortcut_index2 = model.Add(item);
- EXPECT_EQ(2, app_shortcut_index2);
+ EXPECT_EQ(browser + 2, app_shortcut_index2);
// Check that AddAt() figures out the correct indexes for app shortcuts.
item.type = TYPE_APP_SHORTCUT;
int app_shortcut_index3 = model.AddAt(0, item);
- EXPECT_EQ(1, app_shortcut_index3);
+ EXPECT_EQ(browser + 1, app_shortcut_index3);
item.type = TYPE_APP_SHORTCUT;
int app_shortcut_index4 = model.AddAt(5, item);
- EXPECT_EQ(4, app_shortcut_index4);
+ EXPECT_EQ(browser + 4, app_shortcut_index4);
item.type = TYPE_APP_SHORTCUT;
int app_shortcut_index5 = model.AddAt(2, item);
- EXPECT_EQ(2, app_shortcut_index5);
+ EXPECT_EQ(browser + 1, app_shortcut_index5);
// Check that AddAt() figures out the correct indexes for tabs and panels.
item.type = TYPE_TABBED;
int tabbed_index3 = model.AddAt(2, item);
- EXPECT_EQ(6, tabbed_index3);
+ EXPECT_EQ(browser + 6, tabbed_index3);
item.type = TYPE_APP_PANEL;
int app_panel_index1 = model.AddAt(2, item);
- EXPECT_EQ(6, app_panel_index1);
+ EXPECT_EQ(browser + 6, app_panel_index1);
item.type = TYPE_TABBED;
int tabbed_index4 = model.AddAt(11, item);
- EXPECT_EQ(10, tabbed_index4);
+ EXPECT_EQ(browser + 10, tabbed_index4);
item.type = TYPE_APP_PANEL;
int app_panel_index2 = model.AddAt(12, item);
- EXPECT_EQ(11, app_panel_index2);
+ EXPECT_EQ(browser + 11, app_panel_index2);
item.type = TYPE_TABBED;
int tabbed_index5 = model.AddAt(7, item);
- EXPECT_EQ(7, tabbed_index5);
+ EXPECT_EQ(browser + 6, tabbed_index5);
item.type = TYPE_APP_PANEL;
int app_panel_index3 = model.AddAt(8, item);
- EXPECT_EQ(8, app_panel_index3);
+ EXPECT_EQ(browser + 7, app_panel_index3);
- // Browser shortcut and app list should still be first and last, respectively.
- EXPECT_EQ(TYPE_BROWSER_SHORTCUT, model.items()[0].type);
- EXPECT_EQ(TYPE_APP_LIST, model.items()[model.item_count() - 1].type);
+ // Browser shortcut and app list should still be first and second.
+ EXPECT_EQ(TYPE_BROWSER_SHORTCUT, model.items()[1].type);
+ EXPECT_EQ(TYPE_APP_LIST, model.items()[0].type);
}
} // namespace ash
« no previous file with comments | « ash/launcher/launcher_model.cc ('k') | ash/launcher/launcher_navigator_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698