Index: chrome/browser/ui/cocoa/toolbar/toolbar_controller_unittest.mm |
diff --git a/chrome/browser/ui/cocoa/toolbar/toolbar_controller_unittest.mm b/chrome/browser/ui/cocoa/toolbar/toolbar_controller_unittest.mm |
index 0edcb6a60832735866136104fdd05ff429e1a976..1eeffaed319c00f982ce2e01dc2f6f1a1b578789 100644 |
--- a/chrome/browser/ui/cocoa/toolbar/toolbar_controller_unittest.mm |
+++ b/chrome/browser/ui/cocoa/toolbar/toolbar_controller_unittest.mm |
@@ -17,7 +17,6 @@ |
#include "chrome/browser/ui/cocoa/cocoa_profile_test.h" |
#import "chrome/browser/ui/cocoa/image_button_cell.h" |
#import "chrome/browser/ui/cocoa/toolbar/toolbar_controller.h" |
-#import "chrome/browser/ui/cocoa/view_resizer_pong.h" |
#include "chrome/common/pref_names.h" |
#include "chrome/test/base/testing_profile.h" |
#include "testing/gtest/include/gtest/gtest.h" |
@@ -85,12 +84,10 @@ class ToolbarControllerTest : public CocoaProfileTest { |
// ensure they get picked up correct on initialization |
updater->UpdateCommandEnabled(IDC_BACK, false); |
updater->UpdateCommandEnabled(IDC_FORWARD, false); |
- resizeDelegate_.reset([[ViewResizerPong alloc] init]); |
bar_.reset([[TestToolbarController alloc] |
initWithCommands:browser()->command_controller()->command_updater() |
profile:profile() |
- browser:browser() |
- resizeDelegate:resizeDelegate_.get()]); |
+ browser:browser()]); |
EXPECT_TRUE([bar_ view]); |
NSView* parent = [test_window() contentView]; |
[parent addSubview:[bar_ view]]; |
@@ -114,7 +111,6 @@ class ToolbarControllerTest : public CocoaProfileTest { |
[[views objectAtIndex:kHomeIndex] isEnabled] ? true : false); |
} |
- base::scoped_nsobject<ViewResizerPong> resizeDelegate_; |
base::scoped_nsobject<TestToolbarController> bar_; |
}; |
@@ -298,9 +294,11 @@ class BrowserRemovedObserver : public chrome::BrowserListObserver { |
// This can happen because the ToolbarController is retained by both the |
// BrowserWindowController and -[ToolbarController view], the latter of which is |
// autoreleased. |
-// TODO(nhiroki): This is disabled due to http://crbug.com/506745. |
-TEST_F(ToolbarControllerTest, DISABLED_ToolbarDestroyedAfterBrowser) { |
+TEST_F(ToolbarControllerTest, ToolbarDestroyedAfterBrowser) { |
BrowserRemovedObserver observer; |
+ // This is normally called by BrowserWindowController, but since |bar_| is not |
+ // owned by one, call it here. |
+ [bar_ browserWillBeDestroyed]; |
CloseBrowserWindow(); |
observer.WaitUntilBrowserRemoved(); |
// |bar_| is released in TearDown(). |