From 1345899253b20516b9ab88ecb1a6c6de93575cdf Mon Sep 17 00:00:00 2001 From: lilia Date: Sat, 3 Oct 2015 23:19:53 -0700 Subject: [PATCH] Use reject instead of throw This ensures that the containing promise is rejected without triggering the side effects of an uncaught exception, such as causing the debugger to pause. // FREEBIE --- js/libtextsecure.js | 2 +- libtextsecure/message_receiver.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/js/libtextsecure.js b/js/libtextsecure.js index addc3e401..b79916e30 100644 --- a/js/libtextsecure.js +++ b/js/libtextsecure.js @@ -39305,7 +39305,7 @@ MessageReceiver.prototype = { ev.error = error; ev.proto = envelope; this.dispatchEvent(ev); - throw error; // reject this promise + return Promise.reject(error); }.bind(this)); }, handleSentMessage: function(destination, timestamp, message) { diff --git a/libtextsecure/message_receiver.js b/libtextsecure/message_receiver.js index 169c61699..67bfdc04e 100644 --- a/libtextsecure/message_receiver.js +++ b/libtextsecure/message_receiver.js @@ -111,7 +111,7 @@ MessageReceiver.prototype = { ev.error = error; ev.proto = envelope; this.dispatchEvent(ev); - throw error; // reject this promise + return Promise.reject(error); }.bind(this)); }, handleSentMessage: function(destination, timestamp, message) {