Bug 47132

Summary: Remove unused CachedResource::Status values
Product: WebKit Reporter: Nate Chapin <japhet>
Component: WebCore Misc.Assignee: Nate Chapin <japhet>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, commit-queue, xan.lopez
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Attachments:
Description Flags
patch levin: review+

Description Nate Chapin 2010-10-04 16:35:14 PDT 
CachedResource::New and CachedResource::NotCached are not in use.
Comment 1 Nate Chapin 2010-10-04 16:38:57 PDT 
Created attachment 69716 [details]
patch
Comment 2 Xan Lopez 2010-10-04 21:14:31 PDT 
Comment on attachment 69716 [details]
patch

Maybe you can change m_status to be unsigned m_status : 2; now.
Comment 3 WebKit Commit Bot 2010-10-05 01:04:15 PDT 
Comment on attachment 69716 [details]
patch

Rejecting patch 69716 from commit-queue.

Failed to run "['./WebKitTools/Scripts/webkit-patch', '--status-host=queues.webkit.org', 'build', '--no-clean', '--no-update', '--build', '--build-style=both', '--quiet']" exit_code: 1
ERROR: Working directory has local commits, pass --force-clean to continue.

Full output: http://queues.webkit.org/results/4147076
Comment 4 Nate Chapin 2010-10-05 11:25:34 PDT 
Comment on attachment 69716 [details]
patch

Clearing cq+, I just landed this http://trac.webkit.org/changeset/69125