Pop-up chrome....

chrome/browser/cocoa/tab_window_controller.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 "chrome/browser/cocoa/tab_window_controller.h"
 
 #include "base/logging.h"
 #import "chrome/browser/cocoa/tab_strip_view.h"
 
 @interface TabWindowController(PRIVATE)
 - (void)setUseOverlay:(BOOL)useOverlay;
 @end
 
 @implementation TabWindowController
 @synthesize tabStripView = tabStripView_;
 @synthesize tabContentArea = tabContentArea_;
 
 - (void)windowDidLoad {
-  // Place the tab bar 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.
-  NSRect tabFrame = [tabContentArea_ frame];
-  tabFrame.origin = NSMakePoint(0, NSMaxY(tabFrame));
-  tabFrame.size.height = NSHeight([tabStripView_ frame]);
-  [tabStripView_ setFrame:tabFrame];
+  // TODO(jrg): a non-normal window (e.g. for pop-ups) needs more work
+  // than just removal of the tab strip offset.  But this is enough to
+  // avoid confusion (e.g. "new tab" on popup gets created in a
+  // different window).

[pink (ping after 24hrs), 2009/07/01 12:53:00]

does the content area need adjusting up? I think it's positioned lower than
"normal" in the nib assuming a tab strip will go above it. Does it look too low
in the window w/out it?

[John Grabowski, 2009/07/02 20:28:25]

Yes, it is too low.  The right answer is a new nib for pop-ups.  However, in the
immediate term, I want to prevent the most brutal of problems when trying to use
Mac Chrome with Presently.  I'll save the new nib for our UI polish phase, and
will leave the bug open.

+  if ([self isNormalWindow]) {
+    // Place the tab bar 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.
+    NSRect tabFrame = [tabContentArea_ frame];
+    tabFrame.origin = NSMakePoint(0, NSMaxY(tabFrame));
+    tabFrame.size.height = NSHeight([tabStripView_ frame]);
+    [tabStripView_ setFrame:tabFrame];
+  }
   [[[[self window] contentView] superview] addSubview:tabStripView_];
 }
 
 - (void)removeOverlay {
   [self setUseOverlay:NO];
 }
 
 - (void)removeOverlayAfterDelay:(NSTimeInterval)delay {
   [NSObject cancelPreviousPerformRequestsWithTarget:self
                                            selector:@selector(removeOverlay)
@@ skipping 114 matching lines @@
   NOTIMPLEMENTED();
   return 0;
 }
 
 - (NSString*)selectedTabTitle {
   // subclass must implement
   NOTIMPLEMENTED();
   return @"";
 }
 
+- (BOOL)isNormalWindow {
+  // subclass must implement
+  NOTIMPLEMENTED();
+  return YES;
+}
+
 @end