Update drop down menu look & feel

ui/views/controls/menu/menu_controller.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/views/controls/menu/menu_controller.h"
 
 #include "base/i18n/case_conversion.h"
 #include "base/i18n/rtl.h"
 #include "base/run_loop.h"
 #include "base/time.h"
@@ skipping 32 matching lines @@
 // Period of the scroll timer (in milliseconds).
 static const int kScrollTimerMS = 30;
 
 // Delay, in ms, between when menus are selected are moused over and the menu
 // appears.
 static const int kShowDelay = 400;
 
 // Amount of time from when the drop exits the menu and the menu is hidden.
 static const int kCloseOnExitTime = 1200;
 
+// Border width of the menu shadow (used to align drop down position/bounds)
+static const int kMenuBorderOffsetY = 2;
+static const int kMenuBorderOffsetX = 1;
+
 namespace views {
 
 namespace {
 
 // Returns true if the mnemonic of |menu| matches key.
 bool MatchesMnemonic(MenuItemView* menu, char16 key) {
   return menu->GetMnemonic() == key;
 }
 
 // Returns true if |menu| doesn't have a mnemonic and first character of the its
@@ skipping 1484 matching lines @@
 gfx::Rect MenuController::CalculateMenuBounds(MenuItemView* item,
                                               bool prefer_leading,
                                               bool* is_leading) {
   DCHECK(item);
 
   SubmenuView* submenu = item->GetSubmenu();
   DCHECK(submenu);
 
   gfx::Size pref = submenu->GetScrollViewContainer()->GetPreferredSize();
 
+  pref.set_width(pref.width() - (2 * kMenuBorderOffsetX));
+
   // Don't let the menu go too wide.
   if (item->actual_menu_position() != MenuItemView::POSITION_OVER_BOUNDS)
     pref.set_width(std::min(pref.width(),
                             item->GetDelegate()->GetMaxWidthForMenu(item)));
   if (!state_.monitor_bounds.IsEmpty())
     pref.set_width(std::min(pref.width(), state_.monitor_bounds.width()));
 
   // Assume we can honor prefer_leading.
   *is_leading = prefer_leading;
 
   int x, y;
 
   if (!item->GetParentMenuItem()) {
     // First item, position relative to initial location.
-    x = state_.initial_bounds.x();
+    x = state_.initial_bounds.x() + kMenuBorderOffsetX;
     if (item->actual_menu_position() == MenuItemView::POSITION_OVER_BOUNDS)
       y = state_.initial_bounds.y();
     else
-      y = state_.initial_bounds.bottom();
+      y = state_.initial_bounds.bottom() + kMenuBorderOffsetY;
     if (state_.anchor == MenuItemView::TOPRIGHT)
       x = x + state_.initial_bounds.width() - pref.width();
 
     if (!state_.monitor_bounds.IsEmpty() &&
         pref.height() > state_.monitor_bounds.height() &&
         item->actual_menu_position() == MenuItemView::POSITION_OVER_BOUNDS) {
       // Handle very tall menus.
       pref.set_height(state_.monitor_bounds.height());
       y = state_.monitor_bounds.y();
     } else if (!state_.monitor_bounds.IsEmpty() &&
@@ skipping 554 matching lines @@
 
 #if defined(USE_AURA)
 void MenuController::OnWindowActivated(aura::Window* active,
                                        aura::Window* old_active) {
   if (!drag_in_progress_)
     Cancel(EXIT_ALL);
 }
 #endif
 
 }  // namespace views