Switch the login sound to Phonon directly...for now

Review Request #123834 - Created May 18, 2015 and submitted

Information
Martin Klapetek
plasma-workspace
Reviewers
plasma

With its current architecture, KNotification can cause crashes on logout (and also cause asserts in certain situations, that will be another fix). So in the meantime, this replaces the KNotification-in-a-thread with Phonon directly.

This is exactly what KNotification would do. This is for the time being until the crash on logout is sorted out.

Additionally, this also fixes logout sound which was missing before. This uses normal KNotification as at that point we don't need to be threading or anything, so KNotification is just safe.

Login sound works as expected, logout sound as well. Also tested by couple other people.

Martin Klapetek
Review request changed

Status: Closed (submitted)

Change Summary:

Submitted with commit c445f99304bd0190a7a0b83e4700222132bc34d1 by Martin Klapetek to branch master.
Loading...