mac, yosemite: Resize button causes fullscreen effect.

chrome/browser/ui/cocoa/tabs/tab_window_controller.mm

Index: chrome/browser/ui/cocoa/tabs/tab_window_controller.mm
diff --git a/chrome/browser/ui/cocoa/tabs/tab_window_controller.mm b/chrome/browser/ui/cocoa/tabs/tab_window_controller.mm
index c6dfa624f0ef37063166cb708b2cc7b1306d5482..a678e4e1ef48bf8199cf658a4aad0b368af0ea4b 100644
--- a/chrome/browser/ui/cocoa/tabs/tab_window_controller.mm
+++ b/chrome/browser/ui/cocoa/tabs/tab_window_controller.mm
@@ -88,7 +88,14 @@
   // The frame doesn't matter. This class relies on subclasses to do tab strip
   // layout.
   NSView* contentParent = [[self window] cr_windowView];
-  [contentParent addSubview:tabStripView_];
+  if (contentParent == [[[self window] contentView] superview]) {
+    // Add the tab strip directly above the content view, if they are siblings.
+    [contentParent addSubview:tabStripView_
+                   positioned:NSWindowAbove
+                   relativeTo:[[self window] contentView]];

[Andre, 2014/08/30 01:38:02]

We also mess with tabStripView in -[BrowserWindowController
moveViewsForImmersiveFullscreen...].
Do we need to do something similar there?

[erikchen, 2014/09/02 17:35:57]

yup. also have to do something similar for making a new window from a dragged
tab.

+  } else {
+    [contentParent addSubview:tabStripView_];
+  }
 }
 
 - (void)removeOverlay {