Fix listener_sessionActive to handle an event correctly when the session get inactivated

[harenayo]

Describe your PR, what does it fix/add?

I found that Hyprland handles m_sWLRSession->events.active signal by listener_sessionActive.

Hyprland/src/Compositor.cpp

Line 285 in b7f42a1

addWLSignal(&m_sWLRSession->events.active, &Events::listen_sessionActive, m_sWLRSession, "Session");

Hyprland/src/events/Misc.cpp

Lines 32 to 43 in b7f42a1

	void Events::listener_sessionActive(wl_listener* listener, void* data) {
	Debug::log(LOG, "Session got activated!");
	g_pCompositor->m_bSessionActive = true;
	for (auto& m : g_pCompositor->m_vMonitors) {
	g_pCompositor->scheduleFrameForMonitor(m.get());
	g_pHyprRenderer->applyMonitorRule(m.get(), &m->activeMonitorRule, true);
	}
	g_pConfigManager->m_bWantsMonitorReload = true;
	}

listener_sessionActive always does an activation process.

But I think m_sWLRSession->events.active is emitted whether the session get activated or inactivated.

https://github.com/hyprwm/wlroots-hyprland/blob/422207dbcf0949e28042403edab539159282885e/backend/session/session.c#L24-L35

That's why I made the following changes:

• Check m_sWLRSession->active in listener_sessionActive.
• Move the code for switching VTs into listener_sessionActive.

Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)

I'm not sure that the second change (the code for VTs) works correctly.

Is it ready for merging, or does it need work?

It is ready.

[vaxerski]

thanks!