Register objectPath before service
ClosedPublic

Authored by davidedmundson on Aug 29 2017, 8:41 AM.

Details

Summary

When the service is regsitered we want to have all the internal routing
pre-configured. Otherwise pending calls (such as pending DBus activated
calls) can be processed before the path is registered and then fail with
an error that the path does not exist.

Also remove the fairly redundant check on registerObject. That can return
false if some object in this process has registered itself on that path; but in that
case, the very last thing we'd want to do is call unregisterService.

Test Plan

Generated a load of notifications
Started plasma
Ouptut showed valid return on GetCapabilities, not an error on missing path

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.
davidedmundson created this revision.Aug 29 2017, 8:41 AM
Restricted Application added a project: Plasma. · View Herald TranscriptAug 29 2017, 8:41 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik accepted this revision.Aug 30 2017, 12:05 PM
This revision is now accepted and ready to land.Aug 30 2017, 12:05 PM
This revision was automatically updated to reflect the committed changes.