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

Unified Diff: ui/app_list/search_controller.h

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
« no previous file with comments | « ui/app_list/search/tokenized_string.cc ('k') | ui/app_list/search_controller.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/app_list/search_controller.h
diff --git a/ui/app_list/search_controller.h b/ui/app_list/search_controller.h
index 3fd942788164ba9af02aeb73148c374b26ff6f32..f8f695836276c35c0f8f9184b0b947276e98ff1e 100644
--- a/ui/app_list/search_controller.h
+++ b/ui/app_list/search_controller.h
@@ -10,7 +10,6 @@
#include <memory>
#include "base/macros.h"
-#include "base/memory/scoped_vector.h"
#include "base/timer/timer.h"
#include "ui/app_list/app_list_export.h"
#include "ui/app_list/app_list_model.h"
@@ -49,8 +48,6 @@ class APP_LIST_EXPORT SearchController {
void AddProvider(size_t group_id, std::unique_ptr<SearchProvider> provider);
private:
- typedef ScopedVector<SearchProvider> Providers;
-
// Invoked when the search results are changed.
void OnResultsChanged();
@@ -60,6 +57,8 @@ class APP_LIST_EXPORT SearchController {
// If true, the search results are shown on the launcher start page.
bool query_for_recommendation_ = false;
+
+ using Providers = std::vector<std::unique_ptr<SearchProvider>>;
Providers providers_;
std::unique_ptr<Mixer> mixer_;
History* history_; // KeyedService, not owned.
« no previous file with comments | « ui/app_list/search/tokenized_string.cc ('k') | ui/app_list/search_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698