Merge branch 'master' into fix-1975

5 files changed, 70 insertions, 173 deletions

diff --git a/sway/desktop/output.c b/sway/desktop/output.c
index 577279ac..0350e437 100644
--- a/sway/desktop/output.c
+++ b/sway/desktop/output.c
@@ -11,7 +11,6 @@
 #include <wlr/types/wlr_output_layout.h>
 #include <wlr/types/wlr_output.h>
 #include <wlr/types/wlr_surface.h>
-#include <wlr/types/wlr_wl_shell.h>
 #include <wlr/util/region.h>
 #include "log.h"
 #include "sway/config.h"
diff --git a/sway/desktop/wl_shell.c b/sway/desktop/wl_shell.c
deleted file mode 100644
index 99e8947b..00000000
--- a/sway/desktop/wl_shell.c
+++ /dev/null
@@ -1,165 +0,0 @@
-#define _POSIX_C_SOURCE 199309L
-#include <stdbool.h>
-#include <stdlib.h>
-#include <wayland-server.h>
-#include <wlr/types/wlr_wl_shell.h>
-#include "sway/tree/container.h"
-#include "sway/tree/layout.h"
-#include "sway/server.h"
-#include "sway/tree/view.h"
-#include "sway/input/seat.h"
-#include "sway/input/input-manager.h"
-#include "log.h"
-
-static struct sway_wl_shell_view *wl_shell_view_from_view(
-		struct sway_view *view) {
-	if (!sway_assert(view->type == SWAY_VIEW_WL_SHELL,
-			"Expected wl_shell view")) {
-		return NULL;
-	}
-	return (struct sway_wl_shell_view *)view;
-}
-
-static const char *get_prop(struct sway_view *view, enum sway_view_prop prop) {
-	if (wl_shell_view_from_view(view) == NULL) {
-		return NULL;
-	}
-	switch (prop) {
-	case VIEW_PROP_TITLE:
-		return view->wlr_wl_shell_surface->title;
-	case VIEW_PROP_CLASS:
-		return view->wlr_wl_shell_surface->class;
-	default:
-		return NULL;
-	}
-}
-
-static void configure(struct sway_view *view, double ox, double oy, int width,
-		int height) {
-	struct sway_wl_shell_view *wl_shell_view = wl_shell_view_from_view(view);
-	if (wl_shell_view == NULL) {
-		return;
-	}
-	wl_shell_view->pending_width = width;
-	wl_shell_view->pending_height = height;
-	wlr_wl_shell_surface_configure(view->wlr_wl_shell_surface, 0, width, height);
-}
-
-static void _close(struct sway_view *view) {
-	if (wl_shell_view_from_view(view) == NULL) {
-		return;
-	}
-
-	wl_client_destroy(view->wlr_wl_shell_surface->client);
-}
-
-static void destroy(struct sway_view *view) {
-	struct sway_wl_shell_view *wl_shell_view = wl_shell_view_from_view(view);
-	if (wl_shell_view == NULL) {
-		return;
-	}
-	wl_list_remove(&wl_shell_view->commit.link);
-	wl_list_remove(&wl_shell_view->destroy.link);
-	wl_list_remove(&wl_shell_view->request_fullscreen.link);
-	wl_list_remove(&wl_shell_view->set_state.link);
-	free(wl_shell_view);
-}
-
-static void set_fullscreen(struct sway_view *view, bool fullscreen) {
-	// TODO
-}
-
-static const struct sway_view_impl view_impl = {
-	.get_prop = get_prop,
-	.configure = configure,
-	.close = _close,
-	.destroy = destroy,
-	.set_fullscreen = set_fullscreen,
-};
-
-static void handle_commit(struct wl_listener *listener, void *data) {
-	struct sway_wl_shell_view *wl_shell_view =
-		wl_container_of(listener, wl_shell_view, commit);
-	struct sway_view *view = &wl_shell_view->view;
-	// NOTE: We intentionally discard the view's desired width here
-	// TODO: Let floating views do whatever
-	view_update_size(view, wl_shell_view->pending_width,
-		wl_shell_view->pending_height);
-	view_damage_from(view);
-}
-
-static void handle_destroy(struct wl_listener *listener, void *data) {
-	struct sway_wl_shell_view *wl_shell_view =
-		wl_container_of(listener, wl_shell_view, destroy);
-	view_destroy(&wl_shell_view->view);
-}
-
-static void handle_request_fullscreen(struct wl_listener *listener, void *data) {
-	struct sway_wl_shell_view *wl_shell_view =
-		wl_container_of(listener, wl_shell_view, request_fullscreen);
-	view_set_fullscreen(&wl_shell_view->view, true);
-}
-
-static void handle_set_state(struct wl_listener *listener, void *data) {
-	struct sway_wl_shell_view *wl_shell_view =
-		wl_container_of(listener, wl_shell_view, set_state);
-	struct sway_view *view = &wl_shell_view->view;
-	struct wlr_wl_shell_surface *surface = view->wlr_wl_shell_surface;
-	if (view->is_fullscreen &&
-			surface->state != WLR_WL_SHELL_SURFACE_STATE_FULLSCREEN) {
-		view_set_fullscreen(view, false);
-	}
-}
-
-void handle_wl_shell_surface(struct wl_listener *listener, void *data) {
-	struct sway_server *server = wl_container_of(listener, server,
-		wl_shell_surface);
-	struct wlr_wl_shell_surface *shell_surface = data;
-
-	if (shell_surface->state == WLR_WL_SHELL_SURFACE_STATE_POPUP) {
-		// popups don't get views
-		wlr_log(L_DEBUG, "New wl_shell popup");
-		return;
-	}
-
-	// TODO: make transient windows floating
-
-	wlr_log(L_DEBUG, "New wl_shell toplevel title='%s' app_id='%s'",
-			shell_surface->title, shell_surface->class);
-	wlr_wl_shell_surface_ping(shell_surface);
-
-	struct sway_wl_shell_view *wl_shell_view =
-		calloc(1, sizeof(struct sway_wl_shell_view));
-	if (!sway_assert(wl_shell_view, "Failed to allocate view")) {
-		return;
-	}
-
-	view_init(&wl_shell_view->view, SWAY_VIEW_WL_SHELL, &view_impl);
-	wl_shell_view->view.wlr_wl_shell_surface = shell_surface;
-
-	// TODO:
-	// - Wire up listeners
-	// - Look up pid and open on appropriate workspace
-	// - Set new view to maximized so it behaves nicely
-	// - Criteria
-
-	wl_shell_view->commit.notify = handle_commit;
-	wl_signal_add(&shell_surface->surface->events.commit,
-		&wl_shell_view->commit);
-
-	wl_shell_view->destroy.notify = handle_destroy;
-	wl_signal_add(&shell_surface->events.destroy, &wl_shell_view->destroy);
-
-	wl_shell_view->request_fullscreen.notify = handle_request_fullscreen;
-	wl_signal_add(&shell_surface->events.request_fullscreen,
-			&wl_shell_view->request_fullscreen);
-
-	wl_shell_view->set_state.notify = handle_set_state;
-	wl_signal_add(&shell_surface->events.set_state, &wl_shell_view->set_state);
-
-	view_map(&wl_shell_view->view, shell_surface->surface);
-
-	if (shell_surface->state == WLR_WL_SHELL_SURFACE_STATE_FULLSCREEN) {
-		view_set_fullscreen(&wl_shell_view->view, true);
-	}
-}
diff --git a/sway/desktop/xdg_shell.c b/sway/desktop/xdg_shell.c
index 48b659f2..9a0d282b 100644
--- a/sway/desktop/xdg_shell.c
+++ b/sway/desktop/xdg_shell.c
@@ -72,7 +72,7 @@ static struct sway_xdg_shell_view *xdg_shell_view_from_view(
 	return (struct sway_xdg_shell_view *)view;
 }
 
