Ensure DesktopDragDropClientAuraX11::StartEndMoveLoopTimer() is called before

the move loop is ended.

BUG=376163
TEST=Manual, see bug

Review URL: https://codereview.chromium.org/347163002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278888 0039d316-1c4b-4281-b951-d872f2087c98

ui/views/widget/desktop_aura/desktop_drag_drop_client_aurax11.cc

@@ -687,13 +687,16 @@ void DesktopDragDropClientAuraX11::OnMouseReleased() {
     }
 
     if (negotiated_operation_ != ui::DragDropTypes::DRAG_NONE) {
+      // Start timer to end the move loop if the target takes too long to send
+      // an XdndFinished message. It is important that StartEndMoveLoopTimer()
+      // is called before SendXdndDrop() because SendXdndDrop()
+      // sends XdndFinished synchronously if the drop target is a Chrome
+      // window.
+      StartEndMoveLoopTimer();
+
       // We have negotiated an action with the other end.
       source_state_ = SOURCE_STATE_DROPPED;
       SendXdndDrop(source_current_window_);
-
-      // Start timer to end the move loop if the target takes too long to send
-      // an XdndFinished message.
-      StartEndMoveLoopTimer();
       return;
     }
   }