[PATCH 2/3] text: Have only one text_model_factory

[Jan Arne Petersen]

From: Philipp Brüschweiler <blei42 at gmail.com>

Have only one text_model_factory instead of one per seat.

This commit also introduces destruction of an input method when the
corresponding seat is removed.
---
 src/compositor.c   |  1 +
 src/compositor.h   |  3 +++
 src/text-backend.c | 75 +++++++++++++++++++++++++++++++++++++++---------------
 3 files changed, 58 insertions(+), 21 deletions(-)

diff --git a/src/compositor.c b/src/compositor.c
index 7370435..0daa479 100644
--- a/src/compositor.c
+++ b/src/compositor.c
@@ -3171,6 +3171,7 @@ weston_compositor_init(struct weston_compositor *ec,
 
 	screenshooter_create(ec);
 	text_cursor_position_notifier_create(ec);
+	text_model_factory_create(ec);
 
 	wl_data_device_manager_init(ec->wl_display);
 
diff --git a/src/compositor.h b/src/compositor.h
index 7a8058e..d65a797 100644
--- a/src/compositor.h
+++ b/src/compositor.h
@@ -742,6 +742,9 @@ void
 text_cursor_position_notifier_create(struct weston_compositor *ec);
 
 void
+text_model_factory_create(struct weston_compositor *ec);
+
+void
 input_method_create(struct weston_compositor *ec,
 		    struct weston_seat *seat);
 
diff --git a/src/text-backend.c b/src/text-backend.c
index af5d0f1..2835273 100644
--- a/src/text-backend.c
+++ b/src/text-backend.c
@@ -37,13 +37,19 @@ struct text_model {
 	struct wl_surface *surface;
 };
 
+struct text_model_factory {
+	struct wl_global *text_model_factory_global;
+	struct wl_listener destroy_listener;
+	
+	struct weston_compositor *ec;
+};
+
 struct input_method {
 	struct wl_resource *input_method_binding;
 	struct wl_global *input_method_global;
-	struct wl_global *text_model_factory_global;
 	struct wl_listener destroy_listener;
 
-	struct weston_compositor *ec;
+	struct weston_seat *seat;
 	struct text_model *model;
 
 	struct wl_list link;
@@ -183,24 +189,24 @@ static void text_model_factory_create_text_model(struct wl_client *client,
 						 struct wl_resource *resource,
 						 uint32_t id)
 {
-	struct input_method *input_method = resource->data;
+	struct text_model_factory *text_model_factory = resource->data;
 	struct text_model *text_model;
 
 	text_model = calloc(1, sizeof *text_model);
 
-	text_model->resource.destroy = destroy_text_model;
-
 	text_model->resource.object.id = id;
 	text_model->resource.object.interface = &text_model_interface;
 	text_model->resource.object.implementation =
 		(void (**)(void)) &text_model_implementation;
+
 	text_model->resource.data = text_model;
+	text_model->resource.destroy = destroy_text_model;
 
-	text_model->ec = input_method->ec;
-
-	wl_client_add_resource(client, &text_model->resource);
+	text_model->ec = text_model_factory->ec;
 
 	wl_list_init(&text_model->input_methods);
+
+	wl_client_add_resource(client, &text_model->resource);
 };
 
 static const struct text_model_factory_interface text_model_factory_implementation = {
@@ -213,13 +219,43 @@ bind_text_model_factory(struct wl_client *client,
 			uint32_t version,
 			uint32_t id)
 {
-	struct input_method *input_method = data;
+	struct text_model_factory *text_model_factory = data;
 
 	/* No checking for duplicate binding necessary.
 	 * No events have to be sent, so we don't need the return value. */
 	wl_client_add_object(client, &text_model_factory_interface,
 			     &text_model_factory_implementation,
-			     id, input_method);
+			     id, text_model_factory);
+}
+
+static void
+text_model_factory_notifier_destroy(struct wl_listener *listener, void *data)
+{
+	struct text_model_factory *text_model_factory =
+		container_of(listener, struct text_model_factory, destroy_listener);
+
+	wl_display_remove_global(text_model_factory->ec->wl_display,
+				 text_model_factory->text_model_factory_global);
+
+	free(text_model_factory);
+}
+
+WL_EXPORT void
+text_model_factory_create(struct weston_compositor *ec)
+{
+	struct text_model_factory *text_model_factory;
+
+	text_model_factory = calloc(1, sizeof *text_model_factory);
+
+	text_model_factory->ec = ec;
+
+	text_model_factory->text_model_factory_global =
+		wl_display_add_global(ec->wl_display,
+				      &text_model_factory_interface,
+				      text_model_factory, bind_text_model_factory);
+
+	text_model_factory->destroy_listener.notify = text_model_factory_notifier_destroy;
+	wl_signal_add(&ec->destroy_signal, &text_model_factory->destroy_listener);
 }
 
 static void
@@ -278,10 +314,12 @@ input_method_notifier_destroy(struct wl_listener *listener, void *data)
 	struct input_method *input_method =
 		container_of(listener, struct input_method, destroy_listener);
 
-	wl_display_remove_global(input_method->ec->wl_display,
+	if (input_method->model)
+		deactivate_text_model(input_method->model, input_method);
+
+	wl_display_remove_global(input_method->seat->compositor->wl_display,
 				 input_method->input_method_global);
-	wl_display_remove_global(input_method->ec->wl_display,
-				 input_method->text_model_factory_global);
+
 	free(input_method);
 }
 
@@ -315,7 +353,7 @@ input_method_init_seat(struct weston_seat *seat)
 	seat->input_method->focus_listener_initialized = 1;
 }
 
-void
+WL_EXPORT void
 input_method_create(struct weston_compositor *ec,
 		    struct weston_seat *seat)
 {
@@ -323,7 +361,7 @@ input_method_create(struct weston_compositor *ec,
 
 	input_method = calloc(1, sizeof *input_method);
 
-	input_method->ec = ec;
+	input_method->seat = seat;
 	input_method->model = NULL;
 	input_method->focus_listener_initialized = 0;
 
@@ -332,13 +370,8 @@ input_method_create(struct weston_compositor *ec,
 				      &input_method_interface,
 				      input_method, bind_input_method);
 
-	input_method->text_model_factory_global =
-		wl_display_add_global(ec->wl_display,
-				      &text_model_factory_interface,
-				      input_method, bind_text_model_factory);
-
 	input_method->destroy_listener.notify = input_method_notifier_destroy;
-	wl_signal_add(&ec->destroy_signal, &input_method->destroy_listener);
+	wl_signal_add(&seat->seat.destroy_signal, &input_method->destroy_listener);
 
 	seat->input_method = input_method;
 }
-- 
1.7.11.4