Allow app list tiles to show search results in the experimental app list.

ui/app_list/views/tile_item_view.cc

 // Copyright 2014 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 "ui/app_list/views/tile_item_view.h"
 
 #include "base/strings/utf_string_conversions.h"
 #include "ui/app_list/app_list_constants.h"
 #include "ui/app_list/app_list_item.h"
 #include "ui/app_list/app_list_model.h"
 #include "ui/app_list/app_list_view_delegate.h"
+#include "ui/app_list/search_result.h"
 #include "ui/app_list/views/app_list_main_view.h"
 #include "ui/gfx/canvas.h"
 #include "ui/gfx/color_analysis.h"
 #include "ui/gfx/color_utils.h"
 #include "ui/views/background.h"
 #include "ui/views/controls/image_view.h"
 #include "ui/views/controls/label.h"
 #include "ui/views/layout/box_layout.h"
 
 namespace {
@@ skipping 64 matching lines @@
   SetLayoutManager(layout_manager);
 
   icon_->SetImageSize(gfx::Size(kTileImageSize, kTileImageSize));
 
   ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
   title_->SetAutoColorReadabilityEnabled(false);
   title_->SetEnabledColor(kGridTitleColor);
   title_->SetFontList(rb.GetFontList(kItemTextFontStyle));
   title_->SetHorizontalAlignment(gfx::ALIGN_CENTER);
 
-  // When |item_| is NULL, the tile is invisible. Calling SetAppListItem with a
+  // When |item_| is NULL, the tile is invisible. Calling SetSearchResult with a
   // non-NULL item makes the tile visible.
   SetVisible(false);
 
   AddChildView(icon_);
   AddChildView(title_);
 }
 
 TileItemView::~TileItemView() {
+  if (item_)
+    item_->RemoveObserver(this);
 }
 
-void TileItemView::SetAppListItem(AppListItem* item) {
-  // TODO(calamity): This will not update if the contents of |item_| have
-  // changed since it was last assigned. Add an observer to refresh when the
-  // item changes.
-  if (item == item_)
+void TileItemView::SetSearchResult(SearchResult* item) {
+  SetVisible(item != NULL);
+
+  SearchResult* old_item = item_;
+  if (old_item)
+    old_item->RemoveObserver(this);
+
+  item_ = item;
+
+  if (!item)
     return;
 
-  item_ = item;
-  if (!item) {
-    SetVisible(false);
-    icon_->SetImage(NULL);
-    title_->SetText(base::string16());
-    return;
+  item_->AddObserver(this);
+
+  title_->SetText(item_->title());
+
+  // Only refresh the icon if it's different from the old one. This prevents
+  // flickering.
+  if (old_item == NULL ||
+      !item->icon().BackedBySameObjectAs(old_item->icon())) {

[Matt Giuca, 2014/08/06 08:54:53]

Do this for AppListItem in a separate patch. (You address the above TODO in a
nice clean small patch.)

[calamity, 2014/08/12 05:03:37]

As discussed, doing this for AppListItems could have weird consequences. It's
safer to just do here.

+    OnIconChanged();
   }
-
-  SetVisible(true);
-  icon_->SetImage(item_->icon());
-  title_->SetText(base::UTF8ToUTF16(item_->name()));
-
-  background_->set_strip_color(
-      color_utils::CalculateKMeanColorOfBitmap(*item_->icon().bitmap()));
 }
 
 gfx::Size TileItemView::GetPreferredSize() const {
   return gfx::Size(kTileSize, kTileSize);
 }
 
 void TileItemView::ButtonPressed(views::Button* sender,
                                  const ui::Event& event) {
-  item_->Activate(event.flags());
+  item_->Open(event.flags());
+}
+
+void TileItemView::OnIconChanged() {
+  icon_->SetImage(item_->icon());
+  background_->set_strip_color(
+      color_utils::CalculateKMeanColorOfBitmap(*item_->icon().bitmap()));
+  SchedulePaint();
+}
+
+void TileItemView::OnResultDestroying() {
+  if (item_)
+    item_->RemoveObserver(this);
+  item_ = NULL;
 }
 
 }  // namespace app_list