Don't notify ksplash in our wayland session
AcceptedPublic

Authored by davidedmundson on Sat, Sep 7, 12:01 PM.

Details

Reviewers
zzag
Group Reviewers
KWin
Summary

KSplash doesn't expect a kwin notification on wayland as kwin is started
before ksplash.

Diff Detail

Repository
R108 KWin
Branch
master
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 16228
Build 16246: arc lint + arc unit
davidedmundson created this revision.Sat, Sep 7, 12:01 PM
Restricted Application added a project: KWin. · View Herald TranscriptSat, Sep 7, 12:01 PM
Restricted Application added a subscriber: kwin. · View Herald Transcript
davidedmundson requested review of this revision.Sat, Sep 7, 12:01 PM
zzag added a subscriber: zzag.Sat, Sep 7, 12:35 PM

notifyKSplash method needs to be moved to ApplicationX11 class.

+1
in splashapp.cpp

if (KWindowSystem::isPlatformWayland()) {
    setStage(QStringLiteral("wm"));
}

I don't think it's needed to move it. But done anyway.

zzag accepted this revision.Sat, Sep 7, 8:49 PM
This revision is now accepted and ready to land.Sat, Sep 7, 8:49 PM