Removes grab input window and extra grab and ungrab in X11WholeScreenMoveLoop

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

Index: chrome/browser/ui/views/tabs/tab_drag_controller.cc
diff --git a/chrome/browser/ui/views/tabs/tab_drag_controller.cc b/chrome/browser/ui/views/tabs/tab_drag_controller.cc
index 54f836d1eb6b0a2a5b1774deb2f9bb9baf0b500b..47a3dda1e949af433e40507a426aa945b9c5aaec 100644
--- a/chrome/browser/ui/views/tabs/tab_drag_controller.cc
+++ b/chrome/browser/ui/views/tabs/tab_drag_controller.cc
@@ -1177,6 +1177,8 @@ void TabDragController::DetachIntoNewBrowserAndRunMoveLoop(
   WindowPositionManagedUpdater updater;
   dragged_widget->AddObserver(&updater);
   browser->window()->Show();
+  // Set capture after the newly created browser widget is visible.
+  attached_tabstrip_->GetWidget()->SetCapture(attached_tabstrip_);

[varkha, 2014/05/21 20:07:20]

This makes sure that capture is really where we think it is after calling
Attach(). Attach() would normally set capture on attached widget but with Aura
fails to do so because of IsVisible() check in Window::SetCapture(). This was
wrong before but worked because of additional capture in X11WholeScreenMoveLoop.
Simplifying X11WholeScreenMoveLoop exposed this.

   dragged_widget->RemoveObserver(&updater);
   dragged_widget->SetVisibilityChangedAnimationsEnabled(true);
   // Activate may trigger a focus loss, destroying us.