Enforce different min size for popups than tabbed browsers on MacOSX.

chrome/browser/ui/cocoa/browser_window_controller_unittest.mm

 // Copyright (c) 2011 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.
 
 #include "base/memory/scoped_nsobject.h"
 #include "base/memory/scoped_ptr.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/sync/sync_ui_util.h"
 #include "chrome/browser/ui/browser_window.h"
@@ skipping 114 matching lines @@
   BrowserWindowController* controller =
       static_cast<BrowserWindowController*>([cocoaWindow windowController]);
   ASSERT_TRUE([controller isKindOfClass:[BrowserWindowController class]]);
   EXPECT_FALSE([controller isTabbedWindow]);
   EXPECT_FALSE([controller hasTabStrip]);
   EXPECT_TRUE([controller hasTitleBar]);
   EXPECT_FALSE([controller isBookmarkBarVisible]);
   [controller close];
 }
 
+TEST_F(BrowserWindowControllerTest, TestSetBounds) {
+  // Create a normal browser with bounds smaller than the minimum.
+  Browser::CreateParams params(Browser::TYPE_TABBED, browser_helper_.profile());
+  params.initial_bounds = gfx::Rect(0, 0, 50, 50);
+  Browser* browser = Browser::CreateWithParams(params);
+  NSWindow *cocoaWindow = browser->window()->GetNativeHandle();
+  BrowserWindowController* controller =
+    static_cast<BrowserWindowController*>([cocoaWindow windowController]);
+
+  ASSERT_TRUE([controller isTabbedWindow]);
+  BrowserWindow* browserWindow = [controller browserWindow];
+  EXPECT_EQ(browserWindow, browser->window());
+  gfx::Rect bounds = browserWindow->GetBounds();
+  EXPECT_EQ(400, bounds.width());
+  EXPECT_EQ(272, bounds.height());
+
+  // Try to set the bounds smaller than the minimum.
+  browserWindow->SetBounds(gfx::Rect(0, 0, 50, 50));
+  bounds = browserWindow->GetBounds();
+  EXPECT_EQ(400, bounds.width());
+  EXPECT_EQ(272, bounds.height());
+
+  [controller close];
+}
+
+TEST_F(BrowserWindowControllerTest, TestSetBoundsPopup) {
+  // Create a popup with bounds smaller than the minimum.
+  Browser::CreateParams params(Browser::TYPE_POPUP, browser_helper_.profile());
+  params.initial_bounds = gfx::Rect(0, 0, 50, 50);
+  Browser* browser = Browser::CreateWithParams(params);
+  NSWindow *cocoaWindow = browser->window()->GetNativeHandle();
+  BrowserWindowController* controller =
+    static_cast<BrowserWindowController*>([cocoaWindow windowController]);
+
+  ASSERT_FALSE([controller isTabbedWindow]);
+  BrowserWindow* browserWindow = [controller browserWindow];
+  EXPECT_EQ(browserWindow, browser->window());
+  gfx::Rect bounds = browserWindow->GetBounds();
+  EXPECT_EQ(100, bounds.width());
+  EXPECT_EQ(122, bounds.height());
+
+  // Try to set the bounds smaller than the minimum.
+  browserWindow->SetBounds(gfx::Rect(0, 0, 50, 50));
+  bounds = browserWindow->GetBounds();
+  EXPECT_EQ(100, bounds.width());
+  EXPECT_EQ(122, bounds.height());
+
+  [controller close];
+}
+
 TEST_F(BrowserWindowControllerTest, TestTheme) {
   [controller_ userChangedTheme];
 }
 
 TEST_F(BrowserWindowControllerTest, BookmarkBarControllerIndirection) {
   EXPECT_FALSE([controller_ isBookmarkBarVisible]);
 
   // Explicitly show the bar. Can't use bookmark_utils::ToggleWhenVisible()
   // because of the notification issues.
   browser_helper_.profile()->GetPrefs()->
@@ skipping 524 matching lines @@
   fullscreenWindow_.reset(
       [[NSWindow alloc] initWithContentRect:NSMakeRect(0,0,400,400)
                                   styleMask:NSBorderlessWindowMask
                                     backing:NSBackingStoreBuffered
                                       defer:NO]);
   return fullscreenWindow_.get();
 }
 @end
 
 /* TODO(???): test other methods of BrowserWindowController */