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
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
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.