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

Unified Diff: tracing/tracing/ui/extras/chrome/cc/picture_ops_chart_summary_view.html

Issue 2776653002: [ESLint] Fix violations when enabling curly rule in eslint. (Closed)
Patch Set: rebase Created 3 years, 9 months 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: tracing/tracing/ui/extras/chrome/cc/picture_ops_chart_summary_view.html
diff --git a/tracing/tracing/ui/extras/chrome/cc/picture_ops_chart_summary_view.html b/tracing/tracing/ui/extras/chrome/cc/picture_ops_chart_summary_view.html
index 15a2abc23fd6d6cbe596652e40de937d197af0f2..b4c0fe001d42bc6ed081edc53b48990632457aaa 100644
--- a/tracing/tracing/ui/extras/chrome/cc/picture_ops_chart_summary_view.html
+++ b/tracing/tracing/ui/extras/chrome/cc/picture_ops_chart_summary_view.html
@@ -78,8 +78,7 @@ tr.exportTo('tr.ui.e.chrome.cc', function() {
this.picture_ = picture;
this.pictureDataProcessed_ = false;
- if (Polymer.dom(this).classList.contains('hidden'))
- return;
+ if (Polymer.dom(this).classList.contains('hidden')) return;
this.processPictureData_();
this.requiresRedraw = true;
@@ -93,8 +92,7 @@ tr.exportTo('tr.ui.e.chrome.cc', function() {
show: function() {
Polymer.dom(this).classList.remove('hidden');
- if (this.pictureDataProcessed_)
- return;
+ if (this.pictureDataProcessed_) return;
this.processPictureData_();
this.requiresRedraw = true;
@@ -122,15 +120,15 @@ tr.exportTo('tr.ui.e.chrome.cc', function() {
this.currentBarMouseOverTarget_, 0, this.opsTimingData_.length - 1);
}
- if (this.currentBarMouseOverTarget_ === lastBarMouseOverTarget)
- return;
+ if (this.currentBarMouseOverTarget_ === lastBarMouseOverTarget) return;
this.drawChartContents_();
},
updateChartContents: function() {
- if (this.requiresRedraw)
+ if (this.requiresRedraw) {
this.updateChartDimensions_();
+ }
this.drawChartContents_();
},
@@ -155,18 +153,15 @@ tr.exportTo('tr.ui.e.chrome.cc', function() {
this.resetOpsTimingData_();
this.pictureDataProcessed_ = true;
- if (!this.picture_)
- return;
+ if (!this.picture_) return;
var ops = this.picture_.getOps();
- if (!ops)
- return;
+ if (!ops) return;
ops = this.picture_.tagOpsWithTimings(ops);
// Check that there are valid times.
- if (ops[0].cmd_time === undefined)
- return;
+ if (ops[0].cmd_time === undefined) return;
this.collapseOpsToTimingBuckets_(ops);
},
@@ -183,8 +178,7 @@ tr.exportTo('tr.ui.e.chrome.cc', function() {
this.drawBars_();
this.drawLineAtBottomOfChart_();
- if (this.currentBarMouseOverTarget_ === null)
- return;
+ if (this.currentBarMouseOverTarget_ === null) return;
this.drawTooltip_();
},
@@ -391,8 +385,7 @@ tr.exportTo('tr.ui.e.chrome.cc', function() {
for (var i = 0; i < ops.length; i++) {
op = ops[i];
- if (op.cmd_time === undefined)
- continue;
+ if (op.cmd_time === undefined) continue;
// Try to locate the entry for the current operation
// based on its name. If that fails, then create one for it.
@@ -421,8 +414,7 @@ tr.exportTo('tr.ui.e.chrome.cc', function() {
var otherSource = timingData.splice(count, timingData.length - count);
var otherDestination = null;
- if (!otherSource.length)
- return;
+ if (!otherSource.length) return;
timingData.push({
cmd_time: 0,

Powered by Google App Engine
This is Rietveld 408576698