Still the timeout isn't getting cleaned up on destroy (line 315 `extension.js`).

So would something like the following work? async _retryFetchMetadata(retries = 3, delay = 500) { for (let i = 0; i < retries; i++) { try { let metadata = await this._getMetadata(); if (metadata['xesam:artist'] && metadata['xesam:title']) { this._updateTrackInfo(metadata); logDebug('Successfully fetched valid Metadata on retry'); return; } } catch (e) { logError(e, 'Retry fetching Metadata failed'); } // Await the cancellable sleep await this._sleep(delay); } logDebug('Failed to fetch valid Metadata after retries'); } _sleep(delay) { return new Promise((resolve) => { const timeoutID = setTimeout(() => { resolve(); // Remove the timeoutID from activeTimeouts once resolved const index = this._activeTimeouts.indexOf(timeoutID); if (index > -1) { this._activeTimeouts.splice(index, 1); } }, delay); this._activeTimeouts.push(timeoutID); }); } SpotifyIndicator.destroy(): ... // Clear all active timeouts for (let timeoutID of this._activeTimeouts) { clearTimeout(timeoutID); } this._activeTimeouts = [];

Yes, that would be ok.