Removing the Activity wrapper class from the Shell
ClosedPublic

Authored by ivan on Jul 14 2016, 7:38 PM.

Details

Summary

After the previous patch which removed the unused
Activity class related events, it started becoming clear that
said class is only really used to store the name of the
containment plugin that should be loaded for an activity.

Instead of keeping a map of these objects (QObjects) with
all their baggage, the shells now only keep the map of
activity IDs and corresponding plugin names.

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ivan updated this revision to Diff 5179.Jul 14 2016, 7:38 PM
ivan retitled this revision from to Removing the Activity wrapper class from the Shell.
ivan updated this object.
ivan edited the test plan for this revision. (Show Details)
ivan added reviewers: Plasma, mart, davidedmundson, bshah.
Restricted Application added a project: Plasma. · View Herald TranscriptJul 14 2016, 7:38 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson accepted this revision.Jul 14 2016, 8:08 PM
davidedmundson edited edge metadata.
This revision is now accepted and ready to land.Jul 14 2016, 8:08 PM
davidedmundson accepted this revision.Jul 14 2016, 8:09 PM
This revision was automatically updated to reflect the committed changes.