Closed
Bug 786666
Opened 12 years ago
Closed 12 years ago
workaround __exposedProps__ typed array issue in mozTCPSocket
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla18
People
(Reporter: asuth, Assigned: asuth)
References
Details
Attachments
(1 file)
3.48 KB,
patch
|
fabrice
:
review+
|
Details | Diff | Splinter Review |
Bug 786639 makes mozTCPSocket's arraybuffer binary type support useless to content, but there is a simple workaround of creating the uint8array so that it lives in the content window's compartment. Unit-test wise, this will be covered by the fix for bug 784893.
Attachment #656442 -
Flags: review?(fabrice)
Updated•12 years ago
|
Attachment #656442 -
Flags: review?(fabrice) → review+
Assignee | ||
Comment 1•12 years ago
|
||
pushed to inbound: https://hg.mozilla.org/integration/mozilla-inbound/rev/b739aa9d9ede
Comment 2•12 years ago
|
||
Nice! Simple and logical fix.
Comment 3•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/b739aa9d9ede
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
Assignee | ||
Comment 4•12 years ago
|
||
The fix for bug 786639 has landed, so if this patch was backed out, things should still work. But it actually seems like we may want to keep this fix since it is arguably more correct to have the typed array live in the content window's compartment from both a memory-usage tracking perspective (ex: about:memory) and for efficiency. Donovan, does this seem reasonable? And if so, is there anything more required than to update the comment? In other words, is useWin as implemented okay, or should there be some cleanup? I think constructors may actually work now, so it's also possible that book-keeping could just be simplified if/when that ever happens.
Comment 5•12 years ago
|
||
I think it's best to keep the useWin implementation as you did it in the patch. It seems slightly more "correct" to me, and more representative of the intent. The typedarrays are given to the content, so they should be owned by the content. The about:memory thing alone is enough to make it worth it to leave this patch in, I think.
Updated•11 years ago
|
Component: DOM: Mozilla Extensions → DOM
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•