compositor-drm: Fix cursor view size check
authorDaniel Stone <daniels@collabora.com>
Tue, 16 Jun 2015 17:42:23 +0000 (18:42 +0100)
committerDaniel Stone <daniels@collabora.com>
Thu, 16 Jul 2015 15:46:21 +0000 (16:46 +0100)
Instead of testing against a hardcoded 64x64 pixel size to see if a view
is suitable for promotion to a cursor plane, use our cursor_width and
cursor_height variables.

Signed-off-by: Daniel Stone <daniels@collabora.com>
Reviewed-by: Derek Foreman <derekf@osg.samsung.com>
src/compositor-drm.c

index 28b1c7e..c1c6f00 100644 (file)
@@ -127,8 +127,8 @@ struct drm_backend {
 
        struct udev_input input;
 
-       uint32_t cursor_width;
-       uint32_t cursor_height;
+       int32_t cursor_width;
+       int32_t cursor_height;
 };
 
 struct drm_mode {
@@ -1081,7 +1081,8 @@ drm_output_prepare_cursor_view(struct drm_output *output,
                return NULL;
        if (ev->surface->buffer_ref.buffer == NULL ||
            !wl_shm_buffer_get(ev->surface->buffer_ref.buffer->resource) ||
-           ev->surface->width > 64 || ev->surface->height > 64)
+           ev->surface->width > b->cursor_width ||
+           ev->surface->height > b->cursor_height)
                return NULL;
 
        output->cursor_view = ev;