src/foreign.c: refactor

This commit is contained in:
Consolatis 2023-02-01 09:27:25 +01:00 committed by Johan Malm
parent 82e9e866ec
commit 14a0c83ae0
4 changed files with 85 additions and 87 deletions

View file

@ -73,12 +73,15 @@ struct view {
struct ssd *ssd; struct ssd *ssd;
struct wlr_foreign_toplevel_handle_v1 *toplevel_handle; struct foreign_toplevel {
struct wl_listener toplevel_handle_request_maximize; struct wlr_foreign_toplevel_handle_v1 *handle;
struct wl_listener toplevel_handle_request_minimize; struct wl_listener maximize;
struct wl_listener toplevel_handle_request_fullscreen; struct wl_listener minimize;
struct wl_listener toplevel_handle_request_activate; struct wl_listener fullscreen;
struct wl_listener toplevel_handle_request_close; struct wl_listener activate;
struct wl_listener close;
struct wl_listener destroy;
} toplevel;
struct wl_listener map; struct wl_listener map;
struct wl_listener unmap; struct wl_listener unmap;

View file

@ -4,71 +4,71 @@
#include "workspaces.h" #include "workspaces.h"
static void static void
handle_toplevel_handle_request_minimize(struct wl_listener *listener, void *data) handle_request_minimize(struct wl_listener *listener, void *data)
{ {
struct view *view = wl_container_of(listener, view, struct view *view = wl_container_of(listener, view, toplevel.minimize);
toplevel_handle_request_minimize);
struct wlr_foreign_toplevel_handle_v1_minimized_event *event = data; struct wlr_foreign_toplevel_handle_v1_minimized_event *event = data;
if (view) { view_minimize(view, event->minimized);
view_minimize(view, event->minimized);
}
} }
static void static void
handle_toplevel_handle_request_maximize(struct wl_listener *listener, void *data) handle_request_maximize(struct wl_listener *listener, void *data)
{ {
struct view *view = wl_container_of(listener, view, struct view *view = wl_container_of(listener, view, toplevel.maximize);
toplevel_handle_request_maximize);
struct wlr_foreign_toplevel_handle_v1_maximized_event *event = data; struct wlr_foreign_toplevel_handle_v1_maximized_event *event = data;
if (view) { view_maximize(view, event->maximized, /*store_natural_geometry*/ true);
view_maximize(view, event->maximized,
/*store_natural_geometry*/ true);
}
} }
static void static void
handle_toplevel_handle_request_fullscreen(struct wl_listener *listener, void *data) handle_request_fullscreen(struct wl_listener *listener, void *data)
{ {
struct view *view = wl_container_of(listener, view, struct view *view = wl_container_of(listener, view, toplevel.fullscreen);
toplevel_handle_request_fullscreen);
struct wlr_foreign_toplevel_handle_v1_fullscreen_event *event = data; struct wlr_foreign_toplevel_handle_v1_fullscreen_event *event = data;
if (view) { view_set_fullscreen(view, event->fullscreen, NULL);
view_set_fullscreen(view, event->fullscreen, NULL);
}
} }
static void static void
handle_toplevel_handle_request_activate(struct wl_listener *listener, void *data) handle_request_activate(struct wl_listener *listener, void *data)
{ {
struct view *view = wl_container_of(listener, view, struct view *view = wl_container_of(listener, view, toplevel.activate);
toplevel_handle_request_activate);
// struct wlr_foreign_toplevel_handle_v1_activated_event *event = data; // struct wlr_foreign_toplevel_handle_v1_activated_event *event = data;
/* In a multi-seat world we would select seat based on event->seat here. */ /* In a multi-seat world we would select seat based on event->seat here. */
if (view) { if (view->workspace != view->server->workspace_current) {
if (view->workspace != view->server->workspace_current) { workspaces_switch_to(view->workspace);
workspaces_switch_to(view->workspace);
}
desktop_focus_and_activate_view(&view->server->seat, view);
desktop_move_to_front(view);
} }
desktop_focus_and_activate_view(&view->server->seat, view);
desktop_move_to_front(view);
} }
static void static void
handle_toplevel_handle_request_close(struct wl_listener *listener, void *data) handle_request_close(struct wl_listener *listener, void *data)
{ {
struct view *view = wl_container_of(listener, view, struct view *view = wl_container_of(listener, view, toplevel.close);
toplevel_handle_request_close); view_close(view);
if (view) { }
view_close(view);
} static void
handle_destroy(struct wl_listener *listener, void *data)
{
struct view *view = wl_container_of(listener, view, toplevel.destroy);
struct foreign_toplevel *toplevel = &view->toplevel;
wl_list_remove(&toplevel->maximize.link);
wl_list_remove(&toplevel->minimize.link);
wl_list_remove(&toplevel->fullscreen.link);
wl_list_remove(&toplevel->activate.link);
wl_list_remove(&toplevel->close.link);
wl_list_remove(&toplevel->destroy.link);
toplevel->handle = NULL;
} }
void void
foreign_toplevel_handle_create(struct view *view) foreign_toplevel_handle_create(struct view *view)
{ {
view->toplevel_handle = wlr_foreign_toplevel_handle_v1_create( struct foreign_toplevel *toplevel = &view->toplevel;
toplevel->handle = wlr_foreign_toplevel_handle_v1_create(
view->server->foreign_toplevel_manager); view->server->foreign_toplevel_manager);
if (!view->toplevel_handle) { if (!toplevel->handle) {
wlr_log(WLR_ERROR, "cannot create foreign toplevel handle for (%s)", wlr_log(WLR_ERROR, "cannot create foreign toplevel handle for (%s)",
view_get_string_prop(view, "title")); view_get_string_prop(view, "title"));
return; return;
@ -80,33 +80,28 @@ foreign_toplevel_handle_create(struct view *view)
view_get_string_prop(view, "title")); view_get_string_prop(view, "title"));
return; return;
} }
wlr_foreign_toplevel_handle_v1_output_enter(view->toplevel_handle, wlr_foreign_toplevel_handle_v1_output_enter(toplevel->handle, wlr_output);
wlr_output);
view->toplevel_handle_request_maximize.notify = toplevel->maximize.notify = handle_request_maximize;
handle_toplevel_handle_request_maximize; wl_signal_add(&toplevel->handle->events.request_maximize,
wl_signal_add(&view->toplevel_handle->events.request_maximize, &toplevel->maximize);
&view->toplevel_handle_request_maximize);
view->toplevel_handle_request_minimize.notify = toplevel->minimize.notify = handle_request_minimize;
handle_toplevel_handle_request_minimize; wl_signal_add(&toplevel->handle->events.request_minimize,
wl_signal_add(&view->toplevel_handle->events.request_minimize, &toplevel->minimize);
&view->toplevel_handle_request_minimize);
view->toplevel_handle_request_fullscreen.notify = toplevel->fullscreen.notify = handle_request_fullscreen;
handle_toplevel_handle_request_fullscreen; wl_signal_add(&toplevel->handle->events.request_fullscreen,
wl_signal_add(&view->toplevel_handle->events.request_fullscreen, &toplevel->fullscreen);
&view->toplevel_handle_request_fullscreen);
view->toplevel_handle_request_activate.notify = toplevel->activate.notify = handle_request_activate;
handle_toplevel_handle_request_activate; wl_signal_add(&toplevel->handle->events.request_activate,
wl_signal_add(&view->toplevel_handle->events.request_activate, &toplevel->activate);
&view->toplevel_handle_request_activate);
view->toplevel_handle_request_close.notify = toplevel->close.notify = handle_request_close;
handle_toplevel_handle_request_close; wl_signal_add(&toplevel->handle->events.request_close,
wl_signal_add(&view->toplevel_handle->events.request_close, &toplevel->close);
&view->toplevel_handle_request_close);
/* TODO: hook up remaining signals (destroy) */ toplevel->destroy.notify = handle_destroy;
wl_signal_add(&toplevel->handle->events.destroy, &toplevel->destroy);
} }

