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. |