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:

Version:

528+ (Nightly build)

Hardware:

OS:

OS X 10.5

Attachments:

Description	Flags
patch	levin: review+

Nate Chapin

Reported 2010-10-04 16:35:14 PDT

CachedResource::New and CachedResource::NotCached are not in use.

Attachments
patch (2.18 KB, patch) 2010-10-04 16:38 PDT, Nate Chapin	levin: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Nate Chapin

Comment 1 2010-10-04 16:38:57 PDT

Created attachment 69716 [details] patch

Xan Lopez

Comment 2 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.

WebKit Commit Bot

Comment 3 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

Nate Chapin

Comment 4 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

Note You need to log in before you can comment on or make changes to this bug.