Setting the touch wrench menu as default menu for ChromeOS

ui/views/controls/menu/menu_config.cc

Index: ui/views/controls/menu/menu_config.cc
diff --git a/ui/views/controls/menu/menu_config.cc b/ui/views/controls/menu/menu_config.cc
index 49be18967f6a9fef8018359b76b7db16634def9f..c6ff21d0081aa52a228dffc5ead7c8abc4acdb1c 100644
--- a/ui/views/controls/menu/menu_config.cc
+++ b/ui/views/controls/menu/menu_config.cc
@@ -33,6 +33,9 @@ MenuConfig::MenuConfig()
       arrow_width(9),
       gutter_width(0),
       separator_height(6),
+      separator_upper_height(3),
+      separator_lower_height(4),
+      separator_spacing_height(3),
       render_gutter(false),
       show_mnemonics(false),
       scroll_arrow_height(3),
@@ -41,11 +44,13 @@ MenuConfig::MenuConfig()
       show_accelerators(true),
       always_use_icon_to_label_padding(false),
       align_arrow_and_shortcut(false) {
-  // Use 40px tall menu items when running in touch optimized mode.
+#if defined (OS_WIN)

[sky, 2012/08/21 19:36:29]

Do you even need the ifdef here? We don't use ui::LAYOUT_TOUCH on chromeos
anymore.

[Mr4D (OOO till 08-26), 2012/08/21 21:34:54]

Well - fact wise you are right, but the code itself makes only sense for Windows
and it will be confusing for people looking at it otherwise. (not even
mentioning that it adds additional unneeded complexity). If you insist I remove
it of course.

+  // For Windows use 40px tall menu items when running in touch optimized mode.
   if (ui::GetDisplayLayout() == ui::LAYOUT_TOUCH) {
     item_top_margin = item_no_icon_top_margin = 12;
     item_bottom_margin = item_no_icon_bottom_margin = 13;
   }
+#endif
 }
 
 MenuConfig::~MenuConfig() {}
@@ -62,4 +67,14 @@ const MenuConfig& MenuConfig::instance() {
   return *config_instance;
 }
 
+// static
+const bool MenuConfig::IsNewMenu() {
+#if defined(USE_AURA)
+  return true;
+#else
+  return ui::GetDisplayLayout() == ui::LAYOUT_TOUCH;

[sky, 2012/08/21 19:36:29]

Why do we need this else?

[Mr4D (OOO till 08-26), 2012/08/21 21:34:54]

Because I was told that for all other operating systems we want to keep
everything as it is. The lengthy comment for this function (which I have just
removed) was trying to explain this: I was told that for ChromeOS we always want
to have the new menu style, but for the moment we want to keep the menu for all
other OS'ses as it is. There were many eMails around this topic. So we should
really keep it as such...

+#endif
+}
+
+
 }  // namespace views