Bug 28616

Summary:

Inspector: Remove Unused Variable

Product:

WebKit

Reporter:

Joseph Pecoraro <joepeck>

Component:

Web Inspector (Deprecated)

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

aroben, eric, joepeck, kmccullough, timothy

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
[PATCH] Removes the Unused Variable	eric: review+, eric: commit-queue-

Joseph Pecoraro

Reported 2009-08-21 10:28:03 PDT

Fun fact, this has been around since Resource.js was first added! http://trac.webkit.org/browser/trunk/WebCore/page/inspector/Resource.js?rev=23677

Attachments
[PATCH] Removes the Unused Variable (1.05 KB, patch) 2009-08-21 10:29 PDT, Joseph Pecoraro	eric: review+ eric: commit-queue-	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Joseph Pecoraro

Comment 1 2009-08-21 10:29:20 PDT

Created attachment 38379 [details] [PATCH] Removes the Unused Variable

Eric Seidel (no email)

Comment 2 2009-08-21 10:51:25 PDT

Comment on attachment 38379 [details] [PATCH] Removes the Unused Variable OK. The commit-queue would probably barf on this diff, so cq-. I'm working on a pre-commit hook to make the all of our tools safe against this kind of failed diff.

Eric Seidel (no email)

Comment 3 2009-08-21 10:51:42 PDT

I mean specifically the ChangeLog diff.

Joseph Pecoraro

Comment 4 2009-08-21 11:23:04 PDT

Landed in r47633 = fdc58c1d115ce742689311039f033a230a8d6c0c http://trac.webkit.org/changeset/47633

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