
<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Filed a bug to remove the pref at <a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1481898" class="">https://bugzilla.mozilla.org/show_bug.cgi?id=1481898</a>.<br class=""><div><br class=""><blockquote type="cite" class=""><div class="">On Aug 3, 2018, at 7:40 AM, Brian Grinstead <<a href="mailto:bgrinstead@mozilla.com" class="">bgrinstead@mozilla.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">Huh - must have missed that during error console removal. I don’t think the pref name fits close enough with this proposed behavior to reuse it, so I’ll file a bug to remove it.<br class=""><br class="">Brian<br class=""><br class=""><blockquote type="cite" class="">On Aug 2, 2018, at 7:25 PM, Mike Hommey <<a href="mailto:mh@glandium.org" class="">mh@glandium.org</a>> wrote:<br class=""><br class=""><blockquote type="cite" class="">On Fri, Aug 03, 2018 at 12:15:50PM +1000, Mark Hammond wrote:<br class=""><blockquote type="cite" class="">On 3/08/2018 5:16 am, Brian Grinstead wrote:<br class="">In Bug 1439686 we started printing chrome console messages to stdout. At that time we restricted it to chrome callers to avoid extra noise in stdout during local builds - especially since filtering out content messages is a much-requested feature for the Browser Console (Bug 1260877).<br class=""><br class="">Interestingly, a page can already get text out to stdout if browser.dom.window.dump.enabled is true using `window.dump`. But the output isn't great when dealing with objects, and it requires the page to change its code.<br class=""><br class="">We got a request to enable this for console.log in Bug 1480544 - I'm curious if people have opinions on if/how this should be exposed. Should the output be controlled based on the browser.dom.window.dump.enabled pref? Should we have a second pref controlling itf? Something else?<br class=""></blockquote><br class="">This is tricky. I use dump() more than I'd care to admit - it works well in<br class="">content code because you almost never see dump() used in real sites (does<br class="">Chrome even support it?), so there's not a huge amount of "spam" in the<br class="">terminal. Having dump() output from *both* chrome and content can be very<br class="">helpful.<br class=""><br class="">If we also wrote console.log() from content to the terminal, I fear we would<br class="">then end up in a situation where content spams the terminal, with the only<br class="">mitigation being to flip browser.dom.window.dump.enabled to false - but that<br class="">means we also lose dump() output, throwing the baby out with the bathwater.<br class=""><br class="">So my vote would be:<br class=""><br class="">* Leave dump alone - it's basically a firefox-specific easter-egg and no one<br class="">seems to mind that content *could* spam the terminal because it almost never<br class="">does. (I know this wasn't proposed, but it's worth clarifying imo)<br class=""><br class="">* A new pref that would allow content console.* output to be written to this<br class="">terminal. I'd be fine with this defaulting to true wherever<br class="">browser.dom.window.dump.enabled does.<br class=""><br class="">Having "dump" in the name of this pref wouldn't be a good option, because<br class="">IMO, it should *not* affect the behaviour of dump()<br class=""></blockquote><br class="">There's javascript.options.showInConsole, too, but I'm not sure it does<br class="">anything these days, despite being defined in all.js, firefox.js and<br class="">other places, it seems it was made a no-op by the error console removal<br class="">in bug 1278368.<br class=""><br class="">Mike<br class="">_______________________________________________<br class="">firefox-dev mailing list<br class=""><a href="mailto:firefox-dev@mozilla.org" class="">firefox-dev@mozilla.org</a><br class="">https://mail.mozilla.org/listinfo/firefox-dev<br class=""></blockquote></div></div></blockquote></div><br class=""></body></html>