Differential D3239

[kstyle] Delay init of Wayland setup till next event cycle
ClosedPublic
Actions

Authored by graesslin on Nov 3 2016, 9:55 AM.

Details

Reviewers
hpereiradacosta
broulik
Group Reviewers
Plasma
Commits
R31:088c7672981b: [kstyle] Delay init of Wayland setup till next event cycle
R31:adcd98aa1f2a: Revert "[kstyle] Implement application unpolish to delete ShadowHelper"
Summary

This is a change mostly for KWin (the Wayland compositor). The internal
Wayland connection is created after the QStyle is created in the case of
KWin. But both are created before the event dispatcher is run for the
first time. So delaying by one cycle makes it work for KWin and also all
other Wayland applications.

BUG: 372001
FIXED-IN: 5.8.4

Diff Detail

Repository
R31 Breeze
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
graesslin updated this revision to Diff 7848.Nov 3 2016, 9:55 AM
graesslin retitled this revision from to [kstyle] Delay init of Wayland setup till next event cycle.
graesslin updated this object.
graesslin edited the test plan for this revision. (Show Details)
graesslin added reviewers: Plasma, broulik, hpereiradacosta.
Restricted Application added a project: Plasma. · View Herald TranscriptNov 3 2016, 9:55 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
hpereiradacosta accepted this revision.Nov 3 2016, 9:56 AM
hpereiradacosta edited edge metadata.

No objection, you're the expert !
Thanks !

This revision is now accepted and ready to land.Nov 3 2016, 9:56 AM
Closed by commit R31:088c7672981b: [kstyle] Delay init of Wayland setup till next event cycle (authored by graesslin). · Explain WhyNov 7 2016, 11:00 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 7964

View Options

kstyle/breezeshadowhelper.h

Loading...
View Options

kstyle/breezeshadowhelper.cpp

Loading...