Setting the touch wrench menu as default menu for ChromeOS

ui/views/controls/menu/menu_config.h

 // 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.
 
 #ifndef UI_VIEWS_CONTROLS_MENU_MENU_CONFIG_H_
 #define UI_VIEWS_CONTROLS_MENU_MENU_CONFIG_H_
 
 #include "third_party/skia/include/core/SkColor.h"
 #include "ui/gfx/font.h"
 #include "ui/views/views_export.h"
 
 namespace views {
 
 // Layout type information for menu items. Use the instance() method to obtain
 // the MenuConfig for the current platform.
 struct VIEWS_EXPORT MenuConfig {
   MenuConfig();
   ~MenuConfig();
 
   // Resets the single shared MenuConfig instance. The next time instance() is
   // invoked a new MenuConfig is created and configured.
   static void Reset();
 
   // Returns the single shared MenuConfig instance, creating if necessary.
   static const MenuConfig& instance();
 
+  // Going forward, every platform will use the new style menu.

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

You don't need to get into the specific here, something like:

Returns true if the new menu style is used.

is fine, with a:

// TODO(skuhne): fix other OSs and remove this.

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

The TODO makes no sense. As long as there is no consensus how other other OS'ses
will be handled there is no TODO.

Done

+  // However - for now only Aura will use it in all situations and the
+  // other OS'es will follow later and should remain as they are. Since this
+  // is of temporary nature this fix changes as little as possible and
+  // determines with this function if the new type or the old type should be
+  // used.
+  static const bool IsNewMenu();
+
   // Font used by menus.
   gfx::Font font;
 
   // Normal text color.
   SkColor text_color;
 
   // Submenu horizontal margin size.
   int submenu_horizontal_margin_size;
 
   // Submenu vertical margin size.
@@ skipping 37 matching lines @@
   int radio_width;
   int radio_height;
 
   // Size of the submenu arrow.
   int arrow_height;
   int arrow_width;
 
   // Width of the gutter. Only used if render_gutter is true.
   int gutter_width;
 
-  // Height of the separator.
+  // Height of a normal separator (ui::NORMAL_SEPARATOR).
   int separator_height;
 
+  // Height of a ui::UPPER_SEPARATOR.
+  int separator_upper_height;
+
+  // Height of a ui::LOWER_SEPARATOR.
+  int separator_lower_height;
+
+  // Height of a ui::SPACING_SEPARATOR.
+  int separator_spacing_height;
+
   // Whether or not the gutter should be rendered. The gutter is specific to
   // Vista.
   bool render_gutter;
 
   // Are mnemonics shown?
   bool show_mnemonics;
 
   // Height of the scroll arrow.
   int scroll_arrow_height;
 
@@ skipping 15 matching lines @@
 
  private:
   // Creates and configures a new MenuConfig as appropriate for the current
   // platform.
   static MenuConfig* Create();
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_CONTROLS_MENU_MENU_CONFIG_H_