Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(731)

Unified Diff: third_party/WebKit/Source/devtools/front_end/audits/AuditRules.js

Issue 2560043005: DevTools: Remove unused variables. Disallow unused variables with eslint (Closed)
Patch Set: A new unused variable was born Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/devtools/front_end/audits/AuditRules.js
diff --git a/third_party/WebKit/Source/devtools/front_end/audits/AuditRules.js b/third_party/WebKit/Source/devtools/front_end/audits/AuditRules.js
index 8aec3f3b5f758fd3258efc3915611b1663090d9f..73c449b01806515bf835ef8f33d5da16d1cd93e2 100644
--- a/third_party/WebKit/Source/devtools/front_end/audits/AuditRules.js
+++ b/third_party/WebKit/Source/devtools/front_end/audits/AuditRules.js
@@ -84,8 +84,6 @@ Audits.AuditRules.GzipRule = class extends Audits.AuditRule {
*/
doRun(target, requests, result, callback, progress) {
var totalSavings = 0;
- var compressedSize = 0;
- var candidateSize = 0;
var summary = result.addChild('', true);
for (var i = 0, length = requests.length; i < length; ++i) {
var request = requests[i];
@@ -93,11 +91,8 @@ Audits.AuditRules.GzipRule = class extends Audits.AuditRule {
continue; // Do not test cached resources.
if (this._shouldCompress(request)) {
var size = request.resourceSize;
- candidateSize += size;
- if (this._isCompressed(request)) {
- compressedSize += size;
+ if (this._isCompressed(request))
continue;
- }
var savings = 2 * size / 3;
totalSavings += savings;
summary.addFormatted('%r could save ~%s', request.url, Number.bytesToString(savings));

Powered by Google App Engine
This is Rietveld 408576698