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

Unified Diff: chrome/browser/ui/cocoa/tabs/tab_window_controller.mm

Issue 421883006: Merge 286045 "Revert of mac: Fix tab dragging visual bug in Yose..." (Closed) Base URL: svn://svn.chromium.org/chrome/branches/2107/src/
Patch Set: Created 6 years, 5 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/tabs/tab_window_controller.mm
===================================================================
--- chrome/browser/ui/cocoa/tabs/tab_window_controller.mm (revision 286078)
+++ chrome/browser/ui/cocoa/tabs/tab_window_controller.mm (working copy)
@@ -47,10 +47,7 @@
@implementation TabWindowController
- (id)initTabWindowControllerWithTabStrip:(BOOL)hasTabStrip {
- const CGFloat kDefaultWidth = 750;
- const CGFloat kDefaultHeight = 600;
-
- NSRect contentRect = NSMakeRect(60, 229, kDefaultWidth, kDefaultHeight);
+ NSRect contentRect = NSMakeRect(60, 229, 750, 600);
base::scoped_nsobject<FramedBrowserWindow> window(
[[FramedBrowserWindow alloc] initWithContentRect:contentRect
hasTabStrip:hasTabStrip]);
@@ -60,20 +57,14 @@
if ((self = [super initWithWindow:window])) {
[[self window] setDelegate:self];
- chromeContentView_.reset([[NSView alloc]
- initWithFrame:NSMakeRect(0, 0, kDefaultWidth, kDefaultHeight)]);
- [chromeContentView_
- setAutoresizingMask:NSViewWidthSizable | NSViewHeightSizable];
- [[[self window] contentView] addSubview:chromeContentView_];
-
- tabContentArea_.reset(
- [[FastResizeView alloc] initWithFrame:[chromeContentView_ bounds]]);
+ tabContentArea_.reset([[FastResizeView alloc] initWithFrame:
+ NSMakeRect(0, 0, 750, 600)]);
[tabContentArea_ setAutoresizingMask:NSViewWidthSizable |
NSViewHeightSizable];
- [chromeContentView_ addSubview:tabContentArea_];
+ [[[self window] contentView] addSubview:tabContentArea_];
- tabStripView_.reset([[TabStripView alloc]
- initWithFrame:NSMakeRect(0, 0, kDefaultWidth, 37)]);
+ tabStripView_.reset([[TabStripView alloc] initWithFrame:
+ NSMakeRect(0, 0, 750, 37)]);
[tabStripView_ setAutoresizingMask:NSViewWidthSizable |
NSViewMinYMargin];
if (hasTabStrip)
@@ -90,10 +81,6 @@
return tabContentArea_;
}
-- (NSView*)chromeContentView {
- return chromeContentView_;
-}
-
// Add the top tab strop to the window, above the content box and add it to the
// view hierarchy as a sibling of the content view so it can overlap with the
// window frame.
@@ -140,7 +127,7 @@
[overlayWindow_ setOpaque:NO];
[overlayWindow_ setDelegate:self];
- originalContentView_ = self.chromeContentView;
+ originalContentView_ = [window contentView];
[window addChildWindow:overlayWindow_ ordered:NSWindowAbove];
// Explicitly set the responder to be nil here (for restoring later).
@@ -166,10 +153,7 @@
// places. The TabStripView always needs to be in front of the window's
// content view and therefore it should always be added after the content
// view is set.
- [[window contentView] addSubview:originalContentView_
- positioned:NSWindowBelow
- relativeTo:nil];
- originalContentView_.frame = [[window contentView] bounds];
+ [window setContentView:originalContentView_];
[[window cr_windowView] addSubview:[self tabStripView]];
[[window cr_windowView] updateTrackingAreas];
« no previous file with comments | « chrome/browser/ui/cocoa/tabs/tab_window_controller.h ('k') | chrome/browser/ui/cocoa/version_independent_window.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698