Disable bug 716014 for Firefox 15

RESOLVED FIXED

Status

()

Core
JavaScript Engine
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: billm, Assigned: billm)

Tracking

(Blocks: 1 bug)

unspecified
Points:
---

Firefox Tracking Flags

(firefox15+ fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Created attachment 651921 [details] [diff] [review]
patch

Bug 719114 comment 35 has more discussion of this. I'm uncertain whether this would reduce crash volume. However, I think the risk of the doing extra compartment GCs probably outweighs the rewards, which I think are mostly for people with many tabs open.
Attachment #651921 - Flags: review?(bugs)
Blocks: 719114
status-firefox15: --- → affected
tracking-firefox15: --- → +
Attachment #651921 - Flags: review?(bugs) → review+
(Assignee)

Comment 1

5 years ago
Comment on attachment 651921 [details] [diff] [review]
patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 716014
User impact if declined: Possible increase in GC crashes.
Testing completed (on m-c, etc.): On m-c and aurora.
Risk to taking this patch (and alternatives if risky): Low, since this was disabled through part of aurora.
String or UUID changes made by this patch: None
Attachment #651921 - Flags: approval-mozilla-beta?
Attachment #651921 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
(Assignee)

Comment 2

5 years ago
https://hg.mozilla.org/releases/mozilla-beta/rev/65cd4f678ab0
status-firefox15: affected → fixed
(Assignee)

Updated

5 years ago
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Comment 3

5 years ago
(In reply to Bill McCloskey (:billm) from comment #2)
> https://hg.mozilla.org/releases/mozilla-beta/rev/65cd4f678ab0

Do we want to do the same for Beta 16?
tracking-firefox16: --- → ?
(Assignee)

Comment 4

5 years ago
(In reply to Alex Keybl [:akeybl] from comment #3)
> (In reply to Bill McCloskey (:billm) from comment #2)
> > https://hg.mozilla.org/releases/mozilla-beta/rev/65cd4f678ab0
> 
> Do we want to do the same for Beta 16?

It's already disabled in everything beyond 15.

Updated

5 years ago
tracking-firefox16: ? → ---
You need to log in before you can comment on or make changes to this bug.