views/mus: Connect cursor events to the mus window.

views/mus: Connect cursor events to the mus window.

This makes views programs backed by a mus window able to change the
cursor for its bounds. mus already has partial support for per-window
cursors, this just needed to be plumbed.

BUG=548428
Committed: https://crrev.com/19ddf3a6399fbee33681ea74d41529b4915e9c03
Cr-Commit-Position: refs/heads/master@{#391660}

[Elliot Glaysher, 2016-05-04 19:27:57]

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

[sky, 2016-05-04 20:18:59]

https://codereview.chromium.org/1945423002/diff/20001/ui/views/mus/native_wid...
File ui/views/mus/native_widget_mus.cc (right):

https://codereview.chromium.org/1945423002/diff/20001/ui/views/mus/native_wid...
ui/views/mus/native_widget_mus.cc:491: cursor_manager_.reset(new
wm::CursorManager(
The wm also uses NativeWidgetMus. Do we only want this for the client? If so,
you could key off the surface_type. Keying off the surface_type is a bit
obscure. We should really add an enum indicating client or wm, but that can be
done separately.

[Elliot Glaysher, 2016-05-04 20:55:02]

https://codereview.chromium.org/1945423002/diff/20001/ui/views/mus/native_wid...
File ui/views/mus/native_widget_mus.cc (right):

https://codereview.chromium.org/1945423002/diff/20001/ui/views/mus/native_wid...
ui/views/mus/native_widget_mus.cc:491: cursor_manager_.reset(new
wm::CursorManager(
On 2016/05/04 20:18:59, sky wrote:
> The wm also uses NativeWidgetMus. Do we only want this for the client? If so,
> you could key off the surface_type. Keying off the surface_type is a bit
> obscure. We should really add an enum indicating client or wm, but that can be
> done separately.

Added temporary if check around this block which will be removed when I add a
window manager interface to change the pointer in the nonclient area as
discussed over lunch.

[sky, 2016-05-04 21:43:49]

LGTM

[Elliot Glaysher, 2016-05-04 21:44:30]

The CQ bit was checked by erg@chromium.org

[commit-bot: I haz the power, 2016-05-04 21:45:15]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1945423002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1945423002/40001

[commit-bot: I haz the power, 2016-05-04 21:54:08]

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

[commit-bot: I haz the power, 2016-05-04 21:54:11]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Elliot Glaysher, 2016-05-04 21:58:56]

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

[Elliot Glaysher, 2016-05-04 21:58:56]

+sadrul for owners stamp on RWHVA.

[sadrul, 2016-05-04 22:06:35]

lgtm

[Elliot Glaysher, 2016-05-04 22:12:44]

The CQ bit was checked by erg@chromium.org

[commit-bot: I haz the power, 2016-05-04 22:13:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1945423002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1945423002/40001

[commit-bot: I haz the power, 2016-05-04 22:18:28]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-04 22:20:17]

Description was changed from

==========
views/mus: Connect cursor events to the mus window.

This makes views programs backed by a mus window able to change the
cursor for its bounds. mus already has partial support for per-window
cursors, this just needed to be plumbed.

BUG=548428
==========

to

==========
views/mus: Connect cursor events to the mus window.

This makes views programs backed by a mus window able to change the
cursor for its bounds. mus already has partial support for per-window
cursors, this just needed to be plumbed.

BUG=548428

Committed: https://crrev.com/19ddf3a6399fbee33681ea74d41529b4915e9c03
Cr-Commit-Position: refs/heads/master@{#391660}
==========

[commit-bot: I haz the power, 2016-05-04 22:20:18]

Patchset 3 (id:??) landed as
https://crrev.com/19ddf3a6399fbee33681ea74d41529b4915e9c03
Cr-Commit-Position: refs/heads/master@{#391660}