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

Unified Diff: chrome/browser/ui/app_list/search/launcher_search/launcher_search_provider.cc

Issue 2701123002: AppList Performance Optimization 2 (Closed)
Patch Set: Cache the tokenized name 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/app_list/search/launcher_search/launcher_search_provider.cc
diff --git a/chrome/browser/ui/app_list/search/launcher_search/launcher_search_provider.cc b/chrome/browser/ui/app_list/search/launcher_search/launcher_search_provider.cc
index 8f8871f7dae24d4cc075352c823f9c6e9df21452..cf331fb770c916b889e4b6ba6773fcb88295393d 100644
--- a/chrome/browser/ui/app_list/search/launcher_search/launcher_search_provider.cc
+++ b/chrome/browser/ui/app_list/search/launcher_search/launcher_search_provider.cc
@@ -16,8 +16,8 @@ namespace app_list {
namespace {
-const int kLauncherSearchProviderQueryDelayInMs = 100;
-const int kLauncherSearchProviderMaxResults = 6;
+constexpr int kLauncherSearchProviderQueryDelayInMs = 100;
+constexpr int kLauncherSearchProviderMaxResults = 6;
} // namespace
@@ -64,11 +64,12 @@ void LauncherSearchProvider::SetSearchResults(
extension_results_[extension_id] = std::move(results);
// Update results with other extension results.
- ClearResults();
+ SearchProvider::Results new_results;
for (const auto& item : extension_results_) {
for (const auto& result : item.second)
- Add(result->Duplicate());
+ new_results.emplace_back(result->Duplicate());
}
+ SwapResults(&new_results);
}
void LauncherSearchProvider::DelayQuery(const base::Closure& closure) {
« no previous file with comments | « chrome/browser/ui/app_list/search/extension_app_result.cc ('k') | chrome/browser/ui/app_list/search/omnibox_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698