From 539d85f69dc43cb75918ad3f3d705edf80db08f8 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Kristian=20H=C3=B8gsberg?= <krh@bitplanet.net>
Date: Mon, 13 Aug 2012 23:29:53 -0400
Subject: [PATCH] compositor-wayland: Call notify_pointer_focus() for content
 area only

We don't want to send enter/leave events to the compositor when the pointer
enters the surface, only when the pointer enters the content area.  This
avoids hiding the cursor when entering the frame and sending out-of-bounds
coordinates to notify_pointer_focus().
---
 src/compositor-wayland.c | 48 +++++++++++++++++++++++++++++++++++++++++-------
 1 file changed, 41 insertions(+), 7 deletions(-)

diff --git a/src/compositor-wayland.c b/src/compositor-wayland.c
index e5ff658..b45b792 100644
--- a/src/compositor-wayland.c
+++ b/src/compositor-wayland.c
@@ -93,6 +93,9 @@ struct wayland_input {
 	struct wl_touch *touch;
 	struct wl_list link;
 	uint32_t key_serial;
+	uint32_t enter_serial;
+	int focus;
+	struct wayland_output *output;
 };
 
 
@@ -524,6 +527,34 @@ static const struct wl_output_listener output_listener = {
 	display_handle_mode
 };
 
+static void
+check_focus(struct wayland_input *input, wl_fixed_t x, wl_fixed_t y)
+{
+	struct wayland_compositor *c = input->compositor;
+	int width, height, inside;
+
+	width = input->output->mode.width;
+	height = input->output->mode.height;
+
+	inside = c->border.left <= wl_fixed_to_int(x) &&
+		wl_fixed_to_int(x) < width + c->border.left &&
+		c->border.top <= wl_fixed_to_int(y) &&
+		wl_fixed_to_int(y) < height + c->border.top;
+
+	if (!input->focus && inside) {
+		notify_pointer_focus(&input->base, &input->output->base,
+				     x - wl_fixed_from_int(c->border.left),
+				     y = wl_fixed_from_int(c->border.top));
+		wl_pointer_set_cursor(input->pointer,
+				      input->enter_serial, NULL, 0, 0);
+	} else if (input->focus && !inside) {
+		notify_pointer_focus(&input->base, NULL, 0, 0);
+		/* FIXME: Should set default cursor here. */
+	}
+
+	input->focus = inside;
+}
+
 /* parent input interface */
 static void
 input_handle_pointer_enter(void *data, struct wl_pointer *pointer,
@@ -531,13 +562,12 @@ input_handle_pointer_enter(void *data, struct wl_pointer *pointer,
 			   wl_fixed_t x, wl_fixed_t y)
 {
 	struct wayland_input *input = data;
-	struct wayland_output *output;
 
 	/* XXX: If we get a modifier event immediately before the focus,
 	 *      we should try to keep the same serial. */
-	output = wl_surface_get_user_data(surface);
-	notify_pointer_focus(&input->base, &output->base, x, y);
-	wl_pointer_set_cursor(input->pointer, serial, NULL, 0, 0);
+	input->enter_serial = serial;
+	input->output = wl_surface_get_user_data(surface);
+	check_focus(input, x, y);
 }
 
 static void
@@ -547,6 +577,8 @@ input_handle_pointer_leave(void *data, struct wl_pointer *pointer,
 	struct wayland_input *input = data;
 
 	notify_pointer_focus(&input->base, NULL, 0, 0);
+	input->output = NULL;
+	input->focus = 0;
 }
 
 static void
@@ -556,9 +588,11 @@ input_handle_motion(void *data, struct wl_pointer *pointer,
 	struct wayland_input *input = data;
 	struct wayland_compositor *c = input->compositor;
 
-	notify_motion(&input->base, time,
-		      x - wl_fixed_from_int(c->border.left),
-		      y - wl_fixed_from_int(c->border.top));
+	check_focus(input, x, y);
+	if (input->focus)
+		notify_motion(&input->base, time,
+			      x - wl_fixed_from_int(c->border.left),
+			      y - wl_fixed_from_int(c->border.top));
 }
 
 static void
-- 
2.7.4