Turn off CSP warning command line output in opt builds

RESOLVED FIXED in mozilla2.0b8

Status

()

defect
RESOLVED FIXED
9 years ago
4 months ago

People

(Reporter: geekboy, Assigned: geekboy)

Tracking

(Blocks 1 bug)

Trunk
mozilla2.0b8
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

CSP echos warnings and errors placed in the console to dump() as well.  We should turn off the spew in opt builds for sure, and maybe change it to PR_LOGGING output for debugging.
Posted patch fixSplinter Review
Removes the dump() calls from the CSPWarn CSPError and CSPdebug functions.  Information will still be placed in the error console.
Attachment #487032 - Flags: review?(jst)
Status: NEW → ASSIGNED
Comment on attachment 487032 [details] [diff] [review]
fix

r+a=jst, feel free to land once beta7 is out the door (or at least branched etc).
Attachment #487032 - Flags: review?(jst)
Attachment #487032 - Flags: review+
Attachment #487032 - Flags: approval2.0+
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/cc7061bbe843
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.