Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(171)

Unified Diff: ui/views/controls/menu/menu_model_adapter_unittest.cc

Issue 10837317: Setting the touch wrench menu as default menu for ChromeOS (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed second review Created 8 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/views/controls/menu/menu_model_adapter_unittest.cc
diff --git a/ui/views/controls/menu/menu_model_adapter_unittest.cc b/ui/views/controls/menu/menu_model_adapter_unittest.cc
index 8f86335aa5a688d04642e4c336b38380b2c6d51e..7dfd573aa2c09098c3caeb7fd54d93f29fd40e9f 100644
--- a/ui/views/controls/menu/menu_model_adapter_unittest.cc
+++ b/ui/views/controls/menu/menu_model_adapter_unittest.cc
@@ -50,6 +50,12 @@ class MenuModelBase : public ui::MenuModel {
return items_[index - GetFirstItemIndex(NULL)].type;
}
+ virtual ui::MenuSeparatorType GetSeparatorStyleAt(
+ int index) const OVERRIDE {
+ return items_[index - GetFirstItemIndex(NULL)].type == TYPE_SEPARATOR ?
+ ui::NORMAL_SEPARATOR : ui::NORMAL_SEPARATOR;
sky 2012/08/21 19:36:29 Both values return ui::NORMAL_SEPARATOR here.
Mr4D (OOO till 08-26) 2012/08/21 21:34:54 Right. Happened after your last change request. Wh
+ }
+
virtual int GetCommandIdAt(int index) const OVERRIDE {
return index - GetFirstItemIndex(NULL) + command_id_base_;
}

Powered by Google App Engine
This is Rietveld 408576698