-static const char *get_prop(struct sway_view *view, enum sway_view_prop prop) {
+static const char *get_string_prop(struct sway_view *view, enum sway_view_prop prop) {
 	if (xdg_shell_view_from_view(view) == NULL) {
 		return NULL;
 	}
@@ -150,7 +150,7 @@ static void destroy(struct sway_view *view) {
 }
 
 static const struct sway_view_impl view_impl = {
-	.get_prop = get_prop,
+	.get_string_prop = get_string_prop,
 	.configure = configure,
 	.set_activated = set_activated,
 	.set_fullscreen = set_fullscreen,
diff --git a/sway/desktop/xdg_shell_v6.c b/sway/desktop/xdg_shell_v6.c
index e9051b6c..d098c797 100644
--- a/sway/desktop/xdg_shell_v6.c
+++ b/sway/desktop/xdg_shell_v6.c
@@ -72,7 +72,7 @@ static struct sway_xdg_shell_v6_view *xdg_shell_v6_view_from_view(
 	return (struct sway_xdg_shell_v6_view *)view;
 }
 
-static const char *get_prop(struct sway_view *view, enum sway_view_prop prop) {
+static const char *get_string_prop(struct sway_view *view, enum sway_view_prop prop) {
 	if (xdg_shell_v6_view_from_view(view) == NULL) {
 		return NULL;
 	}
@@ -150,7 +150,7 @@ static void destroy(struct sway_view *view) {
 }
 
 static const struct sway_view_impl view_impl = {
-	.get_prop = get_prop,
+	.get_string_prop = get_string_prop,
 	.configure = configure,
 	.set_activated = set_activated,
 	.set_fullscreen = set_fullscreen,
diff --git a/sway/desktop/xwayland.c b/sway/desktop/xwayland.c
index 1d3c857d..6a99a66a 100644
--- a/sway/desktop/xwayland.c
+++ b/sway/desktop/xwayland.c
@@ -122,7 +122,7 @@ static struct sway_xwayland_view *xwayland_view_from_view(
 	return (struct sway_xwayland_view *)view;
 }
 
-static const char *get_prop(struct sway_view *view, enum sway_view_prop prop) {
+static const char *get_string_prop(struct sway_view *view, enum sway_view_prop prop) {
 	if (xwayland_view_from_view(view) == NULL) {
 		return NULL;
 	}
@@ -131,11 +131,27 @@ static const char *get_prop(struct sway_view *view, enum sway_view_prop prop) {
 		return view->wlr_xwayland_surface->title;
 	case VIEW_PROP_CLASS:
 		return view->wlr_xwayland_surface->class;
+	case VIEW_PROP_INSTANCE:
+		return view->wlr_xwayland_surface->instance;
 	default:
 		return NULL;
 	}
 }
 
+static uint32_t get_int_prop(struct sway_view *view, enum sway_view_prop prop) {
+	if (xwayland_view_from_view(view) == NULL) {
+		return 0;
+	}
+	switch (prop) {
+	case VIEW_PROP_X11_WINDOW_ID:
+		return view->wlr_xwayland_surface->window_id;
+	case VIEW_PROP_WINDOW_TYPE:
+		return *view->wlr_xwayland_surface->window_type;
+	default:
+		return 0;
+	}
+}
+
 static void configure(struct sway_view *view, double ox, double oy, int width,
 		int height) {
 	struct sway_xwayland_view *xwayland_view = xwayland_view_from_view(view);
@@ -196,13 +212,17 @@ static void destroy(struct sway_view *view) {
 	wl_list_remove(&xwayland_view->destroy.link);
 	wl_list_remove(&xwayland_view->request_configure.link);
 	wl_list_remove(&xwayland_view->request_fullscreen.link);
+	wl_list_remove(&xwayland_view->set_title.link);
+	wl_list_remove(&xwayland_view->set_class.link);
+	wl_list_remove(&xwayland_view->set_window_type.link);
 	wl_list_remove(&xwayland_view->map.link);
 	wl_list_remove(&xwayland_view->unmap.link);
 	free(xwayland_view);
 }
 
 static const struct sway_view_impl view_impl = {
-	.get_prop = get_prop,
+	.get_string_prop = get_string_prop,
+	.get_int_prop = get_int_prop,
 	.configure = configure,
 	.set_activated = set_activated,
 	.set_fullscreen = set_fullscreen,
@@ -219,7 +239,6 @@ static void handle_commit(struct wl_listener *listener, void *data) {
 	view_update_size(view, xwayland_view->pending_width,
 		xwayland_view->pending_height);
 	view_damage_from(view);
-	view_update_title(view, false);
 }
 
 static void handle_unmap(struct wl_listener *listener, void *data) {
@@ -281,6 +300,40 @@ static void handle_request_fullscreen(struct wl_listener *listener, void *data)
 	view_set_fullscreen(view, xsurface->fullscreen);
 }
 
+static void handle_set_title(struct wl_listener *listener, void *data) {
+	struct sway_xwayland_view *xwayland_view =
+		wl_container_of(listener, xwayland_view, set_title);
+	struct sway_view *view = &xwayland_view->view;
+	struct wlr_xwayland_surface *xsurface = view->wlr_xwayland_surface;
+	if (!xsurface->mapped) {
+		return;
+	}
+	view_update_title(view, false);
+	view_execute_criteria(view);
+}
+
+static void handle_set_class(struct wl_listener *listener, void *data) {
+	struct sway_xwayland_view *xwayland_view =
+		wl_container_of(listener, xwayland_view, set_class);
+	struct sway_view *view = &xwayland_view->view;
+	struct wlr_xwayland_surface *xsurface = view->wlr_xwayland_surface;
+	if (!xsurface->mapped) {
+		return;
+	}
+	view_execute_criteria(view);
+}
+
+static void handle_set_window_type(struct wl_listener *listener, void *data) {
+	struct sway_xwayland_view *xwayland_view =
+		wl_container_of(listener, xwayland_view, set_window_type);
+	struct sway_view *view = &xwayland_view->view;
+	struct wlr_xwayland_surface *xsurface = view->wlr_xwayland_surface;
+	if (!xsurface->mapped) {
+		return;
+	}
+	view_execute_criteria(view);
+}
+
 void handle_xwayland_surface(struct wl_listener *listener, void *data) {
 	struct sway_server *server = wl_container_of(listener, server,
 		xwayland_surface);
@@ -319,6 +372,16 @@ void handle_xwayland_surface(struct wl_listener *listener, void *data) {
 		&xwayland_view->request_fullscreen);
 	xwayland_view->request_fullscreen.notify = handle_request_fullscreen;
 
+	wl_signal_add(&xsurface->events.set_title, &xwayland_view->set_title);
+	xwayland_view->set_title.notify = handle_set_title;
+
+	wl_signal_add(&xsurface->events.set_class, &xwayland_view->set_class);
+	xwayland_view->set_class.notify = handle_set_class;
+
+	wl_signal_add(&xsurface->events.set_window_type,
+			&xwayland_view->set_window_type);
+	xwayland_view->set_window_type.notify = handle_set_window_type;
+
 	wl_signal_add(&xsurface->events.unmap, &xwayland_view->unmap);
 	xwayland_view->unmap.notify = handle_unmap;