In mushrome, keep the state of the cursor locally.

In mushrome, keep the state of the cursor locally.

In addition to telling the window server about state changes, record them
locally in the window manager so that queries about the current cursor state
return the correct values.

BUG=734806
Review-Url: https://codereview.chromium.org/2955623003
Cr-Commit-Position: refs/heads/master@{#482102}
Committed: https://chromium.googlesource.com/chromium/src/+/bf104c1b3d2dfae83774369a9c6a890ad57b9d6e

[Elliot Glaysher, 2017-06-23 22:47:54]

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

[commit-bot: I haz the power, 2017-06-23 22:48:20]

Dry run: CQ is trying da patch.

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

[Elliot Glaysher, 2017-06-23 22:59:39]

erg@chromium.org changed reviewers:
+ sky@chromium.org

[Elliot Glaysher, 2017-06-23 22:59:39]

Even though this doesn't do anything about displaying the cursor, we still need
to maintain this state window manager side.

[sky, 2017-06-23 23:31:28]

LGTM

https://codereview.chromium.org/2955623003/diff/1/ash/wm/native_cursor_manage...
File ash/wm/native_cursor_manager_ash_mus.cc (right):

https://codereview.chromium.org/2955623003/diff/1/ash/wm/native_cursor_manage...
ash/wm/native_cursor_manager_ash_mus.cc:50: for (aura::Window::Windows::iterator
iter = root_windows.begin();
for (aura::Window* root : Shell::Get()->GetAllRootWindows())

[Elliot Glaysher, 2017-06-23 23:34:06]

The CQ bit was unchecked by erg@chromium.org

[Elliot Glaysher, 2017-06-23 23:35:20]

The CQ bit was checked by erg@chromium.org

[Elliot Glaysher, 2017-06-23 23:35:20]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/2955623003/#ps20001
(title: "c++11 loop")

[commit-bot: I haz the power, 2017-06-23 23:35:35]

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-06-24 01:01:14]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1498260920192710,
"parent_rev": "602f4cdf9e0944a68fe91d12ead5b2ece186b8e3", "commit_rev":
"bf104c1b3d2dfae83774369a9c6a890ad57b9d6e"}

[commit-bot: I haz the power, 2017-06-24 01:01:23]

Description was changed from

==========
In mushrome, keep the state of the cursor locally.

In addition to telling the window server about state changes, record them
locally in the window manager so that queries about the current cursor state
return the correct values.

BUG=734806
==========

to

==========
In mushrome, keep the state of the cursor locally.

In addition to telling the window server about state changes, record them
locally in the window manager so that queries about the current cursor state
return the correct values.

BUG=734806

Review-Url: https://codereview.chromium.org/2955623003
Cr-Commit-Position: refs/heads/master@{#482102}
Committed:
https://chromium.googlesource.com/chromium/src/+/bf104c1b3d2dfae83774369a9c6a...
==========

[commit-bot: I haz the power, 2017-06-24 01:01:25]

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