Closed Bug 705020 Opened 11 years ago Closed 10 years ago

remove enable-js-diagnostics flag from nightly mozconfig


(Firefox for Android Graveyard :: General, defect)

Not set


(firefox20 fixed, firefox21 fixed)

Firefox 21
Tracking Status
firefox20 --- fixed
firefox21 --- fixed


(Reporter: dougt, Assigned: kats)



(2 files)

From email:

we are building with --enable-js-diagnostics in our nightly builds.  anyone know what this means w.r.t perf?  Also, if this is a high cost, do we know the value?

David Mandelin:
I don't think it's too expensive. But I also doubt it would really be useful to you--that stuff was for trying to diagnose GC crashes, and mostly it just told us that all of our actionable ideas about what was causing crashes weren't right.
Attached patch patch v.1Splinter Review
Assignee: nobody → doug.turner
Attachment #576657 - Flags: review?(mbrubeck)
Attachment #576657 - Flags: review?(mbrubeck) → review+
Closed: 11 years ago
Resolution: --- → FIXED
This option still exists in the mobile/xul nightly mozconfig, and should probably be removed there. The reason I care is because it is now in the mozilla-aurora tree as well, even though it's explicitly supposed to only be on nightlies. Clearly something went wrong.
Resolution: FIXED → ---
Attachment #699203 - Flags: review?(doug.turner)
Comment on attachment 699203 [details] [diff] [review]
Remove from mobile/xul

[Approval Request Comment]
Bug caused by (feature/regressing bug #): version bump when aurora went from 19 -> 20
User impact if declined: in theory, aurora builds from XUL fennec will have some js diagnostic stuff turned on, but i don't think anybody is building that so it doesn't really matter. this does show up when i diff the FF19 tree at the end of aurora and the start of beta so that's kind of annoying
Testing completed (on m-c, etc.): none
Risk to taking this patch (and alternatives if risky): should be no risk, this isn't even being used anywhere really
String or UUID changes made by this patch: none
Attachment #699203 - Flags: approval-mozilla-aurora?
Actually, sorry, it's unrelated to the 19 -> 20 version bump, I misread the hg history. It's been on aurora for a long time.
Comment on attachment 699203 [details] [diff] [review]
Remove from mobile/xul

Review of attachment 699203 [details] [diff] [review]:

i do not care about mobile/xul, but the patch lgtm.  i'd love to r+ a patch that removed mobile/xul from m-c.
Attachment #699203 - Flags: review?(doug.turner) → review+
This was NPOTB - for future patches like this, please can you use DONTBUILD :-)
(d326a33ad6f6 that is)
Assignee: doug.turner → bugmail.mozilla
Closed: 11 years ago10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 21
Attachment #699203 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.