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

Unified Diff: chrome/browser/ui/cocoa/browser/new_avatar_button_controller_unittest.mm

Issue 117533002: [Mac] Redesign of the avatar menu button (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix tabstrip width Created 6 years, 11 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: chrome/browser/ui/cocoa/browser/new_avatar_button_controller_unittest.mm
diff --git a/chrome/browser/ui/cocoa/browser/new_avatar_button_controller_unittest.mm b/chrome/browser/ui/cocoa/browser/new_avatar_button_controller_unittest.mm
new file mode 100644
index 0000000000000000000000000000000000000000..83e1ccb1ad3e653dca3aaf7160de781ab6a5835e
--- /dev/null
+++ b/chrome/browser/ui/cocoa/browser/new_avatar_button_controller_unittest.mm
@@ -0,0 +1,73 @@
+// Copyright 2014 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.
+
+#import "chrome/browser/ui/cocoa/browser/new_avatar_button_controller.h"
+
+#include "base/command_line.h"
+#include "base/mac/scoped_nsobject.h"
+#include "base/strings/sys_string_conversions.h"
+#include "chrome/browser/profiles/profiles_state.h"
+#include "chrome/browser/profiles/profile_manager.h"
+#include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/browser_window.h"
+#import "chrome/browser/ui/cocoa/browser/profile_chooser_controller.h"
+#include "chrome/browser/ui/cocoa/cocoa_profile_test.h"
+#include "chrome/browser/ui/cocoa/info_bubble_window.h"
+#include "chrome/common/chrome_switches.h"
+
+static const char kDefaultProfileName[] = "default";
+
+class NewAvatarButtonControllerTest : public CocoaProfileTest {
+ public:
+ virtual void SetUp() OVERRIDE {
+ CommandLine::ForCurrentProcess()->AppendSwitch(
+ switches::kNewProfileManagement);
+ DCHECK(CommandLine::ForCurrentProcess()->HasSwitch(
+ switches::kNewProfileManagement));
+ DCHECK(profiles::IsMultipleProfilesEnabled());
+
+ CocoaProfileTest::SetUp();
+ ASSERT_TRUE(browser());
+
+ controller_.reset(
+ [[NewAvatarButtonController alloc] initWithBrowser:browser()]);
+ }
+
+ virtual void TearDown() OVERRIDE {
+ browser()->window()->Close();
+ CocoaProfileTest::TearDown();
+ }
+
+ NSButton* button() { return [controller_ buttonView]; }
+
+ NSView* view() { return [controller_ view]; }
+
+ NewAvatarButtonController* controller() { return controller_.get(); }
+
+ private:
+ base::scoped_nsobject<NewAvatarButtonController> controller_;
+};
+
+TEST_F(NewAvatarButtonControllerTest, ButtonShown) {
+ EXPECT_FALSE([view() isHidden]);
+ EXPECT_EQ(kDefaultProfileName, base::SysNSStringToUTF8([button() title]));
+}
+
+TEST_F(NewAvatarButtonControllerTest, DoubleOpen) {
+ EXPECT_FALSE([controller() menuController]);
+
+ [button() performClick:button()];
+
+ ProfileChooserController* menu = [controller() menuController];
+ EXPECT_TRUE(menu);
+
+ [button() performClick:button()];
+ EXPECT_EQ(menu, [controller() menuController]);
+
+ // Do not animate out because that is hard to test around.
+ static_cast<InfoBubbleWindow*>(menu.window).allowedAnimations =
+ info_bubble::kAnimateNone;
+ [menu close];
+ EXPECT_FALSE([controller() menuController]);
+}

Powered by Google App Engine
This is Rietveld 408576698