make nsICycleCollectorListener builtinclass

RESOLVED FIXED in mozilla13

Status

()

Core
XPCOM
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: mccr8, Assigned: mccr8)

Tracking

Trunk
mozilla13
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Running JS during a CC is a bad idea, and Luke found that with his JS-singlethreading patch, doing so causes some kind of infinite loop.  So we should just ban listeners implemented in JS.
(Assignee)

Comment 1

5 years ago
Created attachment 592271 [details] [diff] [review]
simple patch, untested
Assignee: nobody → continuation
(Assignee)

Comment 2

5 years ago
Try run looked good.
(Assignee)

Comment 3

5 years ago
Comment on attachment 592271 [details] [diff] [review]
simple patch, untested

Not a huge priority, but it would be nice to fix this.
Attachment #592271 - Flags: review?(bugs)

Comment 4

5 years ago
Comment on attachment 592271 [details] [diff] [review]
simple patch, untested

Just to be safe, could you update the uuid.
Attachment #592271 - Flags: review?(bugs) → review+
(Assignee)

Comment 5

5 years ago
Ah, right, I meant to ask about that.  I'll do that, thanks.
(Assignee)

Comment 6

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/a949ded5a23b
Target Milestone: --- → mozilla13
(Assignee)

Comment 7

5 years ago
https://hg.mozilla.org/mozilla-central/rev/a949ded5a23b
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.