Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3094)

Unified Diff: chrome/browser/ui/cocoa/browser_window_controller.mm

Issue 2742813003: [Mac] Lay out the browser window when adding the download shelf. (Closed)
Patch Set: Test name and whitespace tweaks. Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/cocoa/browser_window_controller.mm
diff --git a/chrome/browser/ui/cocoa/browser_window_controller.mm b/chrome/browser/ui/cocoa/browser_window_controller.mm
index 57efa23c3a539b33a5b809834e36bddee98065c2..34182c124cce4ee1f2eb4b6836a76093a4e37913 100644
--- a/chrome/browser/ui/cocoa/browser_window_controller.mm
+++ b/chrome/browser/ui/cocoa/browser_window_controller.mm
@@ -244,11 +244,8 @@ bool IsTabDetachingInFullscreenEnabled() {
[[window contentView] setWantsLayer:YES];
windowShim_.reset(new BrowserWindowCocoa(browser, self));
- // Set different minimum sizes on tabbed windows vs non-tabbed, e.g. popups.
// This has to happen before -enforceMinWindowSize: is called further down.
- NSSize minSize = [self isTabbedWindow] ?
- NSMakeSize(400, 272) : NSMakeSize(100, 122);
- [[self window] setMinSize:minSize];
+ [[self window] setMinSize:MinWindowSizeForBrowserType(browser->type())];
// Lion will attempt to automagically save and restore the UI. This
// functionality appears to be leaky (or at least interacts badly with our
@@ -1367,6 +1364,7 @@ bool IsTabDetachingInFullscreenEnabled() {
downloadShelfController_.reset([[DownloadShelfController alloc]
initWithBrowser:browser_.get() resizeDelegate:self]);
[self.chromeContentView addSubview:[downloadShelfController_ view]];
+ [self layoutSubviews];
}
}

Powered by Google App Engine
This is Rietveld 408576698