Clear previous actions on update
ClosedPublic

Authored by davidedmundson on Aug 31 2017, 1:08 PM.

Details

Summary

addAction() doesn't check if an action already exists with that name so
we have a tiny memory leak.

Also I wanted to try and support actions potentially changing if a
remote service gets restarted. So we should clear our current actions
before repopulating

Test Plan

Builds

Diff Detail

Repository
R308 KRunner
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.Aug 31 2017, 1:08 PM
Restricted Application added projects: Plasma, Frameworks. · View Herald TranscriptAug 31 2017, 1:08 PM
Restricted Application added subscribers: Frameworks, plasma-devel. · View Herald Transcript
broulik accepted this revision.Aug 31 2017, 1:10 PM
This revision is now accepted and ready to land.Aug 31 2017, 1:10 PM
This revision was automatically updated to reflect the committed changes.