The default bug view has changed. See this FAQ.

Workers: Possible to release nsGlobalWindow on a background thread

RESOLVED FIXED

Status

()

Core
DOM
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Ben Turner (not reading bugmail, use the needinfo flag!), Assigned: Ben Turner (not reading bugmail, use the needinfo flag!))

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(blocking1.9.2 needed, status1.9.2 .4-fixed, blocking1.9.1 needed, status1.9.1 .10-fixed)

Details

(Whiteboard: [sg:moderate] [qa-noaction-191] [qa-noaction-192])

Attachments

(1 attachment)

Created attachment 431415 [details] [diff] [review]
Patch

I'm utterly surprised, but the nsISupportsHashKey implementation actually addrefs/releases its key value when used. Simple fix.
Attachment #431415 - Flags: superreview?(jst)
Attachment #431415 - Flags: review?(jst)

Updated

7 years ago
Attachment #431415 - Flags: superreview?(jst)
Attachment #431415 - Flags: superreview+
Attachment #431415 - Flags: review?(jst)
Attachment #431415 - Flags: review+
Attachment #431415 - Flags: approval1.9.2.2?
Attachment #431415 - Flags: approval1.9.1.9?
Whiteboard: [sg:moderate]
Comment on attachment 431415 [details] [diff] [review]
Patch

We're code frozen on 1.9.1.9 and 1.9.2.2, so moving these to the next version.

This will need to land and bake on trunk, first.
Attachment #431415 - Flags: approval1.9.2.3?
Attachment #431415 - Flags: approval1.9.2.2?
Attachment #431415 - Flags: approval1.9.1.9?
Attachment #431415 - Flags: approval1.9.1.10?
http://hg.mozilla.org/mozilla-central/rev/e68368ef5c04
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
blocking1.9.1: ? → needed
blocking1.9.2: ? → needed
Attachment #431415 - Flags: approval1.9.2.3?
Attachment #431415 - Flags: approval1.9.2.3+
Attachment #431415 - Flags: approval1.9.1.10?
Attachment #431415 - Flags: approval1.9.1.10+
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/d5bfbe40cf5f
blocking1.9.2: needed → ---
status1.9.2: --- → .3-fixed
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/698585763a49
blocking1.9.1: needed → ---
status1.9.1: --- → .10-fixed
Please don't remove the "needed" part from the blocking1.9.2/1.9.1 flags. That means something to drivers and should be left alone.
blocking1.9.1: --- → ?
blocking1.9.2: --- → ?
I'm guessing that was an accident ... /me shakesfist at bugzilla!
blocking1.9.1: ? → needed
blocking1.9.2: ? → needed
Is there anything for QA to verify with this bug? I suspect not.
(In reply to comment #7)
> Is there anything for QA to verify with this bug? I suspect not.

Yeah, not really. I just magically hit it one day... It requires precise timing.
Whiteboard: [sg:moderate] → [sg:moderate] [qa-noaction-191] [qa-noaction-192]
Group: core-security
You need to log in before you can comment on or make changes to this bug.