Index: chrome/browser/ui/app_list/search/app_search_provider_unittest.cc |
diff --git a/chrome/browser/ui/app_list/search/app_search_provider_unittest.cc b/chrome/browser/ui/app_list/search/app_search_provider_unittest.cc |
index ac28762643bf1f3ee25df36efba5b34025256af9..43eaf9b282245adbbbe209ade64a2bae2ffbb1e4 100644 |
--- a/chrome/browser/ui/app_list/search/app_search_provider_unittest.cc |
+++ b/chrome/browser/ui/app_list/search/app_search_provider_unittest.cc |
@@ -244,30 +244,29 @@ TEST_F(AppSearchProviderTest, FetchUnlaunchedRecommendations) { |
extensions::ExtensionPrefs* prefs = |
extensions::ExtensionPrefs::Get(profile_.get()); |
- // The order of unlaunched recommendations should be based on the app list |
- // order and be sorted below launched items. |
- prefs->SetLastLaunchTime(kHostedAppId, |
- kTestCurrentTime - base::TimeDelta::FromSeconds(5)); |
+ // The order of unlaunched recommendations should be based on the install time |
+ // order. |
+ prefs->SetLastLaunchTime(kHostedAppId, base::Time::Now()); |
prefs->SetLastLaunchTime(kPackagedApp1Id, base::Time::FromInternalValue(0)); |
prefs->SetLastLaunchTime(kPackagedApp2Id, base::Time::FromInternalValue(0)); |
EXPECT_EQ("Hosted App,Packaged App 1,Packaged App 2", RunQuery("")); |
- // Switching the app list order should change the query result. |
+ // Switching the app list order should not change the query result. |
model()->SetItemPosition( |
model()->FindItem(kPackagedApp2Id), |
model()->FindItem(kPackagedApp1Id)->position().CreateBefore()); |
- EXPECT_EQ("Hosted App,Packaged App 2,Packaged App 1", RunQuery("")); |
+ EXPECT_EQ("Hosted App,Packaged App 1,Packaged App 2", RunQuery("")); |
- // Moving an app into a folder deprioritizes it. |
+ // Moving an app into a folder should not deprioritize it. |
model()->AddItem(std::unique_ptr<AppListFolderItem>( |
new AppListFolderItem(kFolderId, AppListFolderItem::FOLDER_TYPE_NORMAL))); |
- model()->MoveItemToFolder(model()->FindItem(kPackagedApp2Id), kFolderId); |
+ model()->MoveItemToFolder(model()->FindItem(kPackagedApp1Id), kFolderId); |
EXPECT_EQ("Hosted App,Packaged App 1,Packaged App 2", RunQuery("")); |
// The position of the folder shouldn't matter. |
model()->SetItemPosition( |
model()->FindItem(kFolderId), |
- model()->FindItem(kPackagedApp1Id)->position().CreateBefore()); |
+ model()->FindItem(kPackagedApp2Id)->position().CreateBefore()); |
EXPECT_EQ("Hosted App,Packaged App 1,Packaged App 2", RunQuery("")); |
} |