Closed
Bug 785196
Opened 13 years ago
Closed 11 years ago
promise/core.js fails test against hostile promises
Categories
(Add-on SDK Graveyard :: General, defect, P2)
Add-on SDK Graveyard
General
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 881047
People
(Reporter: Yoric, Assigned: jsantell)
References
Details
Attachments
(1 file)
|
4.66 KB,
patch
|
Details | Diff | Splinter Review |
This bug is rather surprising, as I seem to remember that promise.js (from the addon-sdk on github) succesfully rejected hostile promises.
Irakli, could you take a look and tell me if I got something wrong somewhere?
Attachment #654750 -
Flags: feedback?(rFobic)
Comment 1•13 years ago
|
||
That's surprising indeed, I'll incorporate your tests into our test suite and will submit fix if there is a issue.
Updated•13 years ago
|
Assignee: nobody → rFobic
Priority: -- → P2
| Reporter | ||
Comment 2•13 years ago
|
||
Could you reproduce the issue?
| Assignee | ||
Comment 3•12 years ago
|
||
This can probably be closed once bug 881047 is finished and we use Promise.jsm as the backend for our promises.
Depends on: 881047
Comment 4•12 years ago
|
||
Can you please include test from the submitted patch to either `Promise.jsm` tests or to our tests to make sure it fixed ?
Assignee: rFobic → jsantell
Updated•12 years ago
|
Attachment #654750 -
Flags: feedback?(rFobic)
| Assignee | ||
Comment 5•11 years ago
|
||
With Bug 881047 migrating sdk/core/promise to use Promise.jsm as the back end, I think this bug is redundant once that lands.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•