exo: Improve motion event synchronization.

exo: Improve motion event synchronization.

This improves the synchronization by deferring the client side
wait until commit time.

BUG=
TEST=wayland_motion_events

Committed: https://crrev.com/f9ac10db3685a55ceac6105d9d3b8136f86b00b0
Cr-Commit-Position: refs/heads/master@{#437783}

[reveman, 2016-12-11 01:33:49]

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

[Daniele Castagna, 2016-12-11 01:44:15]

https://codereview.chromium.org/2563263002/diff/1/components/exo/wayland/clie...
File components/exo/wayland/clients/motion_events.cc (right):

https://codereview.chromium.org/2563263002/diff/1/components/exo/wayland/clie...
components/exo/wayland/clients/motion_events.cc:691: if
(buffer->egl_sync->is_valid()) {
Can't egl_sync be nullptr?
Should this be egl_sync && egl_sync->is_valid?

[reveman, 2016-12-11 01:50:51]

https://codereview.chromium.org/2563263002/diff/1/components/exo/wayland/clie...
File components/exo/wayland/clients/motion_events.cc (right):

https://codereview.chromium.org/2563263002/diff/1/components/exo/wayland/clie...
components/exo/wayland/clients/motion_events.cc:691: if
(buffer->egl_sync->is_valid()) {
On 2016/12/11 at 01:44:15, Daniele Castagna wrote:
> Can't egl_sync be nullptr?
> Should this be egl_sync && egl_sync->is_valid?

Should be "if (buffer->egl_sync)". Done.

[Daniele Castagna, 2016-12-11 01:52:34]

lgtm

[reveman, 2016-12-11 01:52:56]

The CQ bit was checked by reveman@chromium.org

[commit-bot: I haz the power, 2016-12-11 01:53:06]

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-11 02:02:53]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481421176225840,
"parent_rev": "22c5ca0f0ee97ba1940ccb764b019a8824457bba", "commit_rev":
"6b2d10256884c3dae67031e8096fdf7889404244"}

[commit-bot: I haz the power, 2016-12-11 02:03:36]

Description was changed from

==========
exo: Improve motion event synchronization.

This improves the synchronization by deferring the client side
wait until commit time.

BUG=
TEST=wayland_motion_events
==========

to

==========
exo: Improve motion event synchronization.

This improves the synchronization by deferring the client side
wait until commit time.

BUG=
TEST=wayland_motion_events

Review-Url: https://codereview.chromium.org/2563263002
==========

[commit-bot: I haz the power, 2016-12-11 02:03:37]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-12 15:07:35]

Description was changed from

==========
exo: Improve motion event synchronization.

This improves the synchronization by deferring the client side
wait until commit time.

BUG=
TEST=wayland_motion_events

Review-Url: https://codereview.chromium.org/2563263002
==========

to

==========
exo: Improve motion event synchronization.

This improves the synchronization by deferring the client side
wait until commit time.

BUG=
TEST=wayland_motion_events

Committed: https://crrev.com/f9ac10db3685a55ceac6105d9d3b8136f86b00b0
Cr-Commit-Position: refs/heads/master@{#437783}
==========

[commit-bot: I haz the power, 2016-12-12 15:07:35]

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