diff options
| author | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-05-25 16:25:09 +0000 |
|---|---|---|
| committer | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-05-25 16:25:09 +0000 |
| commit | 199c68e2b33c9b3b5a25177006c0fb371f2e0be1 (patch) | |
| tree | 8be8192696e9e595585a35dc0a22f7dbe7cef607 /sandbox | |
| parent | f232a7f4e57fc85cb529c3aa18744b5972b79ee3 (diff) | |
| download | chromium_src-199c68e2b33c9b3b5a25177006c0fb371f2e0be1.zip chromium_src-199c68e2b33c9b3b5a25177006c0fb371f2e0be1.tar.gz chromium_src-199c68e2b33c9b3b5a25177006c0fb371f2e0be1.tar.bz2 | |
Fixes couple of issues with bookmarks in wrench menu:
. Crash when deleting via context menu. This was happening because
BookmarkMenuDelegate didn't keep a handle to the menu item it added
the items to.
. If other bookmarks folder is empty it wouldn't show a menu item for
(empty).
. Deleting the last item in the other folder would prematurely close
the menu.
. Hit DCHECK when adding empty menu (long standing issue).
BUG=83746
TEST=from the wrench menu on windows right click a bookmark, delete
it, and make sure you don't crash.
R=ben@chromium.org
Review URL: http://codereview.chromium.org/7066034
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@86634 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'sandbox')
0 files changed, 0 insertions, 0 deletions