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

Adam Barth
Reported 2009-12-21 01:35:35 PST
Delete the boring "Empty queue" status messages in QueueStatusServer. This should make the dashboard faster.
Attachments
Patch (2.42 KB, patch)
2009-12-21 01:36 PST, Adam Barth 		no flags
DetailsFormatted DiffDiff
Patch (4.72 KB, patch)
2009-12-21 01:38 PST, Adam Barth 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Adam Barth
Comment 1 2009-12-21 01:36:38 PST
Created attachment 45315 [details] Patch
WebKit Review Bot
Comment 2 2009-12-21 01:37:12 PST
style-queue ran check-webkit-style on attachment 45315 [details] without any errors.
Adam Barth
Comment 3 2009-12-21 01:38:10 PST
Created attachment 45316 [details] Patch
WebKit Review Bot
Comment 4 2009-12-21 01:42:32 PST
style-queue ran check-webkit-style on attachment 45316 [details] without any errors.
Eric Seidel (no email)
Comment 5 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?
Adam Barth
Comment 6 2009-12-21 11:32:18 PST
If you like, but why 6? Is there any value to the one beyond the first?
Eric Seidel (no email)
Comment 7 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.
WebKit Commit Bot
Comment 8 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>
WebKit Commit Bot
Comment 9 2009-12-21 12:19:23 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.