Fix crash on fullscreening of popup.

chrome/browser/cocoa/toolbar_controller_unittest.mm

 // Copyright (c) 2009 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>
 
 #import "base/scoped_nsobject.h"
 #include "chrome/app/chrome_dll_resource.h"
 #include "chrome/browser/cocoa/browser_test_helper.h"
 #import "chrome/browser/cocoa/cocoa_test_helper.h"
@@ skipping 66 matching lines @@
 
 // Make sure a "titlebar only" toolbar works
 TEST_F(ToolbarControllerTest, TitlebarOnly) {
   NSView* view = [bar_ view];
   EXPECT_TRUE([bar_ bookmarkBarController]);
 
   [bar_ setHasToolbar:NO];
   EXPECT_NE(view, [bar_ view]);
   EXPECT_FALSE([bar_ bookmarkBarController]);
 
+  // Simulate a popup going fullscreen and back.
+  NSView* superview = [view superview];
+  // TODO(jrg): find a way to add an [NSAutoreleasePool drain] in
+  // here.  I don't have access to the current
+  // scoped_nsautorelease_pool to do it properly :-(
+  [view removeFromSuperview];
+  [superview addSubview:view];
+
   [bar_ setHasToolbar:YES];
   EXPECT_EQ(view, [bar_ view]);
   EXPECT_TRUE([bar_ bookmarkBarController]);
 
   // Leave it off to make sure that's fine
   [bar_ setHasToolbar:NO];
 }
 
 
 // Make some changes to the enabled state of a few of the buttons and ensure
@@ skipping 81 matching lines @@
   // Toggle the pref and make sure the buttons changed state and the other
   // views moved (or in the case of the location bar, it changed width).
   prefs->SetBoolean(prefs::kShowPageOptionsButtons, !showButtons);
   EXPECT_EQ(showButtons, [pageButton isHidden]);
   EXPECT_EQ(showButtons, [wrenchButton isHidden]);
   EXPECT_NE(NSMinX(originalGoFrame), NSMinX([goButton frame]));
   EXPECT_NE(NSWidth(originalLocationBarFrame), NSWidth([locationBar frame]));
 }
 
 }  // namespace