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

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

Issue 1071093002: Implement setSearchResults. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rename to item_id. Created 5 years, 8 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_result.cc
diff --git a/chrome/browser/ui/app_list/search/launcher_search/launcher_search_result.cc b/chrome/browser/ui/app_list/search/launcher_search/launcher_search_result.cc
new file mode 100644
index 0000000000000000000000000000000000000000..9af8e620d3ad22922c38fc50d7cb7b129f6e2967
--- /dev/null
+++ b/chrome/browser/ui/app_list/search/launcher_search/launcher_search_result.cc
@@ -0,0 +1,71 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/ui/app_list/search/launcher_search/launcher_search_result.h"
+
+#include "base/strings/utf_string_conversions.h"
+#include "chrome/browser/chromeos/launcher_search_provider/service.h"
+#include "chrome/browser/extensions/extension_util.h"
+#include "extensions/common/manifest_handlers/icons_handler.h"
+
+namespace {
+
+const char delimiter = ':';
Matt Giuca 2015/04/14 03:15:44 kResultIdDelimiter.
yawano 2015/04/14 05:42:00 Done.
+
+} // namespace
+
+namespace app_list {
+
+LauncherSearchResult::LauncherSearchResult(
+ const std::string& item_id,
+ const GURL& icon_url,
+ const int discrete_value_relevance,
+ Profile* profile,
+ const extensions::Extension* extension)
+ : item_id_(item_id),
+ icon_url_(icon_url),
+ discrete_value_relevance_(discrete_value_relevance),
+ profile_(profile),
+ extension_(extension) {
+ DCHECK_GE(discrete_value_relevance, 0);
+ DCHECK_LE(discrete_value_relevance,
+ chromeos::launcher_search_provider::kMaxSearchResultScore);
+
+ set_id(extension->id() + delimiter + item_id);
Matt Giuca 2015/04/14 03:15:44 // The search result ID is comprised of the extens
yawano 2015/04/14 05:42:00 Done.
+ set_relevance(static_cast<double>(discrete_value_relevance) /
+ static_cast<double>(
+ chromeos::launcher_search_provider::kMaxSearchResultScore));
+ set_details(base::UTF8ToUTF16(extension->name()));
+
+ // TODO(yawano) Decode passed icon url and show it badged with extension
+ // icon.
+ extension_icon_image_.reset(new extensions::IconImage(
+ profile, extension, extensions::IconsInfo::GetIcons(extension),
+ GetPreferredIconDimension(), extensions::util::GetDefaultExtensionIcon(),
+ this));
+ UpdateIcon();
+}
+
+LauncherSearchResult::~LauncherSearchResult() {
+}
+
+scoped_ptr<SearchResult> LauncherSearchResult::Duplicate() const {
+ LauncherSearchResult* duplicated_result = new LauncherSearchResult(
+ item_id_, icon_url_, discrete_value_relevance_, profile_, extension_);
+ duplicated_result->set_title(title());
+ return make_scoped_ptr(duplicated_result);
+}
+
+void LauncherSearchResult::OnExtensionIconImageChanged(
+ extensions::IconImage* image) {
+ DCHECK_EQ(image, extension_icon_image_.get());
+ UpdateIcon();
+}
+
+void LauncherSearchResult::UpdateIcon() {
+ if (!extension_icon_image_->image_skia().isNull())
+ SetIcon(extension_icon_image_->image_skia());
+}
+
+} // namespace app_list

Powered by Google App Engine
This is Rietveld 408576698