From b4c2901e47f8ce3256faf51381d1884334c3c4ca Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Florian=20M=C3=BCllner?= <fmuellner@gnome.org>
Date: Mon, 10 Jul 2023 00:00:04 +0200
Subject: [PATCH] dbusServices/extensions: Check earlier for existing dialog

There is no point in making a D-Bus call if we are going to bail
out anyway.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/2832>
---
 js/dbusServices/extensions/extensionsService.js | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/js/dbusServices/extensions/extensionsService.js b/js/dbusServices/extensions/extensionsService.js
index ab31a6485..4ef58babf 100644
--- a/js/dbusServices/extensions/extensionsService.js
+++ b/js/dbusServices/extensions/extensionsService.js
@@ -114,11 +114,10 @@ export const ExtensionsService = class extends ServiceImplementation {
         const [uuid, parentWindow, options] = params;
 
         try {
-            const [serialized] = await this._proxy.GetExtensionInfoAsync(uuid);
-
             if (this._prefsDialog)
                 throw new Error('Already showing a prefs dialog');
 
+            const [serialized] = await this._proxy.GetExtensionInfoAsync(uuid);
             const extension = ExtensionUtils.deserializeExtension(serialized);
 
             this._prefsDialog = new ExtensionPrefsDialog(extension);