diff options
| author | sky@google.com <sky@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-11-10 21:31:59 +0000 |
|---|---|---|
| committer | sky@google.com <sky@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-11-10 21:31:59 +0000 |
| commit | f704ee742d36f65d92893873f9c23e743cbc0608 (patch) | |
| tree | 610817635f24e86646f7810259f5639cafec4cca /chrome/browser/views/bookmark_manager_view.cc | |
| parent | f7a391a1808906f50e978355e911e2f7c2c31eb2 (diff) | |
| download | chromium_src-f704ee742d36f65d92893873f9c23e743cbc0608.zip chromium_src-f704ee742d36f65d92893873f9c23e743cbc0608.tar.gz chromium_src-f704ee742d36f65d92893873f9c23e743cbc0608.tar.bz2 | |
Changes tree/table to pass in an appropriate location when the context
menu is invoked from the keyboard.
BUG=4029
TEST=In the bookmark manager bring up the context menu of the
tree/table via the menu key and make sure the menu appears at a
reasonable location.
Review URL: http://codereview.chromium.org/10256
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@5125 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/views/bookmark_manager_view.cc')
| -rw-r--r-- | chrome/browser/views/bookmark_manager_view.cc | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/chrome/browser/views/bookmark_manager_view.cc b/chrome/browser/views/bookmark_manager_view.cc index fc6f1e3..e5bd6ae 100644 --- a/chrome/browser/views/bookmark_manager_view.cc +++ b/chrome/browser/views/bookmark_manager_view.cc @@ -463,14 +463,6 @@ void BookmarkManagerView::ShowContextMenu(views::View* source, bool is_mouse_gesture) { DCHECK(source == table_view_ || source == tree_view_); bool is_table = (source == table_view_); - if (is_table && x == -1 && y == -1) { - // TODO(sky): promote code to tableview that determines the location based - // on the selection. This is temporary until I fix that. - gfx::Point location(table_view_->width() / 2, table_view_->height() / 2); - View::ConvertPointToScreen(table_view_, &location); - x = location.x(); - y = location.y(); - } ShowMenu(GetContainer()->GetHWND(), x, y, is_table ? BookmarkContextMenu::BOOKMARK_MANAGER_TABLE : BookmarkContextMenu::BOOKMARK_MANAGER_TREE); |