diff --git a/CMakeLists.txt b/CMakeLists.txt index e134f9c6..aa3219d9 100755 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -265,6 +265,7 @@ protocolNew("staging/tearing-control/tearing-control-v1.xml" "tearing-control-v1 protocolNew("staging/fractional-scale/fractional-scale-v1.xml" "fractional-scale-v1" false) protocolNew("unstable/xdg-output/xdg-output-unstable-v1.xml" "xdg-output-unstable-v1" false) protocolNew("staging/cursor-shape/cursor-shape-v1.xml" "cursor-shape-v1" false) +protocolNew("unstable/idle-inhibit/idle-inhibit-unstable-v1.xml" "idle-inhibit-unstable-v1" false) # tools add_subdirectory(hyprctl) diff --git a/protocols/meson.build b/protocols/meson.build index 009643f9..dd2a2358 100644 --- a/protocols/meson.build +++ b/protocols/meson.build @@ -43,6 +43,7 @@ new_protocols = [ [wl_protocol_dir, 'staging/fractional-scale/fractional-scale-v1.xml'], [wl_protocol_dir, 'unstable/xdg-output/xdg-output-unstable-v1.xml'], [wl_protocol_dir, 'staging/cursor-shape/cursor-shape-v1.xml'], + [wl_protocol_dir, 'unstable/idle-inhibit/idle-inhibit-unstable-v1.xml'], ] wl_protos_src = [] diff --git a/src/Compositor.cpp b/src/Compositor.cpp index 824d7335..f9ecb18e 100644 --- a/src/Compositor.cpp +++ b/src/Compositor.cpp @@ -235,8 +235,6 @@ void CCompositor::initServer() { m_sWLRForeignRegistry = wlr_xdg_foreign_registry_create(m_sWLDisplay); - m_sWLRIdleInhibitMgr = wlr_idle_inhibit_v1_create(m_sWLDisplay); - wlr_xdg_foreign_v1_create(m_sWLDisplay, m_sWLRForeignRegistry); wlr_xdg_foreign_v2_create(m_sWLDisplay, m_sWLRForeignRegistry); @@ -300,7 +298,6 @@ void CCompositor::initAllSignals() { addWLSignal(&m_sWLRVirtPtrMgr->events.new_virtual_pointer, &Events::listen_newVirtPtr, m_sWLRVirtPtrMgr, "VirtPtrMgr"); addWLSignal(&m_sWLRVKeyboardMgr->events.new_virtual_keyboard, &Events::listen_newVirtualKeyboard, m_sWLRVKeyboardMgr, "VKeyboardMgr"); addWLSignal(&m_sWLRRenderer->events.destroy, &Events::listen_RendererDestroy, m_sWLRRenderer, "WLRRenderer"); - addWLSignal(&m_sWLRIdleInhibitMgr->events.new_inhibitor, &Events::listen_newIdleInhibitor, m_sWLRIdleInhibitMgr, "WLRIdleInhibitMgr"); addWLSignal(&m_sWLROutputPowerMgr->events.set_mode, &Events::listen_powerMgrSetMode, m_sWLROutputPowerMgr, "PowerMgr"); addWLSignal(&m_sWLRIMEMgr->events.input_method, &Events::listen_newIME, m_sWLRIMEMgr, "IMEMgr"); addWLSignal(&m_sWLRTextInputMgr->events.text_input, &Events::listen_newTextInput, m_sWLRTextInputMgr, "TextInputMgr"); @@ -352,7 +349,6 @@ void CCompositor::removeAllSignals() { removeWLSignal(&Events::listen_newVirtPtr); removeWLSignal(&Events::listen_newVirtualKeyboard); removeWLSignal(&Events::listen_RendererDestroy); - removeWLSignal(&Events::listen_newIdleInhibitor); removeWLSignal(&Events::listen_powerMgrSetMode); removeWLSignal(&Events::listen_newIME); removeWLSignal(&Events::listen_newTextInput); diff --git a/src/Compositor.hpp b/src/Compositor.hpp index 8cca11c2..96fee44e 100644 --- a/src/Compositor.hpp +++ b/src/Compositor.hpp @@ -71,7 +71,6 @@ class CCompositor { wlr_foreign_toplevel_manager_v1* m_sWLRToplevelMgr; wlr_tablet_manager_v2* m_sWLRTabletManager; wlr_xdg_foreign_registry* m_sWLRForeignRegistry; - wlr_idle_inhibit_manager_v1* m_sWLRIdleInhibitMgr; wlr_pointer_gestures_v1* m_sWLRPointerGestures; wlr_output_power_manager_v1* m_sWLROutputPowerMgr; wlr_input_method_manager_v2* m_sWLRIMEMgr; diff --git a/src/events/Events.hpp b/src/events/Events.hpp index 445b884c..4336bfd5 100644 --- a/src/events/Events.hpp +++ b/src/events/Events.hpp @@ -105,8 +105,6 @@ namespace Events { // Renderer destroy LISTENER(RendererDestroy); - LISTENER(newIdleInhibitor); - // session LISTENER(sessionActive); diff --git a/src/helpers/WLClasses.hpp b/src/helpers/WLClasses.hpp index 89af21c3..d6e24d53 100644 --- a/src/helpers/WLClasses.hpp +++ b/src/helpers/WLClasses.hpp @@ -269,18 +269,6 @@ struct STabletPad { } }; -struct SIdleInhibitor { - wlr_idle_inhibitor_v1* pWlrInhibitor = nullptr; - CWindow* pWindow = nullptr; - std::shared_ptr onWindowDestroy; - - DYNLISTENER(Destroy); - - bool operator==(const SIdleInhibitor& b) const { - return pWlrInhibitor == b.pWlrInhibitor; - } -}; - struct SSwipeGesture { PHLWORKSPACE pWorkspaceBegin = nullptr; diff --git a/src/includes.hpp b/src/includes.hpp index 23e96425..d7cb7ce8 100644 --- a/src/includes.hpp +++ b/src/includes.hpp @@ -72,7 +72,6 @@ extern "C" { #include #include #include -#include #include #include #include diff --git a/src/managers/ProtocolManager.cpp b/src/managers/ProtocolManager.cpp index 67b3bdeb..8bf7a0d9 100644 --- a/src/managers/ProtocolManager.cpp +++ b/src/managers/ProtocolManager.cpp @@ -4,11 +4,13 @@ #include "../protocols/FractionalScale.hpp" #include "../protocols/XDGOutput.hpp" #include "../protocols/CursorShape.hpp" +#include "../protocols/IdleInhibit.hpp" #include "tearing-control-v1.hpp" #include "fractional-scale-v1.hpp" #include "xdg-output-unstable-v1.hpp" #include "cursor-shape-v1.hpp" +#include "idle-inhibit-unstable-v1.hpp" CProtocolManager::CProtocolManager() { @@ -16,6 +18,7 @@ CProtocolManager::CProtocolManager() { PROTO::fractional = std::make_unique(&wp_fractional_scale_manager_v1_interface, 1, "FractionalScale"); PROTO::xdgOutput = std::make_unique(&zxdg_output_manager_v1_interface, 3, "XDGOutput"); PROTO::cursorShape = std::make_unique(&wp_cursor_shape_manager_v1_interface, 1, "CursorShape"); + PROTO::idleInhibit = std::make_unique(&zwp_idle_inhibit_manager_v1_interface, 1, "IdleInhibit"); // Old protocol implementations. // TODO: rewrite them to use hyprwayland-scanner. diff --git a/src/managers/input/IdleInhibitor.cpp b/src/managers/input/IdleInhibitor.cpp index 72f60702..21f1652f 100644 --- a/src/managers/input/IdleInhibitor.cpp +++ b/src/managers/input/IdleInhibitor.cpp @@ -1,61 +1,39 @@ #include "InputManager.hpp" #include "../../Compositor.hpp" +#include "../../protocols/IdleInhibit.hpp" -void Events::listener_newIdleInhibitor(wl_listener* listener, void* data) { - const auto WLRIDLEINHIBITOR = (wlr_idle_inhibitor_v1*)data; +void CInputManager::newIdleInhibitor(std::any inhibitor) { + const auto PINHIBIT = m_vIdleInhibitors.emplace_back(std::make_unique()).get(); + PINHIBIT->inhibitor = std::any_cast>(inhibitor); - if (!WLRIDLEINHIBITOR) + Debug::log(LOG, "New idle inhibitor registered for surface {:x}", (uintptr_t)PINHIBIT->inhibitor->surface); + + PINHIBIT->inhibitor->listeners.destroy = PINHIBIT->inhibitor->resource.lock()->events.destroy.registerListener( + [this, PINHIBIT](std::any data) { std::erase_if(m_vIdleInhibitors, [PINHIBIT](const auto& other) { return other.get() == PINHIBIT; }); }); + + const auto PWINDOW = g_pCompositor->getWindowFromSurface(PINHIBIT->inhibitor->surface); + + if (!PWINDOW) { + Debug::log(WARN, "Inhibitor is for no window?"); return; + } - g_pInputManager->newIdleInhibitor(WLRIDLEINHIBITOR); -} - -static void destroyInhibitor(SIdleInhibitor* inhibitor) { - g_pHookSystem->unhook(inhibitor->onWindowDestroy); - - g_pInputManager->m_lIdleInhibitors.remove(*inhibitor); - - Debug::log(LOG, "Destroyed an idleinhibitor"); - - g_pInputManager->recheckIdleInhibitorStatus(); -} - -void CInputManager::newIdleInhibitor(wlr_idle_inhibitor_v1* pInhibitor) { - const auto PINHIBIT = &m_lIdleInhibitors.emplace_back(); - - Debug::log(LOG, "New idle inhibitor registered"); - - PINHIBIT->pWlrInhibitor = pInhibitor; - - PINHIBIT->onWindowDestroy = g_pHookSystem->hookDynamic("closeWindow", [PINHIBIT](void* self, SCallbackInfo& info, std::any data) { - if (PINHIBIT->pWindow == std::any_cast(data)) - destroyInhibitor(PINHIBIT); + PINHIBIT->pWindow = PWINDOW; + PINHIBIT->windowDestroyListener = PWINDOW->events.destroy.registerListener([PINHIBIT](std::any data) { + Debug::log(WARN, "Inhibitor got its window destroyed before its inhibitor resource."); + PINHIBIT->pWindow = nullptr; }); - PINHIBIT->hyprListener_Destroy.initCallback( - &pInhibitor->events.destroy, - [](void* owner, void* data) { - const auto PINH = (SIdleInhibitor*)owner; - - destroyInhibitor(PINH); - }, - PINHIBIT, "IdleInhibitor"); - - PINHIBIT->pWindow = g_pCompositor->getWindowFromSurface(pInhibitor->surface); - - if (PINHIBIT->pWindow) - Debug::log(LOG, "IdleInhibitor got window {}", PINHIBIT->pWindow); - recheckIdleInhibitorStatus(); } void CInputManager::recheckIdleInhibitorStatus() { - for (auto& ii : m_lIdleInhibitors) { - if (!ii.pWindow) { + for (auto& ii : m_vIdleInhibitors) { + if (!ii->pWindow) { g_pCompositor->setIdleActivityInhibit(false); return; - } else if (g_pHyprRenderer->shouldRenderWindow(ii.pWindow)) { + } else if (g_pHyprRenderer->shouldRenderWindow(ii->pWindow)) { g_pCompositor->setIdleActivityInhibit(false); return; } diff --git a/src/managers/input/InputManager.cpp b/src/managers/input/InputManager.cpp index bf784af1..114a6c83 100644 --- a/src/managers/input/InputManager.cpp +++ b/src/managers/input/InputManager.cpp @@ -5,6 +5,7 @@ #include "../../config/ConfigValue.hpp" #include "../../desktop/Window.hpp" #include "../../protocols/CursorShape.hpp" +#include "../../protocols/IdleInhibit.hpp" CInputManager::CInputManager() { m_sListeners.setCursorShape = PROTO::cursorShape->events.setShape.registerListener([this](std::any data) { @@ -29,6 +30,8 @@ CInputManager::CInputManager() { m_sCursorSurfaceInfo.inUse = true; g_pHyprRenderer->setCursorFromName(m_sCursorSurfaceInfo.name); }); + + m_sListeners.newIdleInhibitor = PROTO::idleInhibit->events.newIdleInhibitor.registerListener([this](std::any data) { this->newIdleInhibitor(data); }); } CInputManager::~CInputManager() { @@ -38,7 +41,7 @@ CInputManager::~CInputManager() { m_lTablets.clear(); m_lTabletTools.clear(); m_lTabletPads.clear(); - m_lIdleInhibitors.clear(); + m_vIdleInhibitors.clear(); m_lTouchDevices.clear(); m_lSwitches.clear(); } diff --git a/src/managers/input/InputManager.hpp b/src/managers/input/InputManager.hpp index dd8f5c63..47773fff 100644 --- a/src/managers/input/InputManager.hpp +++ b/src/managers/input/InputManager.hpp @@ -2,6 +2,7 @@ #include "../../defines.hpp" #include +#include #include "../../helpers/WLClasses.hpp" #include "../../helpers/Timer.hpp" #include "InputMethodRelay.hpp" @@ -9,6 +10,7 @@ class CConstraint; class CWindow; +class CIdleInhibitor; enum eClickBehaviorMode { CLICKMODE_DEFAULT = 0, @@ -132,9 +134,6 @@ class CInputManager { std::list m_lTabletTools; std::list m_lTabletPads; - // idle inhibitors - std::list m_lIdleInhibitors; - // Touch devices std::list m_lTouchDevices; @@ -150,7 +149,7 @@ class CInputManager { void newTabletTool(wlr_input_device*); void newTabletPad(wlr_input_device*); void focusTablet(STablet*, wlr_tablet_tool*, bool motion = false); - void newIdleInhibitor(wlr_idle_inhibitor_v1*); + void newIdleInhibitor(std::any); void recheckIdleInhibitorStatus(); void onSwipeBegin(wlr_pointer_swipe_begin_event*); @@ -201,6 +200,7 @@ class CInputManager { // Listeners struct { CHyprSignalListener setCursorShape; + CHyprSignalListener newIdleInhibitor; } m_sListeners; bool m_bCursorImageOverridden = false; @@ -237,6 +237,14 @@ class CInputManager { // for releasing mouse buttons std::list m_lCurrentlyHeldButtons; + // idle inhibitors + struct SIdleInhibitor { + std::shared_ptr inhibitor; + CWindow* pWindow = nullptr; + CHyprSignalListener windowDestroyListener; + }; + std::vector> m_vIdleInhibitors; + // swipe void beginWorkspaceSwipe(); void updateWorkspaceSwipe(double); diff --git a/src/protocols/IdleInhibit.cpp b/src/protocols/IdleInhibit.cpp new file mode 100644 index 00000000..0b90e284 --- /dev/null +++ b/src/protocols/IdleInhibit.cpp @@ -0,0 +1,48 @@ +#include "IdleInhibit.hpp" + +CIdleInhibitor::CIdleInhibitor(SP resource_, wlr_surface* surf_) : resource(resource_), surface(surf_) { + ; +} + +CIdleInhibitorResource::CIdleInhibitorResource(SP resource_, wlr_surface* surface_) : resource(resource_), surface(surface_) { + hyprListener_surfaceDestroy.initCallback( + &surface->events.destroy, [this](void* owner, void* data) { PROTO::idleInhibit->removeInhibitor(this); }, this, "CIdleInhibitorResource"); + + resource->setOnDestroy([this](CZwpIdleInhibitorV1* p) { PROTO::idleInhibit->removeInhibitor(this); }); + resource->setDestroy([this](CZwpIdleInhibitorV1* p) { PROTO::idleInhibit->removeInhibitor(this); }); +} + +CIdleInhibitorResource::~CIdleInhibitorResource() { + hyprListener_surfaceDestroy.removeCallback(); + events.destroy.emit(); +} + +CIdleInhibitProtocol::CIdleInhibitProtocol(const wl_interface* iface, const int& ver, const std::string& name) : IWaylandProtocol(iface, ver, name) { + ; +} + +void CIdleInhibitProtocol::onManagerResourceDestroy(wl_resource* res) { + std::erase_if(m_vManagers, [res](const auto& other) { return other->resource() == res; }); +} + +void CIdleInhibitProtocol::bindManager(wl_client* client, void* data, uint32_t ver, uint32_t id) { + const auto RESOURCE = m_vManagers.emplace_back(std::make_unique(client, ver, id)).get(); + RESOURCE->setOnDestroy([this](CZwpIdleInhibitManagerV1* p) { this->onManagerResourceDestroy(p->resource()); }); + + RESOURCE->setDestroy([this](CZwpIdleInhibitManagerV1* pMgr) { this->onManagerResourceDestroy(pMgr->resource()); }); + RESOURCE->setCreateInhibitor( + [this](CZwpIdleInhibitManagerV1* pMgr, uint32_t id, wl_resource* surface) { this->onCreateInhibitor(pMgr, id, wlr_surface_from_resource(surface)); }); +} + +void CIdleInhibitProtocol::removeInhibitor(CIdleInhibitorResource* resource) { + std::erase_if(m_vInhibitors, [resource](const auto& el) { return el.get() == resource; }); +} + +void CIdleInhibitProtocol::onCreateInhibitor(CZwpIdleInhibitManagerV1* pMgr, uint32_t id, wlr_surface* surface) { + const auto CLIENT = wl_resource_get_client(pMgr->resource()); + const auto RESOURCE = + m_vInhibitors.emplace_back(std::make_shared(std::make_shared(CLIENT, wl_resource_get_version(pMgr->resource()), id), surface)); + + RESOURCE->inhibitor = std::make_shared(RESOURCE, surface); + events.newIdleInhibitor.emit(RESOURCE->inhibitor); +} \ No newline at end of file diff --git a/src/protocols/IdleInhibit.hpp b/src/protocols/IdleInhibit.hpp new file mode 100644 index 00000000..3283333f --- /dev/null +++ b/src/protocols/IdleInhibit.hpp @@ -0,0 +1,63 @@ +#pragma once + +#include +#include +#include "WaylandProtocol.hpp" +#include "idle-inhibit-unstable-v1.hpp" +#include "../helpers/signal/Signal.hpp" + +class CIdleInhibitorResource; + +class CIdleInhibitor { + public: + CIdleInhibitor(SP resource_, wlr_surface* surf_); + + struct { + CHyprSignalListener destroy; + } listeners; + + WP resource; + wlr_surface* surface = nullptr; +}; + +class CIdleInhibitorResource { + public: + CIdleInhibitorResource(SP resource_, wlr_surface* surface_); + ~CIdleInhibitorResource(); + + SP inhibitor; + + struct { + CSignal destroy; + } events; + + private: + SP resource; + wlr_surface* surface = nullptr; + + DYNLISTENER(surfaceDestroy); +}; + +class CIdleInhibitProtocol : public IWaylandProtocol { + public: + CIdleInhibitProtocol(const wl_interface* iface, const int& ver, const std::string& name); + + virtual void bindManager(wl_client* client, void* data, uint32_t ver, uint32_t id); + + void onManagerResourceDestroy(wl_resource* res); + void onCreateInhibitor(CZwpIdleInhibitManagerV1* pMgr, uint32_t id, wlr_surface* surface); + + void removeInhibitor(CIdleInhibitorResource*); + + struct { + CSignal newIdleInhibitor; // data: SP + } events; + + private: + std::vector> m_vManagers; + std::vector> m_vInhibitors; +}; + +namespace PROTO { + inline UP idleInhibit; +} \ No newline at end of file diff --git a/src/protocols/WaylandProtocol.hpp b/src/protocols/WaylandProtocol.hpp index 588a1420..d23a0e48 100644 --- a/src/protocols/WaylandProtocol.hpp +++ b/src/protocols/WaylandProtocol.hpp @@ -11,6 +11,7 @@ #define SP std::shared_ptr #define UP std::unique_ptr +#define WP std::weak_ptr #define PROTO NProtocols