Reland of Reenable Tabdragging tests failing because of IsWindowPositionManaged()

Reland of Reenable Tabdragging tests failing because of IsWindowPositionManaged()

It was reverted previously as it caused a crash in
WorkspaceEventHandlerTest.DeleteWhileInRunLoop.

It used to be exiting the move loop and clearing the move_loop_widget_ before
EndDrag() had a chance to set the window position to manageable again.

This CL adds a maximized browser window waiter, and moves setting the window
position managed logic to ash.

TBR=pkasting@chromium.org,sky@chromium.org,skuhne@chromium.org,oshima@chromium.org,erg@chromium.org
BUG=626761, 331924
TEST=interactive_ui_tests --gtest_filter=TabDragging*

Committed: https://crrev.com/83e8abcce5801546b4bb2ff03fdaf5cb4590f0e5
Cr-Commit-Position: refs/heads/master@{#435775}

[afakhry, 2016-12-01 23:04:38]

Patchset #1 (id:1) has been deleted

[afakhry, 2016-12-01 23:09:23]

The CQ bit was checked by afakhry@chromium.org

[commit-bot: I haz the power, 2016-12-01 23:09:43]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-01 23:54:55]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1480633763233760,
"parent_rev": "5f451d7cc6d3bd803e3344c8788c0113fd700477", "commit_rev":
"c1e841c3022217ced88f4cd1d6fae3ab5664d41a"}

[commit-bot: I haz the power, 2016-12-01 23:55:22]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-12-01 23:58:05]

Description was changed from

==========
Reland of Reenable Tabdragging tests failing because of
IsWindowPositionManaged()

It was reverted previously as it caused a crash in
WorkspaceEventHandlerTest.DeleteWhileInRunLoop.

It used to be exiting the move loop and clearing the move_loop_widget_ before
EndDrag() had a chance to set the window position to manageable again.

This CL adds a maximized browser window waiter, and moves setting the window
position managed logic to ash.

TBR=pkasting@chromium.org,sky@chromium.org,skuhne@chromium.org,oshima@chromiu...
BUG=626761, 331924
TEST=interactive_ui_tests --gtest_filter=TabDragging*
==========

to

==========
Reland of Reenable Tabdragging tests failing because of
IsWindowPositionManaged()

It was reverted previously as it caused a crash in
WorkspaceEventHandlerTest.DeleteWhileInRunLoop.

It used to be exiting the move loop and clearing the move_loop_widget_ before
EndDrag() had a chance to set the window position to manageable again.

This CL adds a maximized browser window waiter, and moves setting the window
position managed logic to ash.

TBR=pkasting@chromium.org,sky@chromium.org,skuhne@chromium.org,oshima@chromiu...
BUG=626761, 331924
TEST=interactive_ui_tests --gtest_filter=TabDragging*

Committed: https://crrev.com/83e8abcce5801546b4bb2ff03fdaf5cb4590f0e5
Cr-Commit-Position: refs/heads/master@{#435775}
==========

[commit-bot: I haz the power, 2016-12-01 23:58:06]

Patchset 2 (id:??) landed as
https://crrev.com/83e8abcce5801546b4bb2ff03fdaf5cb4590f0e5
Cr-Commit-Position: refs/heads/master@{#435775}