Workers postMessage doesn't set its return value

RESOLVED FIXED in mozilla26

Status

()

Core
DOM: Workers
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: Ms2ger, Assigned: baku)

Tracking

Trunk
mozilla26
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 3 obsolete attachments)

(Reporter)

Description

4 years ago
So we end up reading the callee back.
(Reporter)

Comment 1

4 years ago
Test case: type "postMessage()" into http://mozilla.pettay.fi/workerconsole/
(Assignee)

Comment 2

4 years ago
Created attachment 798521 [details] [diff] [review]
patch
Attachment #798521 - Flags: review?
(Assignee)

Updated

4 years ago
Attachment #798521 - Flags: review? → review?(mrbkap)
(Assignee)

Comment 3

4 years ago
Created attachment 798522 [details] [diff] [review]
patch

mochitest
Attachment #798522 - Flags: review?(mrbkap)
We really need to fix up all the JSNatives in dom/workers.
Comment on attachment 798522 [details] [diff] [review]
patch

r=me on this, but khuey's right: there's a bunch more functions that need this treatment.
Attachment #798522 - Flags: review?(mrbkap) → review+
Comment on attachment 798521 [details] [diff] [review]
patch

I think this was technically subsumed by attachment 798522 [details] [diff] [review], but r=me twice!
Attachment #798521 - Flags: review?(mrbkap) → review+
(Assignee)

Comment 7

4 years ago
Created attachment 798770 [details] [diff] [review]
patch
Attachment #798521 - Attachment is obsolete: true
Attachment #798522 - Attachment is obsolete: true
(Assignee)

Comment 8

4 years ago
https://tbpl.mozilla.org/?tree=Try&rev=f2d37485a945
Keywords: checkin-needed
(Assignee)

Comment 9

4 years ago
Created attachment 798771 [details] [diff] [review]
patch

rebased
Attachment #798770 - Attachment is obsolete: true
(Assignee)

Comment 10

4 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/61788b5c3d55
Keywords: checkin-needed
Please file another bug on fixing up everything else.
Assignee: nobody → amarchesini
https://hg.mozilla.org/mozilla-central/rev/61788b5c3d55
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
You need to log in before you can comment on or make changes to this bug.