Replace wlr_key_state with wl_keyboard_key_state

[ifreund]

There's no reason to have duplicate enums.

---

Breaking change: enum wlr_key_state has been replaced with enum wl_keyboard_key_state.

[ifreund]

I've just realized that we have the same issue with wlr_button_state. If this is deemed to be the direction we want to take I would be happy to make the parallel change there.

[emersion]

I think the original intention was to keep apart the protocol bits and the backend interface bits. But duplicating the enums doesn't really make sense to me, since wlroots is a Wayland compositor library, not a generic GUI library.

[ifreund]

Found another: wlr_edges duplicates xdg_toplevel.resize_edge.

[Xyene]

Note that Sway, at least, uses wlr_edges to represent resize edges of non-xdg_toplevel things, e.g. Xwayland containers. (It seems to be used to similar effect in xwayland/xwm.c etc.)

[emersion]

LGTM

The enum wl_keyboard_key_state values have the same values as enum wlr_key_state.

[emersion]

Thanks!