Fix jittering cursor in screen magnifier mode

Fix jittering cursor in screen magnifier mode

The cursor is moved back to original position after animation of zoom in
and zoom out in screen magnifier mode. Moving cursor across the displays
will cause the cursor to move around the border of the displays.

This CL fixes this issue by leaving the cursor unchanged after animation
caused by root window switch.

BUG=709683
TEST=ash_unittests --gtest_filter=MagnificationControllerTest.MoveMouseToSecondDisplay

Review-Url: https://codereview.chromium.org/2886843004
Cr-Commit-Position: refs/heads/master@{#472664}
Committed: https://chromium.googlesource.com/chromium/src/+/2e1f4b7afbcd7917c8a577e8a47ea02e920590c2

[weidongg, 2017-05-17 04:13:07]

The CQ bit was checked by weidongg@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-17 04:14:10]

Dry run: CQ is trying da patch.

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

[weidongg, 2017-05-17 04:40:28]

weidongg@chromium.org changed reviewers:
+ derat@chromium.org

[commit-bot: I haz the power, 2017-05-17 04:51:27]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-17 04:51:28]

Dry run: This issue passed the CQ dry run.

[Daniel Erat, 2017-05-17 13:26:59]

is it possible to add a unit test for this?

i'll be out for the rest of the day, by the way.

[weidongg, 2017-05-17 18:06:52]

On 2017/05/17 13:26:59, Daniel Erat wrote:
> is it possible to add a unit test for this?
> 
> i'll be out for the rest of the day, by the way.

Yes, I just added the ash unit test. The test passes with the CL and crashes
without the CL.
That's ok, I am in no hurry. Thanks.

[weidongg, 2017-05-17 22:01:28]

Description was changed from

==========
Fix jittering cursor in screen magnifier mode

The cursor is moved back to original position after animation of zoom in
and zoom out in screen magnifier mode. Moving cursor across the displays
will cause the cursor to move around the border of the displays.

This CL fixes this issue by leaving the cursor unchanged after animation
caused by root window switch.

BUG=709683
TEST=NONE
==========

to

==========
Fix jittering cursor in screen magnifier mode

The cursor is moved back to original position after animation of zoom in
and zoom out in screen magnifier mode. Moving cursor across the displays
will cause the cursor to move around the border of the displays.

This CL fixes this issue by leaving the cursor unchanged after animation
caused by root window switch.

BUG=709683
TEST=ash_unittests
--gtest_filter=MagnificationControllerTest.MoveMouseToSecondDisplay
==========

[Daniel Erat, 2017-05-18 03:15:44]

lgtm

[weidongg, 2017-05-18 03:52:01]

The CQ bit was checked by weidongg@chromium.org

[weidongg, 2017-05-18 03:52:19]

On 2017/05/18 03:15:44, Daniel Erat wrote:
> lgtm

Thanks for code review!

[commit-bot: I haz the power, 2017-05-18 03:53:04]

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, 2017-05-18 04:31:00]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1495079521003400,
"parent_rev": "2a81129250714241687f3a8f51fb0915aa4f1a6f", "commit_rev":
"2e1f4b7afbcd7917c8a577e8a47ea02e920590c2"}

[commit-bot: I haz the power, 2017-05-18 04:31:08]

Description was changed from

==========
Fix jittering cursor in screen magnifier mode

The cursor is moved back to original position after animation of zoom in
and zoom out in screen magnifier mode. Moving cursor across the displays
will cause the cursor to move around the border of the displays.

This CL fixes this issue by leaving the cursor unchanged after animation
caused by root window switch.

BUG=709683
TEST=ash_unittests
--gtest_filter=MagnificationControllerTest.MoveMouseToSecondDisplay
==========

to

==========
Fix jittering cursor in screen magnifier mode

The cursor is moved back to original position after animation of zoom in
and zoom out in screen magnifier mode. Moving cursor across the displays
will cause the cursor to move around the border of the displays.

This CL fixes this issue by leaving the cursor unchanged after animation
caused by root window switch.

BUG=709683
TEST=ash_unittests
--gtest_filter=MagnificationControllerTest.MoveMouseToSecondDisplay

Review-Url: https://codereview.chromium.org/2886843004
Cr-Commit-Position: refs/heads/master@{#472664}
Committed:
https://chromium.googlesource.com/chromium/src/+/2e1f4b7afbcd7917c8a577e8a47e...
==========

[commit-bot: I haz the power, 2017-05-18 04:31:10]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/2e1f4b7afbcd7917c8a577e8a47e...