diff options
author | rohitrao@chromium.org <rohitrao@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-03 23:55:55 +0000 |
---|---|---|
committer | rohitrao@chromium.org <rohitrao@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-03 23:55:55 +0000 |
commit | fc701bfe7df9de83bfbf5c93f23b455a47ba6bd8 (patch) | |
tree | 74a4fba00d45604b03d26378db27259a7b8a6ebc /chrome/browser/cocoa/infobar_container_controller_unittest.mm | |
parent | 5e36967c2b96745b13f3bc8d9ca56f62d888ae1d (diff) | |
download | chromium_src-fc701bfe7df9de83bfbf5c93f23b455a47ba6bd8.zip chromium_src-fc701bfe7df9de83bfbf5c93f23b455a47ba6bd8.tar.gz chromium_src-fc701bfe7df9de83bfbf5c93f23b455a47ba6bd8.tar.bz2 |
[Mac] Enables animations for the infobar. Changes the control flow for infobar opening/closing to match Windows more closely.
Nib file changes:
- Embedded the InfoBarGradientView inside an AnimatableView.
- Rebound [controller view] to the AnimatableView and added an infoBarView_ IBOutlet.
- Bound the AnimatableView's delegate_ to the InfoBarController.
BUG=http://crbug.com/25599
TEST=Infobars should animate in and out, except for during tab switches.
Review URL: http://codereview.chromium.org/354008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@30893 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/cocoa/infobar_container_controller_unittest.mm')
-rw-r--r-- | chrome/browser/cocoa/infobar_container_controller_unittest.mm | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/chrome/browser/cocoa/infobar_container_controller_unittest.mm b/chrome/browser/cocoa/infobar_container_controller_unittest.mm index 274ebec..f5a45b2 100644 --- a/chrome/browser/cocoa/infobar_container_controller_unittest.mm +++ b/chrome/browser/cocoa/infobar_container_controller_unittest.mm @@ -63,23 +63,23 @@ TEST_F(InfoBarContainerControllerTest, AddAndRemoveInfoBars) { MockLinkInfoBarDelegate linkDelegate; MockConfirmInfoBarDelegate confirmDelegate; - [controller_ addInfoBar:&alertDelegate]; + [controller_ addInfoBar:&alertDelegate animate:NO]; EXPECT_EQ(1U, [[view subviews] count]); - [controller_ addInfoBar:&linkDelegate]; + [controller_ addInfoBar:&linkDelegate animate:NO]; EXPECT_EQ(2U, [[view subviews] count]); - [controller_ addInfoBar:&confirmDelegate]; + [controller_ addInfoBar:&confirmDelegate animate:NO]; EXPECT_EQ(3U, [[view subviews] count]); // Just to mix things up, remove them in a different order. - [controller_ removeInfoBarsForDelegate:&linkDelegate]; + [controller_ closeInfoBarsForDelegate:&linkDelegate animate:NO]; EXPECT_EQ(2U, [[view subviews] count]); - [controller_ removeInfoBarsForDelegate:&confirmDelegate]; + [controller_ closeInfoBarsForDelegate:&confirmDelegate animate:NO]; EXPECT_EQ(1U, [[view subviews] count]); - [controller_ removeInfoBarsForDelegate:&alertDelegate]; + [controller_ closeInfoBarsForDelegate:&alertDelegate animate:NO]; EXPECT_EQ(0U, [[view subviews] count]); } @@ -92,9 +92,9 @@ TEST_F(InfoBarContainerControllerTest, RemoveAllInfoBars) { MockLinkInfoBarDelegate linkDelegate; MockConfirmInfoBarDelegate confirmDelegate; - [controller_ addInfoBar:&alertDelegate]; - [controller_ addInfoBar:&linkDelegate]; - [controller_ addInfoBar:&confirmDelegate]; + [controller_ addInfoBar:&alertDelegate animate:NO]; + [controller_ addInfoBar:&linkDelegate animate:NO]; + [controller_ addInfoBar:&confirmDelegate animate:NO]; EXPECT_EQ(3U, [[view subviews] count]); [controller_ removeAllInfoBars]; |