bindings: Make run binding functions take apropriate device instead of a seat
Going from seat to input device requires that we test the device
before relying on the pointer. In all of these binding functions
we can trust exactly one input device type directly. If we pass
that in instead of a seat it's more obvious that we can trust
the one pointer we have.
When a seat is required, we can access through the device we have
and use that to get to other device types for the seat, provided
we validate them appropriately.
Reviewed-by: Jonas Ådahl <jadahl@gmail.com>
Signed-off-by: Derek Foreman <derekf@osg.samsung.com>
diff --git a/src/compositor.h b/src/compositor.h
index ee40ecc..d0c6034 100644
--- a/src/compositor.h
+++ b/src/compositor.h
@@ -1213,31 +1213,32 @@
void
weston_compositor_run_key_binding(struct weston_compositor *compositor,
- struct weston_seat *seat, uint32_t time,
+ struct weston_keyboard *keyboard,
+ uint32_t time,
uint32_t key,
enum wl_keyboard_key_state state);
void
weston_compositor_run_modifier_binding(struct weston_compositor *compositor,
- struct weston_seat *seat,
+ struct weston_keyboard *keyboard,
enum weston_keyboard_modifier modifier,
enum wl_keyboard_key_state state);
void
weston_compositor_run_button_binding(struct weston_compositor *compositor,
- struct weston_seat *seat, uint32_t time,
+ struct weston_pointer *pointer, uint32_t time,
uint32_t button,
enum wl_pointer_button_state value);
void
weston_compositor_run_touch_binding(struct weston_compositor *compositor,
- struct weston_seat *seat, uint32_t time,
+ struct weston_touch *touch, uint32_t time,
int touch_type);
int
weston_compositor_run_axis_binding(struct weston_compositor *compositor,
- struct weston_seat *seat, uint32_t time,
+ struct weston_pointer *pointer, uint32_t time,
uint32_t axis, int32_t value);
int
weston_compositor_run_debug_binding(struct weston_compositor *compositor,
- struct weston_seat *seat, uint32_t time,
+ struct weston_keyboard *keyboard, uint32_t time,
uint32_t key,
enum wl_keyboard_key_state state);