<div dir="ltr"><div>Doing so would make it impossible to see any CSS warnings that come from styles applied during startup (unless the browser console is opened before the first window via a command line flag).</div><div><br></div><div>When I look at the browser console, it appears that content CSS warnings comprise orders of magnitude more messages than the chrome CSS warnings. Would the filtering (if it could be done on the content side) make enough of a reduction to remove the perf issue?<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 11, 2018 at 3:07 PM, Brian Grinstead <span dir="ltr"><<a href="mailto:bgrinstead@mozilla.com" target="_blank">bgrinstead@mozilla.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The content filtering in the Browser Console is pretty close - we have one more platform bug in progress to detect messages coming from chrome contexts and then a relatively small piece of frontend work.<br>
<br>
It sounds like you’d be OK with not ever seeing CSS warnings from content pages in the BC, and that you may or may not care about CSS warnings from the parent process but don’t know yet until we can filter out content messages. Would you care if turning on CSS warnings in the parent process didn’t show ‘past’ CSS warnings but started showing only new ones?<br>
<span class="HOEnZb"><font color="#888888"><br>
Brian<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
> On Apr 11, 2018, at 12:00 PM, Jared Wein <<a href="mailto:jaws@mozilla.com">jaws@mozilla.com</a>> wrote:<br>
><br>
> I agree with Dan. <a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1260877" rel="noreferrer" target="_blank">https://bugzilla.mozilla.org/<wbr>show_bug.cgi?id=1260877</a> looked close to landing a month ago then I think work stalled?<br>
><br>
> - Jared<br>
><br>
> On Wed, Apr 11, 2018 at 2:57 PM, Dan Mosedale <<a href="mailto:dmosedale@mozilla.com">dmosedale@mozilla.com</a>> wrote:<br>
> Because it's so noisy, presumably in the current form people aren't getting a lot of use out of them.  However, I suspect that could change substantially if we stopped showing content CSS errors in the browser console.  So it seems a bit hard to evaluate the usefulness without doing that first.<br>
><br>
> Dan<br>
><br>
> 2018-04-11 11:21 GMT-07:00 Brian Grinstead <<a href="mailto:bgrinstead@mozilla.com">bgrinstead@mozilla.com</a>>:<br>
> The console has a ‘CSS’ filter option that shows CSS warnings / errors. It’s off by default, but when you flip it on you will see warnings like "Unknown property ‘user-select’. Declaration dropped.”. The problem is that we pay a perf penalty for reporting the errors, and they aren’t shown by default. There’s a plan in Bug 1452143 to change that so we would only start reporting the errors once the filter is toggled, but dealing with the Browser Console will require extra work since it’s receiving messages from all windows (including those in the content processes)<br>
><br>
> The main options I thought of for dealing with this are:<br>
>  (1) Loop through all the windows when the filter is toggled and reparse all stylesheets to trigger errors that happened before we were reporting them. We could simplify this by never showing CSS warnings for content pages in the Browser Console.<br>
>  (2) Remove the button entirely and just not ever show any CSS warnings in the Browser Console<br>
>  (3) Allow you to press the button but don't attempt to dynamically begin showing errors. Instead send a message to the Browser Console saying 'restart your browser to start seeing css warnings’<br>
><br>
> If you use this feature and find it valuable, please let me know. I’d prefer to go with (2) if people aren’t getting value out of it. I don’t have them on myself because it causes a huge amount of noise (you see all warnings from all pages plus the parent process).<br>
><br>
> Thanks,<br>
> Brian<br>
> ______________________________<wbr>_________________<br>
> firefox-dev mailing list<br>
> <a href="mailto:firefox-dev@mozilla.org">firefox-dev@mozilla.org</a><br>
> <a href="https://mail.mozilla.org/listinfo/firefox-dev" rel="noreferrer" target="_blank">https://mail.mozilla.org/<wbr>listinfo/firefox-dev</a><br>
><br>
><br>
> ______________________________<wbr>_________________<br>
> firefox-dev mailing list<br>
> <a href="mailto:firefox-dev@mozilla.org">firefox-dev@mozilla.org</a><br>
> <a href="https://mail.mozilla.org/listinfo/firefox-dev" rel="noreferrer" target="_blank">https://mail.mozilla.org/<wbr>listinfo/firefox-dev</a><br>
><br>
><br>
<br>
</div></div></blockquote></div><br></div>