[mac] Implement dragging of multiple tabs.

chrome/browser/ui/cocoa/tabs/tab_strip_controller.h

Index: chrome/browser/ui/cocoa/tabs/tab_strip_controller.h
diff --git a/chrome/browser/ui/cocoa/tabs/tab_strip_controller.h b/chrome/browser/ui/cocoa/tabs/tab_strip_controller.h
index 86e01bd25c62ac7b870870210a975a8b4a59595c..8591d0637c9382a42bc88da450212a49286b997d 100644
--- a/chrome/browser/ui/cocoa/tabs/tab_strip_controller.h
+++ b/chrome/browser/ui/cocoa/tabs/tab_strip_controller.h
@@ -159,18 +159,20 @@ class WebContents;
 // Returns the model behind this controller.
 - (TabStripModel*)tabStripModel;
 
+// Returns all tab views.
+- (NSArray*)tabViews;
+
 // Return the view for the currently active tab.
 - (NSView*)activeTabView;
 
-// Set the frame of the active tab, also updates the internal frame dict.
-- (void)setFrameOfActiveTab:(NSRect)frame;
+// Set the frame of |tabView|, also updates the internal frame dict.
+- (void)setFrame:(NSRect)frame ofTabView:(NSView*)tabView;
 
 // Move the given tab at index |from| in this window to the location of the
 // current placeholder.
 - (void)moveTabFromIndex:(NSInteger)from;
 
-// Drop a given WebContents at the location of the current placeholder.
-// If there is no placeholder, it will go at the end. Used when dragging from
+// Drop a given WebContents at |modelIndex|. Used when dragging from
 // another window when we don't have access to the WebContents as part of our
 // strip. |frame| is in the coordinate system of the tab strip view and
 // represents where the user dropped the new tab so it can be animated into its
@@ -179,8 +181,10 @@ class WebContents;
 // new window. Mini-tabs are either app or pinned tabs; the app state is stored
 // by the |contents|, but the |pinned| state is the caller's responsibility.

[Robert Sesek, 2014/02/24 15:25:20]

Should also discuss what |activate| does.

[Andre, 2014/02/25 01:16:16]

Done.

 - (void)dropWebContents:(content::WebContents*)contents
+                atIndex:(int)modelIndex
               withFrame:(NSRect)frame
-            asPinnedTab:(BOOL)pinned;
+            asPinnedTab:(BOOL)pinned
+               activate:(BOOL)activate;
 
 // Returns the index of the subview |view|. Returns -1 if not present. Takes
 // closing tabs into account such that this index will correctly match the tab
@@ -188,6 +192,8 @@ class WebContents;
 // are no longer in the model.
 - (NSInteger)modelIndexForTabView:(NSView*)view;
 
+- (NSArray*)selectedViews;

[Robert Sesek, 2014/02/24 15:25:20]

Comment all public declarations.

[Andre, 2014/02/25 01:16:16]

Done.

+
 // Return the view at a given index.
 - (NSView*)viewAtIndex:(NSUInteger)index;
 
@@ -200,6 +206,11 @@ class WebContents;
 // This causes this tab to be rendered in an arbitrary position.
 - (void)insertPlaceholderForTab:(TabView*)tab frame:(NSRect)frame;
 
+// Find the model index based on the x coordinate of the placeholder. If there
+// is no placeholder, this returns the end of the tab strip. Closing tabs are
+// not considered in computing the index.
+- (int)indexOfPlaceholder;
+
 // Returns whether a tab is being dragged within the tab strip.
 - (BOOL)isDragSessionActive;