This is just clutter, the constructor is called anyway.

Summary:
Configuration change requests by clients while the Wayland server is
processing a change lead to a failing assert in blockSignals().

Instead in such a situation remember the latest configuration change request
and apply it after the current change has been processed.

Test Plan:
Auto test exposing the problem. Also manually with my output color correction
code.

Reviewers: #plasma, davidedmundson

Reviewed By: #plasma, davidedmundson

Subscribers: apol, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D12517

GIT_SILENT

GIT_SILENT

GIT_SILENT

GIT_SILENT

This patch allows the rotation of an output to be set from kscreen-doctor.

The syntax is
kscreen-doctor output.HDMI-2.rotation.left

Nested classes inherit the visibility, so need to explicitly opt out again

Since Qt 5.6.3/5.7.1 qdbusxml2cpp favours as name for signals argument
annotations the name "org.qtproject.QtDBus.QtTypeName.Out*", matching
the "out" direction nature of the arguments.
".In*" is still supported for backward compat, but with a warning.

official Plasma 5.12 deps

GIT_SILENT

Summary:
We can share the same "guessing" code with the XRandR backend here in order to
detect what type the output is (depending on having some of the well-known
connection types like HDMI, DP, etc. in its name).

The main benefit of this change is that we can know detect built-in laptop
screens on Wayland.

Reviewers: sebas, davidedmundson

Reviewed By: sebas

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D9908

Summary:
This prevents the code from unnecesarilly emitting configChanged() if there
was in fact no change. This could lead to an endless loop when a client applies
current config and despite there not being no changes the backend emits
configChanged() to which the client reacts by applying the current config again
etc.

Reviewers: sebas, davidedmundson

Reviewed By: sebas

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D9906

Summary:
The 'name' argument is in most cases empty by which the caller indicates that
they want the default backend. As such repeated calls to loadBackendInProcess(QString())
cause to instantiate a new backend every time, because the name check in the
condition is never true.

Reviewers: sebas, davidedmundson

Reviewed By: davidedmundson

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D9905

Summary:
Sharing a single config leads to undesired side-effects. The GetConfigOperation
is assumed to always return a new copy of the current config that is not changed
unless registered to ConfigMonitor. When sharing a single copy this assumption is
broken which leads to misbehaviour in KDED.

This change makes GetConfigOperation always return a new copy of the config that
caller can register to ConfigMonitor if they want to, otherwise it remains
unchanged until released.

Reviewers: sebas, davidedmundson

Reviewed By: sebas

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D9904