Comment #4 on issue 14443 by [email protected]: [CRASH] on 2nd attempt
to delete a bookmark entry from bookmark manager
http://code.google.com/p/chromium/issues/detail?id=14443
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=18951
------------------------------------------------------------------------
r18951 | [email protected] | 2009-06-22 13:40:04 -0700 (Mon, 22 Jun 2009)
| 8 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/bookmarks/bookmark_context_menu.h?r1=18951&r2=18950
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/gtk/bookmark_manager_gtk.cc?r1=18951&r2=18950
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/gtk/bookmark_manager_gtk.h?r1=18951&r2=18950
Rebuild the bookmark context menu from scratch when selection changes.
Since the "remove" item always causes selection to change, we don't have to
worry about trying to call remove twice with the same context menu.
TEST=see bug. Also, no new crashes on linux.
BUG=14443
Review URL: http://codereview.chromium.org/132048
------------------------------------------------------------------------
--
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
-~----------~----~----~----~------~----~------~--~---
