Fix to correctly set accessible name of menu item when there is only one child and title is empty

views/controls/menu/menu_item_view.h

Index: views/controls/menu/menu_item_view.h
diff --git a/views/controls/menu/menu_item_view.h b/views/controls/menu/menu_item_view.h
index 85868851091c35d910f12ea7e1522f6eda629bf0..47298ee58b4485d402561f1a518a161e33d88c55 100644
--- a/views/controls/menu/menu_item_view.h
+++ b/views/controls/menu/menu_item_view.h
@@ -393,6 +393,11 @@ class VIEWS_EXPORT MenuItemView : public View {
 
   void set_controller(MenuController* controller) { controller_ = controller; }
 
+  // Returns true when |this| itself has no content and the first child is
+  // responsible for showing content.
+  // (i.e. when the label is empty and there is only one child.)
+  bool FirstChildTakesOver() const;

[sky, 2011/11/10 15:58:48]

This name is confusing. How about IsContainer with a description of:

// Returns true if this MenuItemView contains a single child
// that is responsible for rendering the content.

[hashimoto, 2011/11/11 05:30:45]

Done.

+
   // The delegate. This is only valid for the root menu item. You shouldn't
   // use this directly, instead use GetDelegate() which walks the tree as
   // as necessary.
@@ -423,9 +428,6 @@ class VIEWS_EXPORT MenuItemView : public View {
   // Title.
   string16 title_;
 
-  // Accessible name (doesn't include accelerators, etc.).
-  string16 accessible_name_;
-
   // Icon.
   SkBitmap icon_;