Commit 2c09075b authored by varkha@chromium.org's avatar varkha@chromium.org

Prevent reentrant CompleteDrag when changing display configuration

BUG=315395
TEST=interactive_ui_tests --gtest_filter=*CancelDragTabToWindowIn*Display

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@233541 0039d316-1c4b-4281-b951-d872f2087c98
parent 28343f41
...@@ -376,10 +376,7 @@ void ToplevelWindowEventHandler::EndMoveLoop() { ...@@ -376,10 +376,7 @@ void ToplevelWindowEventHandler::EndMoveLoop() {
return; return;
in_move_loop_ = false; in_move_loop_ = false;
if (window_resizer_) { CompleteDrag(DRAG_REVERT, 0);
window_resizer_->resizer()->RevertDrag();
window_resizer_.reset();
}
quit_closure_.Run(); quit_closure_.Run();
} }
...@@ -387,9 +384,8 @@ void ToplevelWindowEventHandler::OnDisplayConfigurationChanging() { ...@@ -387,9 +384,8 @@ void ToplevelWindowEventHandler::OnDisplayConfigurationChanging() {
if (in_move_loop_) { if (in_move_loop_) {
move_cancelled_ = true; move_cancelled_ = true;
EndMoveLoop(); EndMoveLoop();
} else if (window_resizer_) { } else {
window_resizer_->resizer()->RevertDrag(); CompleteDrag(DRAG_REVERT, 0);
window_resizer_.reset();
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment