Bug 424376 comment 14 provides the details.
(In reply to bug 424376 comment #14) > igor, any suggestion on the patch? The patch is fine.
It's better to post the patch here. I'll obsolete the old one.
Attachment #311341 - Flags: review?(igor)
Comment on attachment 311341 [details] [diff] [review] patch v1 As igor's comment 1 is fine, I'd like to ask for b5 approval for this followup fix.
(In reply to comment #3) > (From update of attachment 311341 [details] [diff] [review]) > As igor's comment 1 is fine, I'd like to ask for b5 approval for this followup > fix. dtrace code is not a part of the default browser build. As such no approval is necessary.
Assignee: general → alfred.peng
Comment on attachment 311341 [details] [diff] [review] patch v1 Cancel the request for b5 approval per igor's comment above.
I checked in the patch from the comment 2 to the trunk: Checking in jsdtracef.c; /cvsroot/mozilla/js/src/jsdtracef.c,v <-- jsdtracef.c new revision: 3.8; previous revision: 3.7 done
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.