MacViews: Implement Tab Dragging

chrome/browser/ui/views/tabs/tab_drag_controller.cc

 // Copyright (c) 2012 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 "chrome/browser/ui/views/tabs/tab_drag_controller.h"
 
 #include <math.h>
 #include <set>
 
 #include "base/auto_reset.h"
@@ skipping 372 matching lines @@
                           point_in_screen.x() -
                           mouse_offset_.x(), 0);
         widget->SetVisibilityChangedAnimationsEnabled(false);
         widget->Restore();
         widget->SetBounds(new_bounds);
         AdjustBrowserAndTabBoundsForDrag(last_tabstrip_width,
                                          point_in_screen,
                                          &drag_bounds);
         widget->SetVisibilityChangedAnimationsEnabled(true);
       }
+
+      views::Widget* widget = GetAttachedBrowserWidget();

[sky, 2016/06/16 15:56:12]

As this part is really a separate bug, could you create a patch just for this
and add coverage in existing interactive ui tests?

Also, I think this should be in the else case of the if that starts on 371.

[themblsha, 2016/06/17 14:58:07]

There's already a bug for this:
https://bugs.chromium.org/p/chromium/issues/detail?id=518740#c2

Ok, reverted this code. DetachToBrowserTabDragControllerTest.DragAll /
DetachToBrowserTabDragControllerTest.MacDragsWindowUsingCocoaMoveLoop both
tested for this bug, guess I'll revert their changes as well.

+      gfx::Rect bounds = widget->GetWindowBoundsInScreen();
+      bounds.Offset(point_in_screen.x() - start_point_in_screen_.x(),
+                    point_in_screen.y() - start_point_in_screen_.y());
+      widget->SetBounds(bounds);
       RunMoveLoop(GetWindowOffset(point_in_screen));
       return;
     }
   }
 
   if (ContinueDragging(point_in_screen) == Liveness::DELETED)
     return;
 }
 
 void TabDragController::EndDrag(EndDragReason reason) {
@@ skipping 1404 matching lines @@
   // TODO(pkotwicz): Fix this properly (crbug.com/358482)
   for (auto* browser : *BrowserList::GetInstance()) {
     if (browser->tab_strip_model()->empty())
       exclude.insert(browser->window()->GetNativeWindow());
   }
 #endif
   base::WeakPtr<TabDragController> ref(weak_factory_.GetWeakPtr());
   *window = window_finder_->GetLocalProcessWindowAtPoint(screen_point, exclude);
   return ref ? Liveness::ALIVE : Liveness::DELETED;
 }