[Views] If a sibling menu is selected, reset the MenuButton's PressedLock

chrome/browser/ui/views/bookmarks/bookmark_bar_view_test.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/bind.h"
 #include "base/callback.h"
 #include "base/compiler_specific.h"
 #include "base/location.h"
 #include "base/message_loop/message_loop.h"
 #include "base/prefs/pref_service.h"
@@ skipping 1698 matching lines @@
         ui_controls::DOWN | ui_controls::UP,
         CreateEventTask(this, &BookmarkBarViewTest18::Step2));
   }
 
  private:
   void Step2() {
     // Menu should be showing.
     views::MenuItemView* menu = bb_view_->GetMenu();
     ASSERT_TRUE(menu != NULL);
     ASSERT_TRUE(menu->GetSubmenu()->IsShowing());
+    // The button should be pressed.
+    EXPECT_EQ(views::Button::STATE_PRESSED,
+              bb_view_->other_bookmarks_button()->state());
 
-    // Move the mouse to the first folder on the bookmark bar
+    // Move the mouse to the first folder on the bookmark bar.
     views::LabelButton* button = GetBookmarkButton(0);
     gfx::Point button_center(button->width() / 2, button->height() / 2);
     views::View::ConvertPointToScreen(button, &button_center);
     ASSERT_TRUE(ui_controls::SendMouseMoveNotifyWhenDone(
         button_center.x(), button_center.y(),
         CreateEventTask(this, &BookmarkBarViewTest18::Step3)));
   }
 
   void Step3() {
     // Make sure the menu is showing.
     views::MenuItemView* menu = bb_view_->GetMenu();
     ASSERT_TRUE(menu != NULL);
     ASSERT_TRUE(menu->GetSubmenu()->IsShowing());
 
     // The menu for the first folder should be in the pressed state (since the
-    // menu is showing for it).
+    // menu is showing for it)...
     EXPECT_EQ(views::CustomButton::STATE_PRESSED,
               GetBookmarkButton(0)->state());
+    // ... And the "other bookmarks" button should no longer be pressed.
+    EXPECT_EQ(views::Button::STATE_NORMAL,
+              bb_view_->other_bookmarks_button()->state());
 
     menu->GetMenuController()->CancelAll();
 
     Done();
   }
 };
 
 #if defined(OS_LINUX) && !defined(OS_CHROMEOS)
 // TODO(erg): linux_aura bringup: http://crbug.com/163931
 #define MAYBE_BookmarkBarViewTest18_SiblingMenu \
@@ skipping 479 matching lines @@
 };
 
 #if defined(USE_OZONE)
 // ozone bringup - http://crbug.com/401304
 #define MAYBE_ContextMenusKeyboardEscape DISABLED_ContextMenusKeyboardEscape
 #else
 #define MAYBE_ContextMenusKeyboardEscape ContextMenusKeyboardEscape
 #endif
 VIEW_TEST(BookmarkBarViewTest24, MAYBE_ContextMenusKeyboardEscape)