Enables hot-tracking for overflow extension buttons in the app menu

ui/views/controls/menu/menu_scroll_view_container.cc

Index: ui/views/controls/menu/menu_scroll_view_container.cc
diff --git a/ui/views/controls/menu/menu_scroll_view_container.cc b/ui/views/controls/menu/menu_scroll_view_container.cc
index 05c8caedb5505b4ac7c596195ec5d4aa02a4f7f0..c25f9062db669a656ae989ca02f7ae452a0eb13d 100644
--- a/ui/views/controls/menu/menu_scroll_view_container.cc
+++ b/ui/views/controls/menu/menu_scroll_view_container.cc
@@ -76,7 +76,6 @@ class MenuScrollButton : public View {
     // The background.
     gfx::Rect item_bounds(0, 0, width(), height());
     NativeTheme::ExtraParams extra;
-    extra.menu_item.is_selected = false;

[sky, 2016/02/19 01:07:14]

Why are you changing this?

[varkha, 2016/02/23 00:41:05]

Because ExtraParams now has a zero-initializing ctor so I thought default is
correct in this case. I guess it is cleaner to pass an explicit value in this
case so I reverted this change in the new patch.

[Peter Kasting, 2016/02/23 02:22:56]

Please put it back.  It is indeed better to not specify initializers that are
equivalent to the default value.

[varkha, 2016/02/23 07:30:45]

Done.

     GetNativeTheme()->Paint(canvas->sk_canvas(),
                             NativeTheme::kMenuItemBackground,
                             NativeTheme::kNormal, item_bounds, extra);