Index: chrome/browser/ui/ash/launcher/chrome_launcher_controller_unittest.cc |
diff --git a/chrome/browser/ui/ash/launcher/chrome_launcher_controller_unittest.cc b/chrome/browser/ui/ash/launcher/chrome_launcher_controller_unittest.cc |
index 634ff70848ceb918014e4e13c6e084fce9172157..405859513705b0661d39f6d47b0bf611d6f096f5 100644 |
--- a/chrome/browser/ui/ash/launcher/chrome_launcher_controller_unittest.cc |
+++ b/chrome/browser/ui/ash/launcher/chrome_launcher_controller_unittest.cc |
@@ -1756,18 +1756,15 @@ TEST_F(ChromeLauncherControllerTest, RestoreDefaultAndLockedAppsResyncOrder) { |
// Going back to a status where there is no requirement for app 2 to be pinned |
// should convert it back to locked but not pinned and state. The position |
- // is determined by the |ShelfModel|'s weight system. Since at the moment |
- // the weight of a running app has the same as a shortcut, it will remain |
- // where it is. Surely note ideal, but since the sync process can shift around |
- // locations - as well as many other edge cases - this gets nearly impossible |
- // to get right. |
- // TODO(skuhne): Filed crbug.com/293761 to track of this. |
+ // is determined by the |ShelfModel|'s weight system and since running |
+ // applications are not allowed to be mixed with shortcuts, it should show up |
+ // at the end of the list. |
base::ListValue policy_value2; |
InsertPrefValue(&policy_value2, 0, extension3_->id()); |
InsertPrefValue(&policy_value2, 1, extension1_->id()); |
profile()->GetTestingPrefService()->SetUserPref(prefs::kPinnedLauncherApps, |
policy_value2.DeepCopy()); |
- EXPECT_EQ("AppList, Chrome, App3, app2, App1, ", GetPinnedAppStatus()); |
+ EXPECT_EQ("AppList, Chrome, App3, App1, app2, ", GetPinnedAppStatus()); |
// Removing an item should simply close it and everything should shift. |
base::ListValue policy_value3; |