View file

@ -109,9 +109,9 @@ _view_set_activated(struct view *view, bool activated)
if (view->impl->set_activated) { if (view->impl->set_activated) {
view->impl->set_activated(view, activated); view->impl->set_activated(view, activated);
} }
if (view->toplevel_handle) { if (view->toplevel.handle) {
wlr_foreign_toplevel_handle_v1_set_activated( wlr_foreign_toplevel_handle_v1_set_activated(
view->toplevel_handle, activated); view->toplevel.handle, activated);
} }
} }
@ -191,9 +191,9 @@ view_minimize(struct view *view, bool minimized)
if (view->minimized == minimized) { if (view->minimized == minimized) {
return; return;
} }
if (view->toplevel_handle) { if (view->toplevel.handle) {
wlr_foreign_toplevel_handle_v1_set_minimized( wlr_foreign_toplevel_handle_v1_set_minimized(
view->toplevel_handle, minimized); view->toplevel.handle, minimized);
} }
view->minimized = minimized; view->minimized = minimized;
if (minimized) { if (minimized) {
@ -514,9 +514,9 @@ set_maximized(struct view *view, bool maximized)
if (view->impl->maximize) { if (view->impl->maximize) {
view->impl->maximize(view, maximized); view->impl->maximize(view, maximized);
} }
if (view->toplevel_handle) { if (view->toplevel.handle) {
wlr_foreign_toplevel_handle_v1_set_maximized( wlr_foreign_toplevel_handle_v1_set_maximized(
view->toplevel_handle, maximized); view->toplevel.handle, maximized);
} }
view->maximized = maximized; view->maximized = maximized;
} }
@ -693,9 +693,9 @@ view_set_fullscreen(struct view *view, bool fullscreen,
if (view->impl->set_fullscreen) { if (view->impl->set_fullscreen) {
view->impl->set_fullscreen(view, fullscreen); view->impl->set_fullscreen(view, fullscreen);
} }
if (view->toplevel_handle) { if (view->toplevel.handle) {
wlr_foreign_toplevel_handle_v1_set_fullscreen( wlr_foreign_toplevel_handle_v1_set_fullscreen(
view->toplevel_handle, fullscreen); view->toplevel.handle, fullscreen);
} }
if (fullscreen) { if (fullscreen) {
/* /*
@ -759,18 +759,18 @@ view_adjust_for_layout_change(struct view *view)
static void static void
view_output_enter(struct view *view, struct wlr_output *wlr_output) view_output_enter(struct view *view, struct wlr_output *wlr_output)
{ {
if (view->toplevel_handle) { if (view->toplevel.handle) {
wlr_foreign_toplevel_handle_v1_output_enter( wlr_foreign_toplevel_handle_v1_output_enter(
view->toplevel_handle, wlr_output); view->toplevel.handle, wlr_output);
} }
} }
static void static void
view_output_leave(struct view *view, struct wlr_output *wlr_output) view_output_leave(struct view *view, struct wlr_output *wlr_output)
{ {
if (view->toplevel_handle) { if (view->toplevel.handle) {
wlr_foreign_toplevel_handle_v1_output_leave( wlr_foreign_toplevel_handle_v1_output_leave(
view->toplevel_handle, wlr_output); view->toplevel.handle, wlr_output);
} }
} }
@ -990,11 +990,11 @@ view_update_title(struct view *view)
{ {
assert(view); assert(view);
const char *title = view_get_string_prop(view, "title"); const char *title = view_get_string_prop(view, "title");
if (!view->toplevel_handle || !title) { if (!view->toplevel.handle || !title) {
return; return;
} }
ssd_update_title(view->ssd); ssd_update_title(view->ssd);
wlr_foreign_toplevel_handle_v1_set_title(view->toplevel_handle, title); wlr_foreign_toplevel_handle_v1_set_title(view->toplevel.handle, title);
} }
void void
@ -1002,11 +1002,11 @@ view_update_app_id(struct view *view)
{ {
assert(view); assert(view);
const char *app_id = view_get_string_prop(view, "app_id"); const char *app_id = view_get_string_prop(view, "app_id");
if (!view->toplevel_handle || !app_id) { if (!view->toplevel.handle || !app_id) {
return; return;
} }
wlr_foreign_toplevel_handle_v1_set_app_id( wlr_foreign_toplevel_handle_v1_set_app_id(
view->toplevel_handle, app_id); view->toplevel.handle, app_id);
} }
void void
@ -1026,8 +1026,8 @@ view_destroy(struct view *view)
struct server *server = view->server; struct server *server = view->server;
bool need_cursor_update = false; bool need_cursor_update = false;
if (view->toplevel_handle) { if (view->toplevel.handle) {
wlr_foreign_toplevel_handle_v1_destroy(view->toplevel_handle); wlr_foreign_toplevel_handle_v1_destroy(view->toplevel.handle);
} }
if (server->grabbed_view == view) { if (server->grabbed_view == view) {

View file

@ -185,9 +185,9 @@ handle_unmap(struct wl_listener *listener, void *data)
* foreign-toplevel protocol to avoid panels and the like still showing * foreign-toplevel protocol to avoid panels and the like still showing
* them. * them.
*/ */
if (view->toplevel_handle) { if (view->toplevel.handle) {
wlr_foreign_toplevel_handle_v1_destroy(view->toplevel_handle); wlr_foreign_toplevel_handle_v1_destroy(view->toplevel.handle);
view->toplevel_handle = NULL; view->toplevel.handle = NULL;
} }
} }
@ -491,7 +491,7 @@ map(struct view *view)
view->scene_node = &tree->node; view->scene_node = &tree->node;
} }
if (!view->toplevel_handle) { if (!view->toplevel.handle) {
foreign_toplevel_handle_create(view); foreign_toplevel_handle_create(view);
} }