diff --git a/chrome/browser/ui/views/bookmarks/bookmark_menu_controller_views.cc b/chrome/browser/ui/views/bookmarks/bookmark_menu_controller_views.cc index 35fa9780491e8..147dd525dec0c 100644 --- a/chrome/browser/ui/views/bookmarks/bookmark_menu_controller_views.cc +++ b/chrome/browser/ui/views/bookmarks/bookmark_menu_controller_views.cc @@ -52,9 +52,8 @@ void BookmarkMenuController::RunMenuAt(BookmarkBarView* bookmark_bar) { bookmark_bar_->GetAnchorPositionForButton(menu_button, &anchor); gfx::Point screen_loc; views::View::ConvertPointToScreen(menu_button, &screen_loc); - // Subtract 1 from the height to make the popup flush with the button border. gfx::Rect bounds(screen_loc.x(), screen_loc.y(), menu_button->width(), - menu_button->height() - 1); + menu_button->height()); menu_delegate_->GetBookmarkModel()->AddObserver(this); // We only delete ourself after the menu completes, so we can safely ignore // the return value. diff --git a/ui/views/controls/menu/menu_controller.cc b/ui/views/controls/menu/menu_controller.cc index 46713d7170a66..c1ba25788dd6b 100644 --- a/ui/views/controls/menu/menu_controller.cc +++ b/ui/views/controls/menu/menu_controller.cc @@ -1445,11 +1445,6 @@ void MenuController::UpdateInitialLocation(const gfx::Rect& bounds, bool context_menu) { pending_state_.context_menu = context_menu; pending_state_.initial_bounds = bounds; - if (bounds.height() > 1) { - // Inset the bounds slightly, otherwise drag coordinates don't line up - // nicely and menus close prematurely. - pending_state_.initial_bounds.Inset(0, 1); - } // Reverse anchor position for RTL languages. if (base::i18n::IsRTL() && @@ -1562,9 +1557,8 @@ bool MenuController::ShowSiblingMenu(SubmenuView* source, // It is currently not possible to show a submenu recursively in a bubble. DCHECK(!MenuItemView::IsBubble(anchor)); - // Subtract 1 from the height to make the popup flush with the button border. UpdateInitialLocation(gfx::Rect(screen_menu_loc.x(), screen_menu_loc.y(), - button->width(), button->height() - 1), + button->width(), button->height()), anchor, state_.context_menu); alt_menu->PrepareForRun( false, has_mnemonics,