Setting the touch wrench menu as default menu for ChromeOS

ui/views/controls/menu/menu_config.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_config.h"
 
 #include "build/build_config.h"
 #include "ui/base/layout.h"
 
 namespace views {
@@ skipping 23 matching lines @@
       gutter_width(0),
       separator_height(6),
       render_gutter(false),
       show_mnemonics(false),
       scroll_arrow_height(3),
       label_to_accelerator_padding(10),
       item_min_height(0),
       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 (ui::GetDisplayLayout() == ui::LAYOUT_TOUCH) {

[sky, 2012/08/21 03:03:25]

I thought we still wanted this for windows?

[Mr4D (OOO till 08-26), 2012/08/21 15:37:16]

I added this explicitly for the wrench menu. However - I do not have a Windows
system to test it out, and it seems safer to keep in then since the separator
changes do not apply there. Will add back in (for Windows only).

-    item_top_margin = item_no_icon_top_margin = 12;
-    item_bottom_margin = item_no_icon_bottom_margin = 13;
-  }
 }
 
 MenuConfig::~MenuConfig() {}
 
 void MenuConfig::Reset() {
   delete config_instance;
   config_instance = NULL;
 }
 
 // static
 const MenuConfig& MenuConfig::instance() {
   if (!config_instance)
     config_instance = Create();
   return *config_instance;
 }
 
+// static
+const bool MenuConfig::IsNewMenu() {
+#if defined(OS_CHROMEOS)

[sky, 2012/08/21 03:03:25]

USE_AURA

[Mr4D (OOO till 08-26), 2012/08/21 15:37:16]

As stated earlier - I am not so sure if this is correct, but you should know.
Done.

+  return true;
+#else
+  return ui::GetDisplayLayout() == ui::LAYOUT_TOUCH;
+#endif
+}
+
+
 }  // namespace views