Closed
Bug 1178076
Opened 9 years ago
Closed 9 years ago
MessagePort should not set listeners nor workerFeature when neutered
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla41
People
(Reporter: baku, Assigned: baku)
Details
Attachments
(1 file, 1 obsolete file)
|
2.98 KB,
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
No description provided.
|
Assignee | |
Comment 1•9 years ago
|
||
Attachment #8626991 -
Flags: review?(bugs)
|
||
Comment 2•9 years ago
|
||
Comment on attachment 8626991 [details] [diff] [review]
mp3.patch
It is not obvious what the test is testing, especially the
'new MessageChannel' in onmessage handler. Could you add some comment there.
(I guess need to make sure to either land this before the merge or land to aurora too)
|
|
||
Comment 3•9 years ago
|
||
Comment on attachment 8626991 [details] [diff] [review]
mp3.patch
So, would like to see a new patch.
Attachment #8626991 -
Flags: review?(bugs) → review-
|
|
Assignee | |
Comment 4•9 years ago
|
||
Comment added. Basically the test itself does nothing expect sending a neutered port to the same window. This should not leak.
Attachment #8626991 -
Attachment is obsolete: true
Attachment #8627181 -
Flags: review?(bugs)
|
|
||
Updated•9 years ago
|
Attachment #8627181 -
Flags: review?(bugs) → review+
|
|
Assignee | |
Comment 5•9 years ago
|
||
|
||
Comment 6•9 years ago
|
||
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox41:
--- → fixed
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
|
||
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•