[ksmserver] Fix session restore check
ClosedPublic

Authored by davidedmundson on Oct 31 2018, 5:04 PM.

Details

Summary

The old code checked ksmserver->defaultSession in the Startup
constructor before ksmserver was set up by main.cpp

This refactor moves it to the job, so we'll be checking if we're
restoring or not after ksmserver has been set up by main.cpp as well as
making the initial code path a bit clearer.

Test Plan

Logged in with session restore
Logged in with a default session

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
davidedmundson created this revision.Oct 31 2018, 5:04 PM
Restricted Application added a project: Plasma. · View Herald TranscriptOct 31 2018, 5:04 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson requested review of this revision.Oct 31 2018, 5:04 PM
ahartmetz accepted this revision.Oct 31 2018, 5:11 PM
ahartmetz added a subscriber: ahartmetz.

This fixes session restore for me.

This revision is now accepted and ready to land.Oct 31 2018, 5:11 PM
This revision was automatically updated to reflect the committed changes.