diff --git a/backend/x11/backend.c b/backend/x11/backend.c index 0983bad0a..bbe46d2f9 100644 --- a/backend/x11/backend.c +++ b/backend/x11/backend.c @@ -110,14 +110,70 @@ static void handle_x11_event(struct wlr_x11_backend *x11, } break; } + case XCB_UNMAP_NOTIFY: { + xcb_unmap_notify_event_t *ev = (xcb_unmap_notify_event_t *)event; + struct wlr_x11_output *output = + get_x11_output_from_window_id(x11, ev->window); + if (output != NULL) { + wlr_log(WLR_DEBUG, "Window unmapped, stopping updates"); + output->mapped = false; + } + break; + } + case XCB_MAP_NOTIFY: { + xcb_unmap_notify_event_t *ev = (xcb_unmap_notify_event_t *)event; + struct wlr_x11_output *output = + get_x11_output_from_window_id(x11, ev->window); + if (output != NULL) { + wlr_log(WLR_DEBUG, "Window mapped%s", !output->hidden ? ", resuming updates" : ""); + output->mapped = true; + if (!output->hidden) { + wlr_output_send_frame(&output->wlr_output); + } + } + break; + } + case XCB_PROPERTY_NOTIFY: { + xcb_property_notify_event_t *ev = (xcb_property_notify_event_t *)event; + struct wlr_x11_output *output = + get_x11_output_from_window_id(x11, ev->window); + if (output == NULL) { + break; + } + if (ev->atom == x11->atoms.net_wm_state) { + xcb_get_property_cookie_t cookie = + xcb_get_property(x11->xcb, false, ev->window, + ev->atom, XCB_ATOM_ATOM, 0, 32); + xcb_get_property_reply_t *reply = + xcb_get_property_reply(x11->xcb, cookie, NULL); + if (reply->type != XCB_ATOM_ATOM) { + free(reply); + break; + } + xcb_atom_t *atoms = xcb_get_property_value(reply); + bool was_hidden = output->hidden; + output->hidden = false; + for (int i = 0; i < xcb_get_property_value_length(reply); i++) { + if (atoms[i] == x11->atoms.net_wm_state_hidden) { + wlr_log(WLR_DEBUG, "Window hidden, stopping updates"); + output->hidden = true; + } + } + free(reply); + if (was_hidden && !output->hidden) { + wlr_log(WLR_DEBUG, "Window no longer hidden%s", output->mapped ? ", resuming updates" : ""); + if (output->mapped) { + wlr_output_send_frame(&output->wlr_output); + } + } + } + break; + } case 0: { xcb_value_error_t *ev = (xcb_value_error_t *)event; handle_x11_error(x11, ev); break; } - case XCB_UNMAP_NOTIFY: - case XCB_MAP_NOTIFY: - break; default: handle_x11_unknown_event(x11, event); break; @@ -414,6 +470,8 @@ struct wlr_backend *wlr_x11_backend_create(struct wl_event_loop *loop, } atom[] = { { .name = "WM_PROTOCOLS", .atom = &x11->atoms.wm_protocols }, { .name = "WM_DELETE_WINDOW", .atom = &x11->atoms.wm_delete_window }, + { .name = "_NET_WM_STATE", .atom = &x11->atoms.net_wm_state }, + { .name = "_NET_WM_STATE_HIDDEN", .atom = &x11->atoms.net_wm_state_hidden }, { .name = "_NET_WM_NAME", .atom = &x11->atoms.net_wm_name }, { .name = "UTF8_STRING", .atom = &x11->atoms.utf8_string }, { .name = "_VARIABLE_REFRESH", .atom = &x11->atoms.variable_refresh }, diff --git a/backend/x11/output.c b/backend/x11/output.c index 0b63a7088..99b8e6f68 100644 --- a/backend/x11/output.c +++ b/backend/x11/output.c @@ -618,7 +618,8 @@ struct wlr_output *wlr_x11_output_create(struct wlr_backend *backend) { XCB_CW_COLORMAP | XCB_CW_CURSOR; uint32_t values[] = { 0, - XCB_EVENT_MASK_EXPOSURE | XCB_EVENT_MASK_STRUCTURE_NOTIFY, + XCB_EVENT_MASK_EXPOSURE | XCB_EVENT_MASK_STRUCTURE_NOTIFY | + XCB_EVENT_MASK_PROPERTY_CHANGE, x11->colormap, x11->transparent_cursor, }; @@ -666,6 +667,8 @@ struct wlr_output *wlr_x11_output_create(struct wlr_backend *backend) { xcb_flush(x11->xcb); + output->mapped = true; + wl_list_insert(&x11->outputs, &output->link); wlr_pointer_init(&output->pointer, &x11_pointer_impl, "x11-pointer"); @@ -786,7 +789,8 @@ void handle_x11_present_event(struct wlr_x11_backend *x11, timespec_from_nsec(&present_event.when, complete_notify->ust * 1000); wlr_output_send_present(&output->wlr_output, &present_event); - wlr_output_send_frame(&output->wlr_output); + if (output->mapped && !output->hidden) + wlr_output_send_frame(&output->wlr_output); break; default: wlr_log(WLR_DEBUG, "Unhandled Present event %"PRIu16, event->event_type); diff --git a/include/backend/x11.h b/include/backend/x11.h index fcea4d257..75cab0f78 100644 --- a/include/backend/x11.h +++ b/include/backend/x11.h @@ -47,6 +47,8 @@ struct wlr_x11_output { pixman_region32_t exposed; uint64_t last_msc; + bool mapped; + bool hidden; struct { struct wlr_swapchain *swapchain; @@ -94,6 +96,8 @@ struct wlr_x11_backend { xcb_atom_t wm_protocols; xcb_atom_t wm_delete_window; xcb_atom_t net_wm_name; + xcb_atom_t net_wm_state; + xcb_atom_t net_wm_state_hidden; xcb_atom_t utf8_string; xcb_atom_t variable_refresh; } atoms;