Closed Bug 531225 Opened 11 years ago Closed 11 years ago
Workers: Share strings across thread boundary
We're copying now, and it's unfortunate. We should share while we can.
11 years ago
jst, review ping!
Comment on attachment 414680 [details] [diff] [review] Patch, v1 Duh, sorry for the way too long delay :( r=jst
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
How safe would this be for 3.6.x? Could be a big perf improvement for a lot of people writing JS stuff out there.
(In reply to comment #4) > How safe would this be for 3.6.x? Could be a big perf improvement for a lot of > people writing JS stuff out there. I can't think of any reason this would be risky, so after it bakes for a bit I'd be all for backporting. The code hasn't really changed much since 3.5.
Comment on attachment 414680 [details] [diff] [review] Patch, v1 I'd like to take this on branch since it avoids copies of data passed by workers. Pretty much every single test for workers exercises this code so we'd have known pretty fast if this change had broken anything.
Attachment #414680 - Flags: approval184.108.40.206?
Comment on attachment 414680 [details] [diff] [review] Patch, v1 a1922=beltzner, man, I get nervous taking patches this big on the branch
Attachment #414680 - Flags: approval220.127.116.11? → approval18.104.22.168+
(In reply to comment #8) > http://hg.mozilla.org/releases/mozilla-1.9.2/rev/ae89af4f7cd0 This appears to have caused the persistent leak on Windows mochitests; see mozilla-1.9.2 tinderboxen: TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 1272 bytes during test execution (threshold set at 484 bytes) TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 159 instances of nsStringBuffer with size 8 bytes each (1272 bytes total) A similar persistent leak shows up on trunk builds on tryserver, though it doesn't get reported on the mozilla-central tinderboxes for some reason. I did a tryserver build with this patch reverted, and confirmed that the leak disappeared.
Yeah, I have no idea why this doesn't bite m-c. Some sort of shutdown ordering thing? Maybe we GC more on m-c than on branch? Who knows.
11 years ago
Duplicate of this bug: 547896
Whiteboard: [evang-wanted-3.6] → [evang-wanted-3.6] [orange]
Whiteboard: [evang-wanted-3.6] [orange] → [evang-wanted-3.6]
You need to log in before you can comment on or make changes to this bug.