[kcmkwin] Properly load effects
AcceptedPublic

Authored by zzag on Mon, Feb 4, 4:09 PM.

Details

Reviewers
davidedmundson
Group Reviewers
KWin
Summary

EffectsModel has to communicate with KWin in order to receive the
actual value of SupportedRole. So, in theory the model should notify
about loaded effects after receiving response from KWin, but that's
not the case.

Test Plan

Desktop Effects KCM no longer flashes when resetting changes.

Diff Detail

Repository
R108 KWin
Branch
split-desktop-effects-kcm
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 7888
Build 7906: arc lint + arc unit
zzag created this revision.Mon, Feb 4, 4:09 PM
Restricted Application added a project: KWin. · View Herald TranscriptMon, Feb 4, 4:09 PM
Restricted Application added a subscriber: kwin. · View Herald Transcript
zzag requested review of this revision.Mon, Feb 4, 4:09 PM
zzag edited the summary of this revision. (Show Details)Mon, Feb 4, 4:17 PM
zzag updated this revision to Diff 50861.Mon, Feb 4, 4:33 PM

edit comment

zzag updated this revision to Diff 50893.Mon, Feb 4, 9:17 PM

Rebase.

davidedmundson accepted this revision.Tue, Feb 5, 9:33 AM
This revision is now accepted and ready to land.Tue, Feb 5, 9:33 AM