https://bugs.kde.org/show_bug.cgi?id=395712

David Edmundson <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://commits.kde.org/kwi |https://commits.kde.org/kwi
                   |n/2693e288c57a66facbe6b9932 |n/835455939f4955baf88e2343c
                   |ce8628a53afb246             |50b1df0f6e8cf5e

--- Comment #5 from David Edmundson <[email protected]> ---
Git commit 835455939f4955baf88e2343c50b1df0f6e8cf5e by David Edmundson.
Committed on 31/03/2019 at 13:51.
Pushed by davidedmundson into branch 'Plasma/5.12'.

Avoid potential assert in SM saving

Summary:
Sesison Manager stores all relevant clients. There's an assert if the
window type is outside of the standard client window types. It assumed
that all windows outside this would be Unmanaged windows rather than
Client objects, something probably true but not something enforced.

This particular crash was probably cased as we have a new window type in
Plasma OSD, which does not set BypassWindowManager in Qt window flags.

Test Plan:
Set to restore session
Logged out and back in
Saw some windows

Set to restore manually saved session
Hit save
No crash

Reviewers: #kwin, graesslin

Reviewed By: #kwin, graesslin

Subscribers: kwin

Tags: #kwin

Differential Revision: https://phabricator.kde.org/D13715

M  +8    -0    sm.cpp

https://commits.kde.org/kwin/835455939f4955baf88e2343c50b1df0f6e8cf5e

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to