Closed Bug 414982 Opened 13 years ago Closed 13 years ago
Zeal setting in ns JSEnvironment, as we once did with WAY _TOO _MUCH _GC
As bz notes in bug 308429, I did not properly convert a WAY_TOO_MUCH_GC preprocessor guard in nsJSEnvironment to use the new runtime gcZeal parameter. This patch does that. It also fixes a warning I noticed while testing my change.
Could've sworn I added this attachment the first time. :(
Comment on attachment 300527 [details] [diff] [review] patch, v1 Looks good.
Comment on attachment 300527 [details] [diff] [review] patch, v1 Would be pretty harmless for b3, but I'll gladly settle for 1.9 approval and then I'll throw a checkin-needed on it.
Comment on attachment 300527 [details] [diff] [review] patch, v1 This should go in for sure, and it could be useful in b3. It is safe. /be
If someone has an opportunity to land this for me today, that would be great. Thanks.
Checking in dom/src/base/nsJSEnvironment.cpp; /cvsroot/mozilla/dom/src/base/nsJSEnvironment.cpp,v <-- nsJSEnvironment.cpp new revision: 1.385; previous revision: 1.384 done
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9beta3
Probably wanted on the 1.8 branch
Should clear the wanted/blocking flags on these bugs, as they are replaced for the 1.8 branch by the rollup patch in bug 426628.
You need to log in before you can comment on or make changes to this bug.