Closed
Bug 1248497
Opened 8 years ago
Closed 8 years ago
Add promise support to the sendMessage APIs.
Categories
(WebExtensions :: Untriaged, defect, P2)
WebExtensions
Untriaged
Tracking
(firefox47 fixed)
Tracking | Status | |
---|---|---|
firefox47 | --- | fixed |
People
(Reporter: kmag, Assigned: kmag)
Details
(Whiteboard: triaged)
Attachments
(1 file)
We should both return promises from the sendMessage functions, and allow the listener functions to return Promise objects, rather than `true`, to send a delayed response.
Assignee | ||
Updated•8 years ago
|
Iteration: --- → 47.3 - Mar 7
Updated•8 years ago
|
Flags: blocking-webextensions?
Priority: -- → P2
Whiteboard: triaged
Assignee | ||
Comment 1•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/36979/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/36979/
Attachment #8724322 -
Flags: review?(wmccloskey)
Comment on attachment 8724322 [details] MozReview Request: Bug 1248497 – Add promise support to the sendMessage APIs. r?billm https://reviewboard.mozilla.org/r/36979/#review33805 ::: browser/components/extensions/test/browser/browser_ext_tabs_sendMessage.js:24 (Diff revision 1) > + browser.test.fail("Got unexpected resposne callback"); response
Attachment #8724322 -
Flags: review?(wmccloskey) → review+
Assignee | ||
Comment 3•8 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/0a65c11157dac3a0940bd834a10bc3ff5702d2f2 Bug 1248497 – Add promise support to the sendMessage APIs. r=billm
Comment 4•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/0a65c11157da
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox47:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Updated•6 years ago
|
Product: Toolkit → WebExtensions
You need to log in
before you can comment on or make changes to this bug.
Description
•