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

Unified Diff: chrome/browser/ui/cocoa/tabs/tab_controller_unittest.mm

Issue 1827083004: UI: Rename MediaState to AlertState (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@bluetooth-tab-indicator
Patch Set: Keep gypi ordered Created 4 years, 9 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
« no previous file with comments | « chrome/browser/ui/cocoa/tabs/tab_controller.mm ('k') | chrome/browser/ui/cocoa/tabs/tab_strip_controller.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/cocoa/tabs/tab_controller_unittest.mm
diff --git a/chrome/browser/ui/cocoa/tabs/tab_controller_unittest.mm b/chrome/browser/ui/cocoa/tabs/tab_controller_unittest.mm
index f5fcce0147a953956165e1e76feac909a0552e55..c4fa7b7a5b443b2b530b43f0b0d02caed96992ca 100644
--- a/chrome/browser/ui/cocoa/tabs/tab_controller_unittest.mm
+++ b/chrome/browser/ui/cocoa/tabs/tab_controller_unittest.mm
@@ -10,7 +10,7 @@
#include "base/message_loop/message_loop.h"
#include "base/strings/utf_string_conversions.h"
#include "chrome/browser/ui/cocoa/cocoa_test_helper.h"
-#import "chrome/browser/ui/cocoa/tabs/media_indicator_button_cocoa.h"
+#import "chrome/browser/ui/cocoa/tabs/alert_indicator_button_cocoa.h"
#import "chrome/browser/ui/cocoa/tabs/tab_controller.h"
#import "chrome/browser/ui/cocoa/tabs/tab_controller_target.h"
#import "chrome/browser/ui/cocoa/tabs/tab_strip_drag_controller.h"
@@ -108,16 +108,16 @@ class TabControllerTest : public CocoaTest {
const TabController* controller) {
// Check whether subviews should be visible when they are supposed to be,
// given Tab size and TabRendererData state.
- const TabMediaState indicatorState =
- [[controller mediaIndicatorButton] showingMediaState];
+ const TabAlertState indicatorState =
+ [[controller alertIndicatorButton] showingAlertState];
if ([controller pinned]) {
EXPECT_EQ(1, [controller iconCapacity]);
- if (indicatorState != TAB_MEDIA_STATE_NONE) {
+ if (indicatorState != TabAlertState::NONE) {
EXPECT_FALSE([controller shouldShowIcon]);
- EXPECT_TRUE([controller shouldShowMediaIndicator]);
+ EXPECT_TRUE([controller shouldShowAlertIndicator]);
} else {
EXPECT_TRUE([controller shouldShowIcon]);
- EXPECT_FALSE([controller shouldShowMediaIndicator]);
+ EXPECT_FALSE([controller shouldShowAlertIndicator]);
}
EXPECT_FALSE([controller shouldShowCloseButton]);
} else if ([controller selected]) {
@@ -126,24 +126,24 @@ class TabControllerTest : public CocoaTest {
case 0:
case 1:
EXPECT_FALSE([controller shouldShowIcon]);
- EXPECT_FALSE([controller shouldShowMediaIndicator]);
+ EXPECT_FALSE([controller shouldShowAlertIndicator]);
break;
case 2:
- if (indicatorState != TAB_MEDIA_STATE_NONE) {
+ if (indicatorState != TabAlertState::NONE) {
EXPECT_FALSE([controller shouldShowIcon]);
- EXPECT_TRUE([controller shouldShowMediaIndicator]);
+ EXPECT_TRUE([controller shouldShowAlertIndicator]);
} else {
EXPECT_TRUE([controller shouldShowIcon]);
- EXPECT_FALSE([controller shouldShowMediaIndicator]);
+ EXPECT_FALSE([controller shouldShowAlertIndicator]);
}
break;
default:
EXPECT_LE(3, [controller iconCapacity]);
EXPECT_TRUE([controller shouldShowIcon]);
- if (indicatorState != TAB_MEDIA_STATE_NONE)
- EXPECT_TRUE([controller shouldShowMediaIndicator]);
+ if (indicatorState != TabAlertState::NONE)
+ EXPECT_TRUE([controller shouldShowAlertIndicator]);
else
- EXPECT_FALSE([controller shouldShowMediaIndicator]);
+ EXPECT_FALSE([controller shouldShowAlertIndicator]);
break;
}
} else { // Tab not selected/active and not pinned tab.
@@ -151,25 +151,25 @@ class TabControllerTest : public CocoaTest {
case 0:
EXPECT_FALSE([controller shouldShowCloseButton]);
EXPECT_FALSE([controller shouldShowIcon]);
- EXPECT_FALSE([controller shouldShowMediaIndicator]);
+ EXPECT_FALSE([controller shouldShowAlertIndicator]);
break;
case 1:
EXPECT_FALSE([controller shouldShowCloseButton]);
- if (indicatorState != TAB_MEDIA_STATE_NONE) {
+ if (indicatorState != TabAlertState::NONE) {
EXPECT_FALSE([controller shouldShowIcon]);
- EXPECT_TRUE([controller shouldShowMediaIndicator]);
+ EXPECT_TRUE([controller shouldShowAlertIndicator]);
} else {
EXPECT_TRUE([controller shouldShowIcon]);
- EXPECT_FALSE([controller shouldShowMediaIndicator]);
+ EXPECT_FALSE([controller shouldShowAlertIndicator]);
}
break;
default:
EXPECT_LE(2, [controller iconCapacity]);
EXPECT_TRUE([controller shouldShowIcon]);
- if (indicatorState != TAB_MEDIA_STATE_NONE)
- EXPECT_TRUE([controller shouldShowMediaIndicator]);
+ if (indicatorState != TabAlertState::NONE)
+ EXPECT_TRUE([controller shouldShowAlertIndicator]);
else
- EXPECT_FALSE([controller shouldShowMediaIndicator]);
+ EXPECT_FALSE([controller shouldShowAlertIndicator]);
break;
}
}
@@ -178,8 +178,8 @@ class TabControllerTest : public CocoaTest {
EXPECT_TRUE([controller shouldShowIcon] ==
(!![controller iconView] && ![[controller iconView] isHidden]));
EXPECT_TRUE([controller pinned] == [[controller tabView] titleHidden]);
- EXPECT_TRUE([controller shouldShowMediaIndicator] ==
- ![[controller mediaIndicatorButton] isHidden]);
+ EXPECT_TRUE([controller shouldShowAlertIndicator] ==
+ ![[controller alertIndicatorButton] isHidden]);
EXPECT_TRUE([controller shouldShowCloseButton] !=
[[controller closeButton] isHidden]);
@@ -195,22 +195,22 @@ class TabControllerTest : public CocoaTest {
EXPECT_LE(NSMinY(tabFrame), NSMinY(iconFrame));
EXPECT_LE(NSMaxY(iconFrame), NSMaxY(tabFrame));
}
- if ([controller shouldShowIcon] && [controller shouldShowMediaIndicator]) {
+ if ([controller shouldShowIcon] && [controller shouldShowAlertIndicator]) {
EXPECT_LE(NSMaxX([[controller iconView] frame]),
- NSMinX([[controller mediaIndicatorButton] frame]));
+ NSMinX([[controller alertIndicatorButton] frame]));
}
- if ([controller shouldShowMediaIndicator]) {
- const NSRect mediaIndicatorFrame =
- [[controller mediaIndicatorButton] frame];
+ if ([controller shouldShowAlertIndicator]) {
+ const NSRect alertIndicatorFrame =
+ [[controller alertIndicatorButton] frame];
if (NSWidth(titleFrame) > 0)
- EXPECT_LE(NSMaxX(titleFrame), NSMinX(mediaIndicatorFrame));
- EXPECT_LE(NSMaxX(mediaIndicatorFrame), NSMaxX(tabFrame));
- EXPECT_LE(NSMinY(tabFrame), NSMinY(mediaIndicatorFrame));
- EXPECT_LE(NSMaxY(mediaIndicatorFrame), NSMaxY(tabFrame));
+ EXPECT_LE(NSMaxX(titleFrame), NSMinX(alertIndicatorFrame));
+ EXPECT_LE(NSMaxX(alertIndicatorFrame), NSMaxX(tabFrame));
+ EXPECT_LE(NSMinY(tabFrame), NSMinY(alertIndicatorFrame));
+ EXPECT_LE(NSMaxY(alertIndicatorFrame), NSMaxY(tabFrame));
}
- if ([controller shouldShowMediaIndicator] &&
+ if ([controller shouldShowAlertIndicator] &&
[controller shouldShowCloseButton]) {
- EXPECT_LE(NSMaxX([[controller mediaIndicatorButton] frame]),
+ EXPECT_LE(NSMaxX([[controller alertIndicatorButton] frame]),
NSMinX([[controller closeButton] frame]));
}
if ([controller shouldShowCloseButton]) {
@@ -474,14 +474,14 @@ TEST_F(TabControllerTest, TitleViewLayout) {
}
// A comprehensive test of the layout and visibility of all elements (favicon,
-// throbber indicators, titile text, media indicator button, and close button)
+// throbber indicators, titile text, alert indicator button, and close button)
// over all relevant combinations of tab state. This test overlaps with parts
// of the other tests above.
// Flaky: https://code.google.com/p/chromium/issues/detail?id=311668
TEST_F(TabControllerTest, DISABLED_LayoutAndVisibilityOfSubviews) {
- static const TabMediaState kMediaStatesToTest[] = {
- TAB_MEDIA_STATE_NONE, TAB_MEDIA_STATE_CAPTURING,
- TAB_MEDIA_STATE_AUDIO_PLAYING, TAB_MEDIA_STATE_AUDIO_MUTING
+ static const TabAlertState kAlertStatesToTest[] = {
+ TabAlertState::NONE, TabAlertState::TAB_CAPTURING,
+ TabAlertState::AUDIO_PLAYING, TabAlertState::AUDIO_MUTING
};
NSWindow* const window = test_window();
@@ -495,32 +495,33 @@ TEST_F(TabControllerTest, DISABLED_LayoutAndVisibilityOfSubviews) {
base::scoped_nsobject<NSImage> favicon(
rb.GetNativeImageNamed(IDR_DEFAULT_FAVICON).CopyNSImage());
- // Trigger TabController to auto-create the MediaIndicatorButton.
- [controller setMediaState:TAB_MEDIA_STATE_AUDIO_PLAYING];
- [controller setMediaState:TAB_MEDIA_STATE_NONE];
- base::scoped_nsobject<MediaIndicatorButton> mediaIndicatorButton(
- [[controller mediaIndicatorButton] retain]);
- ASSERT_TRUE(mediaIndicatorButton.get());
+ // Trigger TabController to auto-create the AlertIndicatorButton.
+ [controller setAlertState:TabAlertState::AUDIO_PLAYING];
+ [controller setAlertState:TabAlertState::NONE];
+ base::scoped_nsobject<AlertIndicatorButton> alertIndicatorButton(
+ [[controller alertIndicatorButton] retain]);
+ ASSERT_TRUE(alertIndicatorButton.get());
// Perform layout over all possible combinations, checking for correct
// results.
for (int isPinnedTab = 0; isPinnedTab < 2; ++isPinnedTab) {
for (int isActiveTab = 0; isActiveTab < 2; ++isActiveTab) {
- for (size_t mediaStateIndex = 0;
- mediaStateIndex < arraysize(kMediaStatesToTest);
- ++mediaStateIndex) {
- const TabMediaState mediaState = kMediaStatesToTest[mediaStateIndex];
+ for (size_t alertStateIndex = 0;
+ alertStateIndex < arraysize(kAlertStatesToTest);
+ ++alertStateIndex) {
+ const TabAlertState alertState = kAlertStatesToTest[alertStateIndex];
SCOPED_TRACE(::testing::Message()
<< (isActiveTab ? "Active" : "Inactive") << ' '
<< (isPinnedTab ? "Pinned " : "")
- << "Tab with media indicator state " << mediaState);
+ << "Tab with alert indicator state "
+ << static_cast<uint8_t>(alertState));
// Simulate what tab_strip_controller would do to set up the
// TabController state.
[controller setPinned:(isPinnedTab ? YES : NO)];
[controller setActive:(isActiveTab ? YES : NO)];
[controller setIconImage:favicon];
- [controller setMediaState:mediaState];
+ [controller setAlertState:alertState];
[controller updateVisibility];
// Test layout for every width from maximum to minimum.
« no previous file with comments | « chrome/browser/ui/cocoa/tabs/tab_controller.mm ('k') | chrome/browser/ui/cocoa/tabs/tab_strip_controller.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698