Handle sync changes on the Start Page tiles while context menus are showing.

ui/app_list/views/search_result_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/search_result_tile_item_view.h"
 
 #include "ui/app_list/search_result.h"
 #include "ui/app_list/views/search_result_container_view.h"
 #include "ui/views/controls/menu/menu_runner.h"
 
 namespace app_list {
 
 SearchResultTileItemView::SearchResultTileItemView(
     SearchResultContainerView* result_container)
     : result_container_(result_container), item_(NULL) {
   // When |item_| is null, the tile is invisible. Calling SetSearchResult with a
   // non-null item makes the tile visible.
   SetVisible(false);
 
   set_context_menu_controller(this);
 }
 
 SearchResultTileItemView::~SearchResultTileItemView() {
   if (item_)
     item_->RemoveObserver(this);
 }
 
 void SearchResultTileItemView::SetSearchResult(SearchResult* item) {
+  // Handle the case where this may be called from a nested run loop while its
+  // context menu is showing. This cancels the menu (it's for the old item).
+  context_menu_runner_.reset();
+
   SetVisible(item != NULL);
 
   SearchResult* old_item = item_;
   if (old_item)
     old_item->RemoveObserver(this);
 
   item_ = item;
 
   if (!item)
     return;
@@ skipping 54 matching lines @@
   // If RunMenuAt() fails, return immediately. This is future-proofing for
   // adding code after this call.
   if (context_menu_runner_->RunMenuAt(
           GetWidget(), nullptr, gfx::Rect(point, gfx::Size()),
           views::MENU_ANCHOR_TOPLEFT,
           source_type) == views::MenuRunner::MENU_DELETED)
     return;
 }
 
 }  // namespace app_list