From 93e3e938b322433aff862bbc46f80c60ab7dc2ab Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Florian=20M=C3=BCllner?= <fmuellner@gnome.org>
Date: Tue, 17 Jan 2023 20:31:21 +0100
Subject: [PATCH] desktop-icons: Don't use blocking IO

---
 extensions/desktop-icons/desktopManager.js | 35 +++++++++++++++++-----
 1 file changed, 28 insertions(+), 7 deletions(-)

diff --git a/extensions/desktop-icons/desktopManager.js b/extensions/desktop-icons/desktopManager.js
index 399aee03..2ce6eefb 100644
--- a/extensions/desktop-icons/desktopManager.js
+++ b/extensions/desktop-icons/desktopManager.js
@@ -53,6 +53,21 @@ function findMonitorIndexForPos(x, y) {
     return getDpy().get_monitor_index_for_rect(new Meta.Rectangle({x, y}));
 }
 
+async function queryInfo(file, attributes = DesktopIconsUtil.DEFAULT_ATTRIBUTES, cancellable = null) {
+    const flags = Gio.FileQueryInfoFlags.NONE;
+    const priority = GLib.PRIORITY_DEFAULT;
+    return new Promise((resolve, reject) => {
+        file.query_info_async(attributes, flags, priority, cancellable, (o, res) => {
+            try {
+                const info = file.query_info_finish(res);
+                resolve(info);
+            } catch (e) {
+                reject(e);
+            }
+        });
+    });
+}
+
 
 var DesktopManager = GObject.registerClass({
     Properties: {
@@ -221,9 +236,7 @@ var DesktopManager = GObject.registerClass({
 
         if (!this._unixMode) {
             let desktopDir = DesktopIconsUtil.getDesktopDir();
-            let fileInfo = desktopDir.query_info(Gio.FILE_ATTRIBUTE_UNIX_MODE,
-                                                 Gio.FileQueryInfoFlags.NONE,
-                                                 null);
+            let fileInfo = await queryInfo(desktopDir, Gio.FILE_ATTRIBUTE_UNIX_MODE);
             this._unixMode = fileInfo.get_attribute_uint32(Gio.FILE_ATTRIBUTE_UNIX_MODE);
             this._setWritableByOthers((this._unixMode & S_IWOTH) != 0);
         }
@@ -268,14 +281,22 @@ var DesktopManager = GObject.registerClass({
                 Gio.FileQueryInfoFlags.NONE,
                 GLib.PRIORITY_DEFAULT,
                 this._desktopEnumerateCancellable,
-                (source, result) => {
+                async (source, result) => {
                     try {
                         let fileEnum = source.enumerate_children_finish(result);
+                        let extraFolders = await Promise.all(DesktopIconsUtil.getExtraFolders()
+                            .map(async ([folder, extras]) => {
+                                const info = await queryInfo(folder,
+                                    DesktopIconsUtil.DEFAULT_ATTRIBUTES,
+                                    this._desktopEnumerateCancellable);
+                                return [folder, info, extras];
+                            }));
+
                         let resultGenerator = function *() {
+                            for (let [newFolder, info, extras] of extraFolders)
+                                yield [newFolder, info, extras];
+
                             let info;
-                            for (let [newFolder, extras] of DesktopIconsUtil.getExtraFolders()) {
-                                yield [newFolder, newFolder.query_info(DesktopIconsUtil.DEFAULT_ATTRIBUTES, Gio.FileQueryInfoFlags.NONE, this._desktopEnumerateCancellable), extras];
-                            }
                             while ((info = fileEnum.next_file(null)))
                                 yield [fileEnum.get_child(info), info, Prefs.FileType.NONE];
                         }.bind(this);
-- 
2.38.1