wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 1027033 - AsyncShutdown errors should cause test failures but should not break AsyncShutdown. r=froydnj

Browse Source
This commit is contained in:
David Rajchenbach-Teller 2014-06-18 05:40:00 -04:00
parent fded9a110a
commit 0fbe095e41
1 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
toolkit/modules/AsyncShutdown.jsm
View File

@ -341,7 +341,13 @@ Spinner.prototype = {
promise.then(() => satisfied = true); // This promise cannot reject
let thread = Services.tm.mainThread;
while (!satisfied) {
thread.processNextEvent(true);
try {
thread.processNextEvent(true);
} catch (ex) {
// An uncaught error should not stop us, but it should still
// be reported and cause tests to fail.
Promise.reject(ex);
}
}
}
};
@ -590,6 +596,10 @@ Barrier.prototype = Object.freeze({
" Phase: " + topic +
" State: " + safeGetState(fetchState);
warn(msg, error);
// The error should remain uncaught, to ensure that it
// still causes tests to fail.
Promise.reject(error);
});
condition.then(() => indirection.resolve());