Improve error reporting of Promise.jsm

RESOLVED FIXED in mozilla14

Status

()

Core
General
--
enhancement
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Yoric, Assigned: Yoric)

Tracking

unspecified
mozilla14
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

At the moment, in many cases, Promise.jsm causes silent dropping of exceptions. Let's fix this.
Created attachment 591035 [details] [diff] [review]
Better error reporting

Sorry for the snafu, this time the patch is attached to the correct bug
Attachment #591035 - Flags: review?(jwalker)
Attachment #591035 - Flags: review?(jwalker) → review+
Blocks: 692420
Depends on: 711126
Created attachment 600383 [details] [diff] [review]
Better error reporting

Attaching a version of this patch that uses function |Promise._error| introduced in bug 711126.
Attachment #600383 - Flags: review?(rcampbell)
Attachment #591035 - Attachment is obsolete: true
Attachment #600383 - Flags: review?(rcampbell) → review?(jwalker)
Attachment #600383 - Flags: review?(jwalker) → review+
Created attachment 603715 [details] [diff] [review]
Better error reporting

Run a few additional tests on dependent libraries, found an error. Here is the same patch, without the error.
Attachment #600383 - Attachment is obsolete: true
Blocks: 735477
Blocks: 711126
No longer depends on: 711126
Keywords: checkin-needed

Updated

6 years ago
Assignee: nobody → dteller
https://hg.mozilla.org/integration/mozilla-inbound/rev/41d14a8398a2
Keywords: checkin-needed
Target Milestone: --- → mozilla14
https://hg.mozilla.org/mozilla-central/rev/41d14a8398a2
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.