Following MR !361, I wanted to make some progress to stop using ScreenBrightnessController's legacy API methods so I was going to move the "action" part from the (now deprecated) BrightnessControl action into the new ScreenBrightnessControl action. But then realized that conflating an always-available D-Bus API and a conditionally-enabled brightness change action really shouldn't be co-located in a single action class.
So this MR introduces a stand-alone class for exposing the D-Bus API, called ScreenBrightnessAgent (after PolicyAgent) with org.kde.ScreenBrightness as a better, action-independent bus name. The recently introduced ScreenBrightnessControl action is still present and handles the aforementioned profile-based brightness change action as well as keyboard shortcuts. BrightnessControl is left with just the deprecated D-Bus API.
The MR is segmented into three commits, one for each moved piece of functionality:
daemon: Move brightness key handling to ScreenBrightnessControl action
The BrightnessControl action is deprecated and we want to moveaway from it.
daemon: Move the new ScreenBrightness D-Bus API out of actions
Being an action ties the interface to a particular path andinterface name. It also ties the lifetime of the D-Bus API tothe isSupported() method, which in turn complicates moving theactual profile-based brightness change action away from thedeprecated BrightnessControl class.
Instead, let's elevate the new interface to the same level asPolicyAgent, initializing it unconditionally in PowerDevilApp andgiving it a nice D-Bus bus name / path / interface at"org.kde.ScreenBrightness".
daemon: Port profile-based brightness changes away from legacy API
The relevant code is moved from the deprecated BrightnessControlaction to the new ScreenBrightnessControl action, where itsisSupported() result is now independent of whether the (deprecated)BrightnessControl D-Bus interface should be exposed or not.
The new per-display implementation of profile-based brightnesschanges will preserve legacy API behavior and limit changesto internal displays if any are available. See code comment fora more detailed explanation.
BrightnessControl is now only responsible for its D-Bus API,and can be removed whenever we decide it's not worth keeping.
Test plan
- Test that per-display brightness changes (applet slider & scrolling, brightness keys) still work as intended.
- Check that
org.kde.ScreenBrightnessnow exists as a separate session bus service in e.g. qdbus. - Open "Power Management" in System Settings, enable "Change screen brightness" also for "On Battery" and apply. Then get on battery from AC power (screen should reduce, but not increase brightness).
- This action is still annoying as heck when there isn't another profile switching back to another pre-defined brightness percentage. We may be able to fix this later by reinterpreting the setting as dimming with brightness value limit, or such. (Another thing to consider for !360.)
Bugs fixed
No reported ones, but this protects 6.2 against expanding the scope of BUG: 484663 to external monitors as well.