Bug 151225 - Web Inspector: Reduce list of saved console messages
Summary: Web Inspector: Reduce list of saved console messages
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Normal
Assignee: Joseph Pecoraro
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2015-11-12 14:30 PST by Joseph Pecoraro
Modified: 2015-11-12 15:27 PST (History)
13 users (show)

See Also:

Attachments
[PATCH] Proposed Fix (2.20 KB, patch)
2015-11-12 14:36 PST, Joseph Pecoraro 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Joseph Pecoraro 2015-11-12 14:30:14 PST 
* SUMMARY
Reduce list of saved console messages.

Most sites do not produce a very large number of console messages. But for pages that live for a long time and generate many errors over time, saving the last 1000 messages in case an inspector frontend opens can result in a lot of wasted memory. Likewise when an inspector first open sending all 1000 messages to the frontend results in a poor experience. Lets reduce this to 100.

When just growing the web and not developing a page, most warnings/errors accumulated are engine generated warnings/errors that themselves can be investigated to reduce noise and further lower the actual number of ConsoleMessages.
Comment 1 Radar WebKit Bug Importer 2015-11-12 14:31:17 PST 
<rdar://problem/23524129>
Comment 2 Joseph Pecoraro 2015-11-12 14:36:52 PST 
Created attachment 265435 [details]
[PATCH] Proposed Fix
Comment 3 Geoffrey Garen 2015-11-12 14:38:07 PST 
Comment on attachment 265435 [details]
[PATCH] Proposed Fix

r=me
Comment 4 WebKit Commit Bot 2015-11-12 15:27:13 PST 
Comment on attachment 265435 [details]
[PATCH] Proposed Fix

Clearing flags on attachment: 265435

Committed r192391: <http://trac.webkit.org/changeset/192391>
Comment 5 WebKit Commit Bot 2015-11-12 15:27:16 PST 
All reviewed patches have been landed.  Closing bug.