lordwelch/downthemall
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use promises in content scripts

Browse Source
This commit is contained in:
Nils Maier 2019-09-04 14:07:18 +02:00
parent 2c18ddaaa8
commit fdcdae0412
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
scripts/gather.ts
View File

@ -295,7 +295,7 @@ class Gatherer {
function gather(msg: any, sender: any, callback: Function) { function gather(msg: any, sender: any, callback: Function) {
try { try {
if (!msg || msg.type !== "DTA:gather" || !callback) { if (!msg || msg.type !== "DTA:gather" || !callback) {
return; return Promise.resolve(null);
} }
const gatherer = new Gatherer(msg); const gatherer = new Gatherer(msg);
const result = { const result = {
@ -313,10 +313,11 @@ function gather(msg: any, sender: any, callback: Function) {
), ),
}; };
urlToUsable(result, result.baseURL); urlToUsable(result, result.baseURL);
callback(result); return Promise.resolve(result);
} }
catch (ex) { catch (ex) {
console.error(ex.toString(), ex.stack, ex); console.error(ex.toString(), ex.stack, ex);
return Promise.resolve(null);
} }
} }