28616 2009-08-21 10:28:03 -0700 Inspector: Remove Unused Variable 2009-08-21 11:23:15 -0700 1 1 1 Unclassified WebKit Web Inspector (Deprecated) 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 0 joepeck webkit-unassigned aroben eric joepeck kmccullough timothy oldest_to_newest 141901 0 joepeck 2009-08-21 10:28:03 -0700 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 141902 1 38379 joepeck 2009-08-21 10:29:20 -0700 Created attachment 38379 [PATCH] Removes the Unused Variable 141904 2 38379 eric 2009-08-21 10:51:25 -0700 Comment on attachment 38379 [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. 141905 3 eric 2009-08-21 10:51:42 -0700 I mean specifically the ChangeLog diff. 141915 4 joepeck 2009-08-21 11:23:04 -0700 Landed in r47633 = fdc58c1d115ce742689311039f033a230a8d6c0c http://trac.webkit.org/changeset/47633 38379 2009-08-21 10:29:20 -0700 2009-08-21 10:51:25 -0700 [PATCH] Removes the Unused Variable patch.patch text/plain 1071 joepeck ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg MmU0NGQ3NC4uZjQwMmUxYiAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvV2Vi Q29yZS9DaGFuZ2VMb2cKQEAgLTIsNiArMiwxNiBAQAogCiAgICAgICAgIFJldmlld2VkIGJ5IE5P Qk9EWSAoT09QUyEpLgogCisgICAgICAgIEluc3BlY3RvcjogUmVtb3ZlIFVudXNlZCBWYXJpYWJs ZQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9Mjg2MTYK KworICAgICAgICAqIGluc3BlY3Rvci9mcm9udC1lbmQvUmVzb3VyY2UuanM6CisgICAgICAgIChX ZWJJbnNwZWN0b3IuUmVzb3VyY2UucHJvdG90eXBlLl9jaGVja1dhcm5pbmcpOgorCisyMDA5LTA4 LTIxICBKb3NlcGggUGVjb3Jhcm8gIDxqb2VwZWNrQHdlYmtpdC5vcmc+CisKKyAgICAgICAgUmV2 aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKICAgICAgICAgSW5zcGVjdG9yOiBDb25zb2xlIEFy cmF5IEZvcm1hdHRlciBTaG93cyBFeHRyYSBQcm9wZXJ0aWVzCiAgICAgICAgIGh0dHBzOi8vYnVn cy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yODYxNQogCmRpZmYgLS1naXQgYS9XZWJDb3Jl L2luc3BlY3Rvci9mcm9udC1lbmQvUmVzb3VyY2UuanMgYi9XZWJDb3JlL2luc3BlY3Rvci9mcm9u dC1lbmQvUmVzb3VyY2UuanMKaW5kZXggYmNiN2IyYS4uZGI1OTMyMCAxMDA2NDQKLS0tIGEvV2Vi Q29yZS9pbnNwZWN0b3IvZnJvbnQtZW5kL1Jlc291cmNlLmpzCisrKyBiL1dlYkNvcmUvaW5zcGVj dG9yL2Zyb250LWVuZC9SZXNvdXJjZS5qcwpAQCAtNTM4LDcgKzUzOCw2IEBAIFdlYkluc3BlY3Rv ci5SZXNvdXJjZS5wcm90b3R5cGUgPSB7CiAKICAgICBfY2hlY2tXYXJuaW5nOiBmdW5jdGlvbih3 YXJuaW5nKQogICAgIHsKLSAgICAgICAgdmFyIGFkZFdhcm5pbmcgPSBmYWxzZTsKICAgICAgICAg dmFyIG1zZzsKICAgICAgICAgc3dpdGNoICh3YXJuaW5nLmlkKSB7CiAgICAgICAgICAgICBjYXNl IFdlYkluc3BlY3Rvci5XYXJuaW5ncy5JbmNvcnJlY3RNSU1FVHlwZS5pZDoK