Setting the touch wrench menu as default menu for ChromeOS

ui/base/models/menu_model.h

Index: ui/base/models/menu_model.h
diff --git a/ui/base/models/menu_model.h b/ui/base/models/menu_model.h
index a684253ba83adaffdd6a84f61b3f44736078c4bc..78af6f61c951835bd1fdeceafb3bcbe2a47daa09 100644
--- a/ui/base/models/menu_model.h
+++ b/ui/base/models/menu_model.h
@@ -7,6 +7,7 @@
 
 #include "base/string16.h"
 #include "ui/base/models/menu_model_delegate.h"
+#include "ui/base/models/separator_types.h"
 #include "ui/base/ui_export.h"
 #include "ui/gfx/image/image_skia.h"
 #include "ui/gfx/native_widget_types.h"
@@ -57,6 +58,9 @@ class UI_EXPORT MenuModel {
   // Returns the type of item at the specified index.
   virtual ItemType GetTypeAt(int index) const = 0;
 
+  // Returns the separator type at the specified index.
+  virtual ui::MenuSeparatorStyle GetSeparatorStyleAt(int index) const = 0;

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

Why put this in a separate file instead of inside this class like the other? I'm
ok with the separate file as long you move ItemType too. It need not be with
this patch. I would name the file menu_types too.

[sky, 2012/08/21 15:27:55]

Leave this as you have it. While I prefer MenuSeparatorType, MenuSeparatorStyle
is fine. We can also move ItemType at a later date if we want to.

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

Because in one of my latest CL's I was asked to do the same thing to avoid
including the header into so many other files. This enum shows exactly the same
problems so I didn't wait for the reviewer to tell me to do so. In contrast to
ItemType however MenuSeparatorStyle is used also in MenuItemView (and thus in a
lot more places). If you still want to get that done - let me know and I do it
in this CL.

+
   // Returns the command id of the item at the specified index.
   virtual int GetCommandIdAt(int index) const = 0;