exo: Implement support for zwp_stylus_v1 protocol

exo: Implement support for zwp_stylus_v1 protocol

This CL foward pointer tool, force and tilt information through the
zwp_stylus_v1 protocol in wayland.

BUG=628332
Committed: https://crrev.com/d2d5264c4fd4114d44d27d00f46963ab65a73527
Cr-Commit-Position: refs/heads/master@{#408468}

[denniskempin, 2016-07-27 00:09:05]

denniskempin@google.com changed reviewers:
+ reveman@chromium.org

[denniskempin, 2016-07-27 00:09:05]

PTAL!

[reveman, 2016-07-28 04:01:45]

lgtm, just a quick question and some minor nits

https://codereview.chromium.org/2153503005/diff/20001/components/exo/pointer.cc
File components/exo/pointer.cc (right):

https://codereview.chromium.org/2153503005/diff/20001/components/exo/pointer....
components/exo/pointer.cc:196: constexpr float kEpsilon =
std::numeric_limits<float>::epsilon();
is this useful? why are these type of events not filtered out at a much lower
level?

https://codereview.chromium.org/2153503005/diff/20001/components/exo/wayland/...
File components/exo/wayland/server.cc (right):

https://codereview.chromium.org/2153503005/diff/20001/components/exo/wayland/...
components/exo/wayland/server.cc:2898: // zwp_pointer_stylus_v1 interface:
nit: "pointer_stylus interface:" is consistent with other interface and will
make it easier to rename these interfaces in the future

https://codereview.chromium.org/2153503005/diff/20001/components/exo/wayland/...
components/exo/wayland/server.cc:2946: // zwp_stylus_v1 interface:
nit: "stylus interface:"

[denniskempin, 2016-07-28 17:17:40]

https://codereview.chromium.org/2153503005/diff/20001/components/exo/pointer.cc
File components/exo/pointer.cc (right):

https://codereview.chromium.org/2153503005/diff/20001/components/exo/pointer....
components/exo/pointer.cc:196: constexpr float kEpsilon =
std::numeric_limits<float>::epsilon();
On 2016/07/28 04:01:45, reveman wrote:
> is this useful? why are these type of events not filtered out at a much lower
> level?

as discussed we will keep this for consistency with OnPointerMotion.

https://codereview.chromium.org/2153503005/diff/20001/components/exo/wayland/...
File components/exo/wayland/server.cc (right):

https://codereview.chromium.org/2153503005/diff/20001/components/exo/wayland/...
components/exo/wayland/server.cc:2898: // zwp_pointer_stylus_v1 interface:
On 2016/07/28 04:01:45, reveman wrote:
> nit: "pointer_stylus interface:" is consistent with other interface and will
> make it easier to rename these interfaces in the future

Done.

https://codereview.chromium.org/2153503005/diff/20001/components/exo/wayland/...
components/exo/wayland/server.cc:2946: // zwp_stylus_v1 interface:
On 2016/07/28 04:01:45, reveman wrote:
> nit: "stylus interface:"

Done.

[denniskempin, 2016-07-28 17:30:23]

did a final end to end test and verified functionality.

[denniskempin, 2016-07-28 17:30:26]

The CQ bit was checked by denniskempin@google.com

[denniskempin, 2016-07-28 17:30:27]

The patchset sent to the CQ was uploaded after l-g-t-m from reveman@chromium.org
Link to the patchset: https://codereview.chromium.org/2153503005/#ps40001
(title: "fixed nits.")

[commit-bot: I haz the power, 2016-07-28 17:30:48]

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

[commit-bot: I haz the power, 2016-07-28 17:30:49]

This CL has an open dependency (Issue 2144283003 Patch 80001). Please resolve
the dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[denniskempin, 2016-07-28 17:32:19]

The CQ bit was checked by denniskempin@google.com

[commit-bot: I haz the power, 2016-07-28 17:32:38]

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

[commit-bot: I haz the power, 2016-07-28 17:32:39]

This CL has an open dependency (Issue 2144283003 Patch 80001). Please resolve
the dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[denniskempin, 2016-07-28 17:33:29]

okay, gotta wait for the dependency to go through the CQ first, it won't pick up
both of them.

[denniskempin, 2016-07-28 18:24:31]

The CQ bit was checked by denniskempin@google.com

[commit-bot: I haz the power, 2016-07-28 18:25:08]

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-07-28 18:35:17]

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

[commit-bot: I haz the power, 2016-07-28 18:35:18]

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

[denniskempin, 2016-07-28 19:36:47]

The CQ bit was checked by denniskempin@google.com

[denniskempin, 2016-07-28 19:36:48]

The patchset sent to the CQ was uploaded after l-g-t-m from reveman@chromium.org
Link to the patchset: https://codereview.chromium.org/2153503005/#ps60001
(title: "fixed compile issues for non-chromeos build")

[denniskempin, 2016-07-28 19:37:07]

The CQ bit was unchecked by denniskempin@google.com

[denniskempin, 2016-07-28 19:38:26]

The CQ bit was checked by denniskempin@google.com

[denniskempin, 2016-07-28 19:38:27]

The patchset sent to the CQ was uploaded after l-g-t-m from reveman@chromium.org
Link to the patchset: https://codereview.chromium.org/2153503005/#ps80001
(title: "2nd try")

[commit-bot: I haz the power, 2016-07-28 19:38:56]

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-07-28 20:40:14]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-07-28 20:41:54]

Description was changed from

==========
exo: Implement support for zwp_stylus_v1 protocol

This CL foward pointer tool, force and tilt information through the
zwp_stylus_v1 protocol in wayland.

BUG=628332
==========

to

==========
exo: Implement support for zwp_stylus_v1 protocol

This CL foward pointer tool, force and tilt information through the
zwp_stylus_v1 protocol in wayland.

BUG=628332

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

[commit-bot: I haz the power, 2016-07-28 20:41:54]

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