isa/gnome-shell
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

status/network: Use connectObject() to connect client signals

Browse Source 
We would want the signals to be disconnected if we ever happened to
destroy the indicator. Even if we don't, connectObject() is simply
nicer when connecting half a dozen handlers at once.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/2406>
This commit is contained in:
Florian Müllner 2022-08-01 01:47:22 +02:00
parent c33b5de174
commit f411228fa0
1 changed files with 13 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
js/ui/status/network.js
View File

@ -1783,17 +1783,19 @@ class Indicator extends PanelMenu.SystemIndicator {
this._syncMainConnection();
this._syncVpnConnections();
this._client.connect('notify::nm-running', this._syncNMState.bind(this));
this._client.connect('notify::networking-enabled', this._syncNMState.bind(this));
this._client.connect('notify::state', this._syncNMState.bind(this));
this._client.connect('notify::primary-connection', this._syncMainConnection.bind(this));
this._client.connect('notify::activating-connection', this._syncMainConnection.bind(this));
this._client.connect('notify::active-connections', this._syncVpnConnections.bind(this));
this._client.connect('notify::connectivity', this._syncConnectivity.bind(this));
this._client.connect('device-added', this._deviceAdded.bind(this));
this._client.connect('device-removed', this._deviceRemoved.bind(this));
this._client.connect('connection-added', this._connectionAdded.bind(this));
this._client.connect('connection-removed', this._connectionRemoved.bind(this));
this._client.connectObject(
'notify::nm-running', () => this._syncNMState(),
'notify::networking-enabled', () => this._syncNMState(),
'notify::state', () => this._syncNMState(),
'notify::primary-connection', () => this._syncMainConnection(),
'notify::activating-connection', () => this._syncMainConnection(),
'notify::active-connections', () => this._syncVpnConnections(),
'notify::connectivity', () => this._syncConnectivity(),
'device-added', this._deviceAdded.bind(this),
'device-removed', this._deviceRemoved.bind(this),
'connection-added', this._connectionAdded.bind(this),
'connection-removed', this._connectionRemoved.bind(this),
this);
try {
this._configPermission = await Polkit.Permission.new(