Fixes alternating keyboard and mouse hot-tracking in menus

ui/views/controls/menu/menu_item_view.cc

Index: ui/views/controls/menu/menu_item_view.cc
diff --git a/ui/views/controls/menu/menu_item_view.cc b/ui/views/controls/menu/menu_item_view.cc
index 5a4f90770d95d3c3a4c133dff1c95bd491ebcb64..8e3013b119756ffaab50866279c442f003116da6 100644
--- a/ui/views/controls/menu/menu_item_view.cc
+++ b/ui/views/controls/menu/menu_item_view.cc
@@ -547,7 +547,7 @@ void MenuItemView::Layout() {
         continue;
       int width = child->GetPreferredSize().width();
       child->SetBounds(x - width, 0, width, height());
-      x -= width - kChildXPadding;
+      x -= width + kChildXPadding;

[varkha, 2016/02/26 22:07:10]

I think this is a typo that I came across while writing a test for hot-tracking.
The reason I think we have never noticed that is because we have container views
that host child buttons in app menu and those container views ensure proper
layout.
Consider line
https://code.google.com/p/chromium/codesearch#chromium/src/ui/views/controls/...
which is using padding correctly to add to the preferred width.

     }
     // Position |icon_view|.
     const MenuConfig& config = MenuConfig::instance();