A question of the browser embedding of the engine, not the engine itself. Reassigning to Browser-General for proper consideration -
Assignee: rogerl → asa
QA Contact: pschwartau → doronr
See also the related bug 50291 -
adding brendan. looking for more input.
Developers want this on, generally -- end users do not. Thoughts: have it on for DEBUG builds; have it on by default in Mozilla binaries but not in Netscape commercial builds; have it off in Mozilla milestone builds too (treat them like commercial builds). This bug should go to someone who can preset the pref. I'm not sure who that is right now, possibly it's a don melton group thang. /be
So who should it be reassigned or cc to?
Ben, can you look at this or bounce to someone else in gramps's group? Thanks, /be
Assignee: asa → ben
This may be mine, not that it really matters who does it. So, Brendan, are you officially approving this change?
For DEBUG builds, email@example.com /be
Easy enough to set a default pref differently between Mozilla and N6, but not as easy to differentiate debug and release XP (on Win and Unix I guess it would be easy enough to echo the pref onto the end of all.js) Instead, how 'bout skipping the pref and setting the default in nsJSEnvironment.cpp. If someone has set their own pref it'll overwrite this value. Will attach patch.
Assignee: ben → dveditz
Thanks, good idea -- how about using #else so mDefaultJSOptions is set to 0 only if DEBUG isn't defined? Do that and r,firstname.lastname@example.org. /be
The assertions are definitely due to the strict option. There is no way I'm checking this in, I'd get lynched. The assertion is at mozilla/rdf/content/src/nsXULPrototypeDocument.cpp line 523, looks like nsXULPDGlobalObject::GetDocShell() was not intended to be reached (hard to tell with the helpful message "waaah!"). CC'ing waterson who appears to have written this code. Looks like NS_ScriptErrorReporter() handles the lack of a docshell just fine, maybe the NS_NOTREACHED() could be changed to a NS_WARNING() if it's important at all.
Status: NEW → ASSIGNED
Waterson's on the road (back Thursday, IIRC), maybe he should get this bug temporarily, then back to dveditz to do the deed? /be
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.