Setting the touch wrench menu as default menu for ChromeOS

ui/views/controls/menu/menu_model_adapter_unittest.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 "base/utf_string_conversions.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/models/menu_model.h"
 #include "ui/base/models/menu_model_delegate.h"
 #include "ui/views/controls/menu/menu_item_view.h"
 #include "ui/views/controls/menu/menu_model_adapter.h"
@@ skipping 32 matching lines @@
   }
 
   virtual int GetItemCount() const OVERRIDE {
     return static_cast<int>(items_.size());
   }
 
   virtual ItemType GetTypeAt(int index) const OVERRIDE {
     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. While refactoring this function
slipped apparently through the cracks.

+  }
+
   virtual int GetCommandIdAt(int index) const OVERRIDE {
     return index - GetFirstItemIndex(NULL) + command_id_base_;
   }
 
   string16 GetLabelAt(int index) const OVERRIDE {
     return items_[index - GetFirstItemIndex(NULL)].label;
   }
 
   virtual bool IsItemDynamicAt(int index) const OVERRIDE {
     return false;
@@ skipping 236 matching lines @@
     submodel->set_last_activation(-1);
   }
 
   // Check that selecting the root item is safe.  The MenuModel does
   // not care about the root so MenuModelAdapter should do nothing
   // (not hit the NOTREACHED check) when the root is selected.
   static_cast<views::MenuDelegate*>(&delegate)->SelectionChanged(menu);
 }
 
 }  // namespace views