Comment #15 on issue 27067 by bugdro...@chromium.org: Crash -  
BookmarkMenuController::BookmarkNodeFavIconLoaded(BookmarkModel  
*,BookmarkNode const *)
http://code.google.com/p/chromium/issues/detail?id=27067

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=31603

------------------------------------------------------------------------
r31603 | s...@chromium.org | 2009-11-10 13:54:49 -0800 (Tue, 10 Nov 2009) |  
10 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/bookmark_menu_controller_views.cc?r1=31603&r2=31602
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/views/controls/menu/menu_item_view.cc?r1=31603&r2=31602
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/views/controls/menu/menu_item_view.h?r1=31603&r2=31602

Fixes possible crash in showing bookmark menu. The problem occurred
when you clicked on one top level menu on the bookmark bar, moused
over another menu and then we loaded favicons for the first menu. The
code needs to figure out which menu has the icon instead of invoking
set icon on menu_.

BUG=27067
TEST=make sure bookmark menus work ok.

Review URL: http://codereview.chromium.org/388007
------------------------------------------------------------------------


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

-- 
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs

Reply via email to