GitLab

To process Left/Right snapping properly, CompleteDrag() in
TabDragController needs some properties, thus r596783 moves
ClearTabDraggingInfo() after CompleteDrag(). However
https://crbug.com/892221 revealed that another propertly
kIsDraggingTabsKey needs to be cleared beforehand, otherwise
some bounds calculation goes wrong.

This CL moves ClearTabDraggingInfo() as my previous CL does,
but clears kIsDraggingTabsKey earlier so issue 892221 won't
be affected.

BUG=880635, 892221
TEST=manually

Change-Id: Ib7edaab5e968e006bc919b027c138ad9f3d5827d
Reviewed-on: https://chromium-review.googlesource.com/c/1266495Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Jun Mukai <mukai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597616}