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

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

Issue 2695493002: [Mac] Touch Bar support for default browser window state (Closed)
Patch Set: Tests are back~ Created 3 years, 10 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_window_touch_bar_unittest.mm
diff --git a/chrome/browser/ui/cocoa/browser_window_touch_bar_unittest.mm b/chrome/browser/ui/cocoa/browser_window_touch_bar_unittest.mm
new file mode 100644
index 0000000000000000000000000000000000000000..71368344c3aae25f344a7c90c81db280ac19165e
--- /dev/null
+++ b/chrome/browser/ui/cocoa/browser_window_touch_bar_unittest.mm
@@ -0,0 +1,58 @@
+// Copyright 2017 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 <Cocoa/Cocoa.h>
+
+#include "base/mac/mac_util.h"
+#include "base/mac/scoped_nsobject.h"
+#include "chrome/app/chrome_command_ids.h"
+#import "chrome/browser/ui/cocoa/browser_window_touch_bar.h"
+#include "chrome/browser/ui/cocoa/test/cocoa_profile_test.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+class BrowserWindowTouchBarUnitTest : public CocoaProfileTest {
+ public:
+ void SetUp() override {
+ CocoaProfileTest::SetUp();
+ ASSERT_TRUE(browser());
+ browserWindowTouchBar_.reset(
+ [[BrowserWindowTouchBar alloc] initWithBrowser:browser()]);
+ }
+
+ void TearDown() override { CocoaProfileTest::TearDown(); }
+
+ base::scoped_nsobject<BrowserWindowTouchBar> browserWindowTouchBar_;
+};
+
+TEST_F(BrowserWindowTouchBarUnitTest, TouchBarItems) {
+ if (base::mac::IsAtMostOS10_12())
+ return;
+
+ NSTouchBar* touchBar = [browserWindowTouchBar_ makeTouchBar];
+ NSArray* touchBarItemIds = [touchBar itemIdentifiers];
+ DCHECK([touchBarItemIds containsObject:@"BackForwardTouchId"]);
Robert Sesek 2017/02/16 19:13:39 Change DCHECK -> EXPECT_TRUE
spqchan 2017/02/16 19:27:48 Done.
+ DCHECK([touchBarItemIds containsObject:@"ReloadOrStopTouchId"]);
+ DCHECK([touchBarItemIds containsObject:@"SearchTouchId"]);
+ DCHECK([touchBarItemIds containsObject:@"NewTabTouchId"]);
+ DCHECK([touchBarItemIds containsObject:@"StarTouchId"]);
+}
+
+TEST_F(BrowserWindowTouchBarUnitTest, ReloadOrStopTouchBarItem) {
+ if (base::mac::IsAtMostOS10_12())
+ return;
+
+ NSTouchBar* touchBar = [browserWindowTouchBar_ makeTouchBar];
+ [browserWindowTouchBar_ setIsPageLoading:NO];
+ NSTouchBarItem* item =
+ [browserWindowTouchBar_ touchBar:touchBar
+ makeItemForIdentifier:@"ReloadOrStopTouchId"];
+
+ DCHECK_EQ(IDC_RELOAD, [[item view] tag]);
Robert Sesek 2017/02/16 19:13:38 DCHECK_EQ -> EXPECT_EQ
spqchan 2017/02/16 19:27:48 Done.
+
+ [browserWindowTouchBar_ setIsPageLoading:YES];
+ item = [browserWindowTouchBar_ touchBar:touchBar
+ makeItemForIdentifier:@"ReloadOrStopTouchId"];
+
+ DCHECK_EQ(IDC_STOP, [[item view] tag]);
Robert Sesek 2017/02/16 19:13:39 Same.
spqchan 2017/02/16 19:27:48 Done.
+}
« no previous file with comments | « chrome/browser/ui/cocoa/browser_window_touch_bar.mm ('k') | chrome/browser/ui/cocoa/framed_browser_window.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698