Move app list search tiles into SearchResultTileItemListView.

ui/app_list/views/search_result_container_view.cc

 // Copyright (c) 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/search_result_container_view.h"
 
 #include "base/bind.h"
 #include "base/message_loop/message_loop.h"
 
 namespace app_list {
 
 SearchResultContainerView::SearchResultContainerView()
     : results_(NULL), update_factory_(this) {
 }
 
 SearchResultContainerView::~SearchResultContainerView() {
   if (results_)
     results_->RemoveObserver(this);
 }
 
 void SearchResultContainerView::SetResults(
     AppListModel::SearchResults* results) {
   if (results_)
     results_->RemoveObserver(this);
 
   results_ = results;
   if (results_)
     results_->AddObserver(this);
 
-  Update();
+  DoUpdate();

[Matt Giuca, 2014/11/13 00:08:51]

I don't see the relevance of this change. Was this a bug before (it was not
invalidating the weak pointer factory)?

[calamity, 2014/11/13 03:44:50]

Oops. This was a relic of when I was had a PreferredSizeChanged() call in
DoUpdate(). Maybe both Layout() and PreferredSizeChanged() should move into
DoUpdate()?

 }
 
 void SearchResultContainerView::ScheduleUpdate() {
   // When search results are added one by one, each addition generates an update
   // request. Consolidates those update requests into one Update call.
   if (!update_factory_.HasWeakPtrs()) {
     base::MessageLoop::current()->PostTask(
         FROM_HERE,
         base::Bind(&SearchResultContainerView::DoUpdate,
                    update_factory_.GetWeakPtr()));
@@ skipping 16 matching lines @@
 void SearchResultContainerView::ListItemsChanged(size_t start, size_t count) {
   ScheduleUpdate();
 }
 
 void SearchResultContainerView::DoUpdate() {
   Update();
   update_factory_.InvalidateWeakPtrs();
 }
 
 }  // namespace app_list