Differential D23294

[Notifications] Bring back dismissed progress in case of crash
ClosedPublic
Actions

Authored by broulik on Aug 20 2019, 3:08 PM.

Details

Reviewers
ngraham
Group Reviewers
Plasma
Commits
R120:5806f4969604: [Notifications] Bring back dismissed progress in case of crash
Summary

When the application crashes, we would mark the job as failed but not undismiss it.

Test Plan

5.16

  • Started a copy in dolphin, minimized the copy popup, killed dolphin

→ Got the popup back saying the app crashed

Diff Detail

Repository
R120 Plasma Workspace
Lint
Lint Skipped
Unit
Unit Tests Skipped
broulik created this revision.Aug 20 2019, 3:08 PM
Restricted Application added a project: Plasma. · View Herald TranscriptAug 20 2019, 3:08 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Aug 20 2019, 3:08 PM
ngraham accepted this revision.Aug 20 2019, 4:19 PM
This revision is now accepted and ready to land.Aug 20 2019, 4:19 PM
Closed by commit R120:5806f4969604: [Notifications] Bring back dismissed progress in case of crash (authored by broulik). · Explain WhyAug 21 2019, 12:28 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 64128

View Options

libnotificationmanager/jobsmodel_p.cpp

Loading...