Bug 195690 – Web Inspector: Console: add a setting for controlling auto-expansion of Trace messages

• Component: Web Inspector
• Status: RESOLVED FIXED

[Devin Rousso]

If a lot of `console.trace` calls are being made rapidly, it can "spam" the log with a lot of information that the developer would have to sift through, especially if all the traces auto-expand.

[Devin Rousso]

Created attachment 364558 [details]
Patch

[Radar WebKit Bug Importer]

<rdar://problem/48917329>

[Joseph Pecoraro]

Comment on attachment 364558 [details]
Patch

Hmm, I'm not sure this deserves a setting. Do we have a good reason for it (other then our Inspector protocol traffic logging?)

[Devin Rousso]

(In reply to Joseph Pecoraro from comment #3)
> Comment on attachment 364558 [details]
> Patch
> 
> Hmm, I'm not sure this deserves a setting. Do we have a good reason for it
> (other then our Inspector protocol traffic logging?)
I don't have a "good" reason, but I have run into situations before where when running `console.trace(a, b)` a LOT of times, I'll only really care about ones with a particular value for `a` or `b`, and having to scroll/filter is a bit annoying.  This solves the issue quite nicely :)

I would also prefer it to not be expanded by default.  Rather than making that choice for everyone, a setting lets each person choose for themself.

[Timothy Hatcher]

I could see this being useful. Would be good to know how often many of these settings get used in the wild, so we can cull the unused ones or ones always enabled.

[WebKit Commit Bot]

Comment on attachment 364558 [details]
Patch

Clearing flags on attachment: 364558

Committed r243220: <https://trac.webkit.org/changeset/243220>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.