fix: revoke downloaded blobs asynchronously

fixes #1163

src/common/download.js

@@ -1,15 +1,10 @@
-export function downloadUrl(url, name, callback) {
+import { makePause } from '#/common';
+
+export function downloadBlob(blob, name) {
+  const url = URL.createObjectURL(blob);
   const a = document.createElement('a');
   a.href = url;
   a.download = name || '';
   a.dispatchEvent(new MouseEvent('click'));
-  if (callback) setTimeout(callback, 3000);
-}
-
-export function downloadBlob(blob, name, callback) {
-  const url = URL.createObjectURL(blob);
-  downloadUrl(url, name, () => {
-    URL.revokeObjectURL(url);
-    if (callback) callback();
-  });
+  makePause(3000).then(() => URL.revokeObjectURL(url));
 }

src/injected/web/gm-api.js

@@ -234,14 +234,18 @@ function getResource(context, name, isBlob) {
   }
 }
 
-async function downloadBlob(res) {
+function downloadBlob(res) {
   const { context: { name, onload }, response } = res;
   const url = createObjectURL(response);
   const a = document::createElementNS(NS_HTML, 'a');
   a::setAttribute('href', url);
   if (name) a::setAttribute('download', name);
   a::dispatchEvent(new MouseEvent('click'));
+  revokeBlobAfterTimeout(url);
+  onload?.(res);
+}
+
+async function revokeBlobAfterTimeout(url) {
   await bridge.send('SetTimeout', 3000);
   revokeObjectURL(url);
-  onload?.(res);
 }