Bug 32818

Summary: Delete the boring "Empty queue" status messages in QueueStatusServer
Product: WebKit Reporter: Adam Barth <abarth>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, eric, evan, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
none
Patch none

Description Adam Barth 2009-12-21 01:35:35 PST
Delete the boring "Empty queue" status messages in QueueStatusServer.  This should make the dashboard faster.
Comment 1 Adam Barth 2009-12-21 01:36:38 PST
Created attachment 45315 [details]
Patch
Comment 2 WebKit Review Bot 2009-12-21 01:37:12 PST
style-queue ran check-webkit-style on attachment 45315 [details] without any errors.
Comment 3 Adam Barth 2009-12-21 01:38:10 PST
Created attachment 45316 [details]
Patch
Comment 4 WebKit Review Bot 2009-12-21 01:42:32 PST
style-queue ran check-webkit-style on attachment 45316 [details] without any errors.
Comment 5 Eric Seidel 2009-12-21 09:29:09 PST
Comment on attachment 45316 [details]
Patch

Shouldn't it delete ones older than the last 6 or something?
Comment 6 Adam Barth 2009-12-21 11:32:18 PST
If you like, but why 6?  Is there any value to the one beyond the first?
Comment 7 Eric Seidel 2009-12-21 11:38:26 PST
Comment on attachment 45316 [details]
Patch

I've decided I was wrong.  We should just kill them all.  There are better ways to track this information than with these silly messages.
Comment 8 WebKit Commit Bot 2009-12-21 12:19:17 PST
Comment on attachment 45316 [details]
Patch

Clearing flags on attachment: 45316

Committed r52453: <http://trac.webkit.org/changeset/52453>
Comment 9 WebKit Commit Bot 2009-12-21 12:19:23 PST
All reviewed patches have been landed.  Closing bug.