Differential D1020

[User Switcher] Fix session swicthing when screen locking is enabled
ClosedPublic
Actions

Authored by broulik on Feb 24 2016, 2:06 PM.

Details

Reviewers
graesslin
Group Reviewers
Plasma
Commits
R120:7a0096ba99d7: [User Switcher] Fix session switching when automatic screen locking is enabled
R871:7a0096ba99d7: [User Switcher] Fix session switching when automatic screen locking is enabled
Summary

When automatic screen locking is enabled, we want to ensure the screen is fully locked
before we switch sessions or go to the login screen to avoid brief exposure of the desktop
when we return.

This check happens asynchronously and because the User Switcher dialog closes immediately
after issuing the switch request, the SessionsModel is already destroyed and the reply is never
actually processed.

BUG: 356945
FIXED-IN: 5.5.5

Test Plan

I couldn't really test this as my screen locker's ActiveChanged signals is broken for some reason and I never actually get told that the screen got locked (Bug 359680)

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.
broulik updated this revision to Diff 2455.Feb 24 2016, 2:06 PM
broulik retitled this revision from to [User Switcher] Fix session swicthing when screen locking is enabled.
broulik updated this object.
broulik edited the test plan for this revision. (Show Details)
broulik added reviewers: Plasma, graesslin.
broulik set the repository for this revision to R120 Plasma Workspace.
broulik added a project: Plasma.
Restricted Application added a subscriber: plasma-devel. · View Herald TranscriptFeb 24 2016, 2:06 PM
broulik updated this object.Feb 24 2016, 2:07 PM
Closed by commit R120:7a0096ba99d7: [User Switcher] Fix session switching when automatic screen locking is enabled (authored by broulik). · Explain WhyFeb 25 2016, 9:18 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 2467

View Options

components/sessionsprivate/sessionsmodel.h

Loading...
View Options

components/sessionsprivate/sessionsmodel.cpp

Loading...
View Options

lookandfeel/contents/userswitcher/UserSwitcher.qml

Loading...