App Launcher: Webstore results are now highlighted.

chrome/browser/ui/app_list/search/app_result.cc

Index: chrome/browser/ui/app_list/search/app_result.cc
diff --git a/chrome/browser/ui/app_list/search/app_result.cc b/chrome/browser/ui/app_list/search/app_result.cc
index 0b1e7121326085b2ca5ca009c167d7a2cbcbcf5e..d8752cb13d6e24c7bc0cf5dde2171909928bcc9a 100644
--- a/chrome/browser/ui/app_list/search/app_result.cc
+++ b/chrome/browser/ui/app_list/search/app_result.cc
@@ -23,8 +23,6 @@
 #include "extensions/common/extension_icon_set.h"
 #include "extensions/common/manifest_handlers/icons_handler.h"
 #include "ui/app_list/app_list_switches.h"
-#include "ui/app_list/search/tokenized_string.h"
-#include "ui/app_list/search/tokenized_string_match.h"
 #include "ui/gfx/color_utils.h"
 #include "ui/gfx/image/image_skia_operations.h"
 
@@ -65,20 +63,6 @@ AppResult::~AppResult() {
   StopObservingExtensionRegistry();
 }
 
-void AppResult::UpdateFromMatch(const TokenizedString& title,
-                                const TokenizedStringMatch& match) {
-  const TokenizedStringMatch::Hits& hits = match.hits();
-
-  Tags tags;
-  tags.reserve(hits.size());
-  for (size_t i = 0; i < hits.size(); ++i)
-    tags.push_back(Tag(Tag::MATCH, hits[i].start(), hits[i].end()));
-
-  set_title(title.text());
-  set_title_tags(tags);
-  set_relevance(match.relevance());
-}
-
 void AppResult::UpdateFromLastLaunched(const base::Time& current_time,
                                        const base::Time& last_launched) {
   base::TimeDelta delta = current_time - last_launched;
@@ -129,6 +113,7 @@ scoped_ptr<SearchResult> AppResult::Duplicate() const {
                     display_type() == DISPLAY_RECOMMENDATION));
   copy->set_title(title());
   copy->set_title_tags(title_tags());
+  copy->set_relevance(relevance());

[calamity, 2015/04/30 06:36:06]

Hmm... Why was this not here before...?

[Matt Giuca, 2015/05/11 04:53:21]

Is not strictly necessary as the only caller of Duplicate sets the relevance to
a new value immediately after. But it's a mistake to not have it.

 
   return copy.Pass();
 }