exo: Implement lazy cursor changes.

exo: Implement lazy cursor changes.

Pointer::SetCursor() is now required to be called in response to
focus change events. This provides more seamless cursor changes
as the cursor will remain the same until it has been re-defined.

Simply clients that never define the cursor will now get the
default cursor instead of no cursor at all. Clients that draw
their own cursor now have a chance to provide a new frame before
the cursor is changed.

BUG=549781
Committed: https://crrev.com/70baca1c3bcf3ac902f4313da5e74b2592f85fb4
Cr-Commit-Position: refs/heads/master@{#396919}

[reveman, 2016-05-30 17:42:32]

reveman@chromium.org changed reviewers:
+ dcastagna@chromium.org

[reveman, 2016-05-30 19:50:31]

Description was changed from

==========
exo: Implement lazy cursor.

Pointer::SetCursor() is now required to be called in response to
focus change event. This provides a more seamless cursor changes
as the cursor will remain the same until it has been re-defined.

Simply clients that never define the cursor will now get the
default cursor instead of no cursor at all. Clients that draw
thier own cursor now has a chance to provide a new frame before
the cursor is changed.

BUG=549781
==========

to

==========
exo: Implement lazy cursor.

Pointer::SetCursor() is now required to be called in response to
focus change event. This provides a more seamless cursor changes
as the cursor will remain the same until it has been re-defined.

Simply clients that never define the cursor will now get the
default cursor instead of no cursor at all. Clients that draw
thier own cursor now has a chance to provide a new frame before
the cursor is changed.

BUG=549781, b/27827591
==========

[reveman, 2016-05-31 03:29:06]

Description was changed from

==========
exo: Implement lazy cursor.

Pointer::SetCursor() is now required to be called in response to
focus change event. This provides a more seamless cursor changes
as the cursor will remain the same until it has been re-defined.

Simply clients that never define the cursor will now get the
default cursor instead of no cursor at all. Clients that draw
thier own cursor now has a chance to provide a new frame before
the cursor is changed.

BUG=549781, b/27827591
==========

to

==========
exo: Implement lazy cursor changes.

Pointer::SetCursor() is now required to be called in response to
focus change events. This provides more seamless cursor changes
as the cursor will remain the same until it has been re-defined.

Simply clients that never define the cursor will now get the
default cursor instead of no cursor at all. Clients that draw
their own cursor now has a chance to provide a new frame before
the cursor is changed.

BUG=549781
==========

[Daniele Castagna, 2016-05-31 19:32:03]

nit: in the CL description "Clients [...] now has"

LGTM.

https://codereview.chromium.org/2028443002/diff/1/components/exo/surface.cc
File components/exo/surface.cc (right):

https://codereview.chromium.org/2028443002/diff/1/components/exo/surface.cc#n...
components/exo/surface.cc:84: return surface_->HasCursorProvider() ?
ui::kCursorNone : ui::kCursorNull;
I understand that when kCursorNull is returns the default cursor is used, but
something like kCursorDefault would make this like more readable.

[reveman, 2016-05-31 19:47:09]

Description was changed from

==========
exo: Implement lazy cursor changes.

Pointer::SetCursor() is now required to be called in response to
focus change events. This provides more seamless cursor changes
as the cursor will remain the same until it has been re-defined.

Simply clients that never define the cursor will now get the
default cursor instead of no cursor at all. Clients that draw
their own cursor now has a chance to provide a new frame before
the cursor is changed.

BUG=549781
==========

to

==========
exo: Implement lazy cursor changes.

Pointer::SetCursor() is now required to be called in response to
focus change events. This provides more seamless cursor changes
as the cursor will remain the same until it has been re-defined.

Simply clients that never define the cursor will now get the
default cursor instead of no cursor at all. Clients that draw
their own cursor now have a chance to provide a new frame before
the cursor is changed.

BUG=549781
==========

[reveman, 2016-05-31 19:47:41]

The CQ bit was checked by reveman@chromium.org

[commit-bot: I haz the power, 2016-05-31 19:48:27]

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

[commit-bot: I haz the power, 2016-05-31 20:35:51]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-31 20:45:28]

Description was changed from

==========
exo: Implement lazy cursor changes.

Pointer::SetCursor() is now required to be called in response to
focus change events. This provides more seamless cursor changes
as the cursor will remain the same until it has been re-defined.

Simply clients that never define the cursor will now get the
default cursor instead of no cursor at all. Clients that draw
their own cursor now have a chance to provide a new frame before
the cursor is changed.

BUG=549781
==========

to

==========
exo: Implement lazy cursor changes.

Pointer::SetCursor() is now required to be called in response to
focus change events. This provides more seamless cursor changes
as the cursor will remain the same until it has been re-defined.

Simply clients that never define the cursor will now get the
default cursor instead of no cursor at all. Clients that draw
their own cursor now have a chance to provide a new frame before
the cursor is changed.

BUG=549781

Committed: https://crrev.com/70baca1c3bcf3ac902f4313da5e74b2592f85fb4
Cr-Commit-Position: refs/heads/master@{#396919}
==========

[commit-bot: I haz the power, 2016-05-31 20:45:29]

Patchset 1 (id:??) landed as
https://crrev.com/70baca1c3bcf3ac902f4313da5e74b2592f85fb4
Cr-Commit-Position: refs/heads/master@{#396919}