Skip to content

[KDynamicJobTracker] Support KUiServerV2JobTracker

Kai Uwe Broulik requested to merge work/kbroulik/dynamicjobviewv2 into master

Track the KUIServer DBus service and check (unfortunately through DBus introspection) whether it supports API version 2 and cache it until the service owner changes or disappears. This way only blocking DBus call is only done initially and subsequent jobs are block-free.

KUiServer is also part of Plasma these days, so there's no need for asking it about whether a job view is present. This cannot really be changed for V1 but for V2 we can skip this.

@dfaure @nicolasfella @davidedmundson

Depends on kjobwidgets!6 (merged)

  • Dolphin uses jobviewv2 out of the box now
  • Quit plasmashell, started another copy, got a widget dialog
  • Restarted plasmashell again, started another copy, got a jobviewv2 again
  • Commented V2 from plasmashell and restarted, got V1 used

KIO creates the dynamic job tracker as soon as you link against KIO, so in theory we could do an async DBus call initially here, but I didn't really want any app that links KIO and might not use jobs to do that. Still nice that we can skip the "needs view" check in subsequent requests.

Merge request reports