[Notifications Data Engine] Expose desktop entry and application service name icon
ClosedPublic

Authored by broulik on Jun 6 2017, 6:47 AM.

Details

Summary

Now that we have "desktop-entry" available, expose this information in the data engine.
This could enable a notification implementation to show the application name or do smarter grouping.
The naming is somewhat suboptimal but "appName" and "appIcon" are already occupied and predate this.
I also didn't just want to overwrite the hints the application gives, especially not the "appIcon" which is used as notification icon.

Test Plan

Had chrome send a notification, got correct "google-chrome-unstable" icon name and "Google Chrome (unstable)" service name

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.
broulik created this revision.Jun 6 2017, 6:47 AM
Restricted Application added a project: Plasma. · View Herald TranscriptJun 6 2017, 6:47 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mart accepted this revision.Jun 6 2017, 8:51 AM
This revision is now accepted and ready to land.Jun 6 2017, 8:51 AM
This revision was automatically updated to reflect the committed changes.