Skip to content

Commit

Permalink
statusNotifierWatcher: Emit StatusNotifierHostRegistered (and Unregis…
Browse files Browse the repository at this point in the history
…tered) signals

Other signals we were missing for long time, the specs includes these
signals and they should be emitted when the watcher appears/disappears

So do this, although it seems that Qt apps are still ignoring it.
  • Loading branch information
3v1n0 committed Mar 18, 2021
1 parent b6c1118 commit 6108cf7
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions statusNotifierWatcher.js
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,7 @@ var StatusNotifierWatcher = class AppIndicatorsStatusNotifierWatcher {
this._lostName.bind(this));
this._items = new Map();

this._dbusImpl.emit_signal('StatusNotifierHostRegistered', null);
this._seekStatusNotifierItems();
}

Expand Down Expand Up @@ -234,6 +235,7 @@ var StatusNotifierWatcher = class AppIndicatorsStatusNotifierWatcher {
// this doesn't do any sync operation and doesn't allow us to hook up the event of being finished
// which results in our unholy debounce hack (see extension.js)
Array.from(this._items.keys()).forEach(i => this._remove(i));
this._dbusImpl.emit_signal('StatusNotifierHostUnregistered', null);
Gio.DBus.session.unown_name(this._ownName);
this._cancellable.cancel();
this._dbusImpl.unexport();
Expand Down

0 comments on commit 6108cf7

Please sign in to comment.