Ensure DesktopDragDropClientAuraX11::StartEndMoveLoopTimer() is called before the move loop is ended

ui/views/widget/desktop_aura/desktop_drag_drop_client_aurax11.cc

Index: ui/views/widget/desktop_aura/desktop_drag_drop_client_aurax11.cc
diff --git a/ui/views/widget/desktop_aura/desktop_drag_drop_client_aurax11.cc b/ui/views/widget/desktop_aura/desktop_drag_drop_client_aurax11.cc
index d3e52f787192b6ec1a11eebd26d7e8843e204637..d8a76e665593cb14d781487c88fc60d6a8d160d4 100644
--- a/ui/views/widget/desktop_aura/desktop_drag_drop_client_aurax11.cc
+++ b/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;
     }
   }