The default bug view has changed. See this FAQ.

Worker interval timers can be deleted prematurely.

RESOLVED FIXED in Firefox 12

Status

()

Core
DOM
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: khuey, Assigned: khuey)

Tracking

Trunk
mozilla13
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox12 fixed)

Details

(Whiteboard: [qa-])

Attachments

(1 attachment)

Created attachment 595910 [details] [diff] [review]
Patch
Attachment #595910 - Flags: review?(bent.mozilla)
Blocks: 629597
No longer blocks: 692597
Attachment #595910 - Flags: review?(bent.mozilla) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/82779bfa0c2c
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla13
https://hg.mozilla.org/mozilla-central/rev/82779bfa0c2c
Comment on attachment 595910 [details] [diff] [review]
Patch

[Approval Request Comment]
Regression caused by (bug #): N/A
User impact if declined: setInterval timers in workers will disappear on slower machines.
Testing completed (on m-c, etc.): On m-c
Risk to taking this patch (and alternatives if risky): Low risk.
String changes made by this patch: N/A

This fixes a bug in a web feature that causes it to randomly break depending on the speed of the machine.  The patch is simple enough that we could uplift it (as a bonus it fixes some randomorange bugs).
Attachment #595910 - Flags: approval-mozilla-beta?
Attachment #595910 - Flags: approval-mozilla-aurora?

Comment 4

5 years ago
Comment on attachment 595910 [details] [diff] [review]
Patch

[Triage Comment]
Our earliest instance of this bug was from 2010 in comment 0 of 612311. Given the low-risk nature of the patch, we'll take it for Aurora 12. But we don't have any significant need for this on Beta, especially when we're only 3 weeks from finishing up the release.
Attachment #595910 - Flags: approval-mozilla-beta?
Attachment #595910 - Flags: approval-mozilla-beta-
Attachment #595910 - Flags: approval-mozilla-aurora?
Attachment #595910 - Flags: approval-mozilla-aurora+
https://hg.mozilla.org/releases/mozilla-aurora/rev/d3c833832242
status-firefox12: --- → fixed
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.