Implement dropping of tabs into an existing tab strip from another window.

chrome/browser/cocoa/browser_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.
 
 #include "base/mac_util.h"
 #include "base/sys_string_conversions.h"
 #include "chrome/app/chrome_dll_resource.h"  // IDC_*
 #include "chrome/browser/browser.h"
 #include "chrome/browser/browser_list.h"
 #include "chrome/browser/tab_contents/web_contents.h"
@@ skipping 257 matching lines @@
   [toolbarController_ setStarredState:isStarred];
 }
 
 // Return the rect, in WebKit coordinates (flipped), of the window's grow box
 // in the coordinate system of the content area of the currently selected tab.
 // |windowGrowBox| needs to be in the window's coordinate system.
 - (NSRect)selectedTabGrowBoxRect {
   return [tabStripController_ selectedTabGrowBoxRect];
 }
 
-- (void)dropTabView:(NSView *)view atIndex:(NSUInteger)index {
-  [tabStripController_ dropTabView:view atIndex:index];
+// Drop a given tab view at the location of the current placeholder. If there
+// is no placeholder, it will go at the end. |dragController| is the window
+// controller of a tab being dropped from a different window. It will be nil
+// if the drag is within the window. The implementation will call
+// |-removePlaceholder| since the drag is now complete. This also calls
+// |-layoutTabs| internally so clients do not need to call it again. When
+// dragging tabs between windows, this should be called *before*
+// |-detachTabView| on the source window since it needs to still be in the
+// source window's tab model for this method to find the information it needs
+// to complete the drop.
+- (void)dropTabView:(NSView*)view
+     fromController:(TabWindowController*)dragController {
+  if (dragController) {
+    // Moving between windows. Figure out the TabContents to drop into our tab
+    // model from the source window's model.
+    BrowserWindowController* dragBWC = (BrowserWindowController*)dragController;

[TVL, 2009/04/29 19:42:01]

nit: maybe do the cast after the class check, just so reading it in order,
you're proving it good before the cast?

+    BOOL isBrowser =
+        [dragBWC isKindOfClass:[BrowserWindowController class]];
+    DCHECK(isBrowser);
+    if (!isBrowser) return;
+    int index = [dragBWC->tabStripController_ indexForTabView:view];
+    TabContents* contents =
+        dragBWC->browser_->tabstrip_model()->GetTabContentsAt(index);
+
+    // Deposit it into our model at the appropriate location (it already knows
+    // where it should go from tracking the drag).
+    [tabStripController_ dropTabContents:contents];
+  } else {
+    // Moving within a window.
+    int index = [tabStripController_ indexForTabView:view];
+    [tabStripController_ moveTabFromIndex:index];
+  }
+
+  // Remove the placeholder since the drag is now complete.
+  [self removePlaceholder];
+}
+
+// Tells the tab strip to forget about this tab in preparation for it being
+// put into a different tab strip, such as during a drop on another window.
+- (void)detachTabView:(NSView*)view {
+  int index = [tabStripController_ indexForTabView:view];
+  browser_->tabstrip_model()->DetachTabContentsAt(index);
 }
 
 - (NSView *)selectedTabView {
   return [tabStripController_ selectedTabView];
 }
 
-- (TabStripController *)tabStripController {
-  return tabStripController_;
-}
-
 - (void)setIsLoading:(BOOL)isLoading {
   [toolbarController_ setIsLoading:isLoading];
 }
 
 // Called to start/stop the loading animations.
 - (void)updateLoadingAnimations:(BOOL)animate {
   if (animate) {
     // TODO(pinkerton): determine what throbber animation is necessary and
     // start a timer to periodically update. Windows tells the tab strip to
     // do this. It uses a single timer to coalesce the multiple things that
@@ skipping 199 matching lines @@
 
 - (NSRect)window:(NSWindow *)window
 willPositionSheet:(NSWindow *)sheet
        usingRect:(NSRect)defaultSheetRect {
   NSRect windowFrame = [window frame];
   defaultSheetRect.origin.y = windowFrame.size.height - 10;
   return defaultSheetRect;
 }
 
 @end