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

Unified Diff: chrome/browser/ui/app_list/search/search_controller_factory.cc

Issue 2929273002: Add the Play Store app search to the launcher. (Closed)
Patch Set: Merge API and implementation. Created 3 years, 6 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/search_controller_factory.cc
diff --git a/chrome/browser/ui/app_list/search/search_controller_factory.cc b/chrome/browser/ui/app_list/search/search_controller_factory.cc
index 6220d78b5ba0e8ee7829352205d30605768620a0..338cab139bee459ff9609b09adf1f4606ad4604d 100644
--- a/chrome/browser/ui/app_list/search/search_controller_factory.cc
+++ b/chrome/browser/ui/app_list/search/search_controller_factory.cc
@@ -18,6 +18,7 @@
#include "chrome/browser/ui/app_list/search/history_factory.h"
#include "chrome/browser/ui/app_list/search/launcher_search/launcher_search_provider.h"
#include "chrome/browser/ui/app_list/search/omnibox_provider.h"
+#include "chrome/browser/ui/app_list/search/playstore/playstore_search_provider.h"
#include "chrome/browser/ui/app_list/search/suggestions/suggestions_search_provider.h"
#include "chrome/browser/ui/app_list/search/webstore/webstore_provider.h"
#include "chrome/common/chrome_switches.h"
@@ -34,6 +35,7 @@ namespace {
// Maximum number of results to show in each mixer group.
constexpr size_t kMaxAppsGroupResults = 8;
constexpr size_t kMaxOmniboxResults = 4;
+constexpr size_t kMaxPlaystoreResults = 2;
Luis Héctor Chávez 2017/06/16 22:24:35 nit: kMaxPlayStoreResults
Jiaquan He 2017/06/22 04:30:04 Done.
constexpr size_t kMaxWebstoreResults = 2;
constexpr size_t kMaxSuggestionsResults = 6;
constexpr size_t kMaxLauncherSearchResults = 2;
@@ -109,6 +111,12 @@ std::unique_ptr<SearchController> CreateSearchController(
controller->AddProvider(search_api_group_id,
base::MakeUnique<LauncherSearchProvider>(profile));
}
+ // TODO: these params are arbitrary
+ size_t playstore_api_group_id =
+ controller->AddGroup(kMaxPlaystoreResults, 1.0);
+ controller->AddProvider(
+ playstore_api_group_id,
+ base::MakeUnique<PlaystoreSearchProvider>(kMaxPlaystoreResults));
return controller;
}

Powered by Google App Engine
This is Rietveld 408576698