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

Side by Side Diff: chrome/browser/ui/app_list/search/arc/arc_playstore_search_provider.cc

Issue 2929273002: Add the Play Store app search to the launcher. (Closed)
Patch Set: Reuse context menu. 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "chrome/browser/ui/app_list/search/arc/arc_playstore_search_provider.h"
6
7 #include <utility>
8
9 #include "base/strings/utf_string_conversions.h"
10 #include "chrome/browser/ui/app_list/arc/arc_app_list_prefs.h"
11 #include "chrome/browser/ui/app_list/search/arc/arc_playstore_search_result.h"
12 #include "components/arc/arc_bridge_service.h"
13 #include "components/arc/arc_service_manager.h"
14
15 namespace app_list {
16
17 ArcPlayStoreSearchProvider::ArcPlayStoreSearchProvider(
18 int max_results,
19 Profile* profile,
20 AppListControllerDelegate* list_controller)
21 : max_results_(max_results),
22 profile_(profile),
23 list_controller_(list_controller),
24 weak_ptr_factory_(this) {}
25
26 ArcPlayStoreSearchProvider::~ArcPlayStoreSearchProvider() = default;
27
28 void ArcPlayStoreSearchProvider::Start(bool is_voice_query,
29 const base::string16& query) {
30 // TODO(crbug.com/736027): should search for voice?
31 arc::mojom::AppInstance* app_instance =
32 arc::ArcServiceManager::Get()
33 ? ARC_GET_INSTANCE_FOR_METHOD(
34 arc::ArcServiceManager::Get()->arc_bridge_service()->app(),
35 GetRecentAndSuggestedAppsFromPlayStore)
36 : nullptr;
37
38 if (app_instance == nullptr)
39 return;
40
41 ClearResults();
42 app_instance->GetRecentAndSuggestedAppsFromPlayStore(
43 UTF16ToUTF8(query), max_results_,
44 base::Bind(&ArcPlayStoreSearchProvider::OnResults,
45 weak_ptr_factory_.GetWeakPtr()));
46 }
47
48 void ArcPlayStoreSearchProvider::Stop() {}
49
50 void ArcPlayStoreSearchProvider::OnResults(
51 std::vector<arc::mojom::AppDiscoveryResultPtr> results) {
52 for (auto& result : results) {
53 Add(base::MakeUnique<ArcPlayStoreSearchResult>(std::move(result), profile_,
dcheng 2017/06/24 00:16:47 Is it a problem if results.size() > max_results_?
Jiaquan He 2017/06/24 00:48:08 The Play Store service guarantees that the amount
dcheng 2017/06/24 00:58:28 Right, but we don't trust the ARC++ process as muc
54 list_controller_));
55 }
56 }
57
58 } // namespace app_list
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698