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

Unified Diff: tracing/tracing/ui/analysis/multi_event_sub_view.html

Issue 2776653002: [ESLint] Fix violations when enabling curly rule in eslint. (Closed)
Patch Set: Fix test 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/analysis/multi_event_sub_view.html
diff --git a/tracing/tracing/ui/analysis/multi_event_sub_view.html b/tracing/tracing/ui/analysis/multi_event_sub_view.html
index d7e494a70c6d07765c61962f5725cb458da26ea7..cd4216130a03cc3ccc704fce052b11cad9886449 100644
--- a/tracing/tracing/ui/analysis/multi_event_sub_view.html
+++ b/tracing/tracing/ui/analysis/multi_event_sub_view.html
@@ -80,8 +80,7 @@ tr.exportTo('tr.ui.analysis', function() {
var diagnostics = {};
for (var item of EVENT_FIELD) {
var fieldName = item.key;
- if (slice[fieldName] === undefined)
- continue;
+ if (slice[fieldName] === undefined) continue;
diagnostics[fieldName] = new tr.v.d.Scalar(new tr.b.Scalar(
tr.b.Unit.byName.timeDurationInMs, slice[fieldName]));
}
@@ -118,8 +117,9 @@ tr.exportTo('tr.ui.analysis', function() {
},
set selection(selection) {
- if (selection.length <= 1)
+ if (selection.length <= 1) {
throw new Error('Only supports multiple items');
+ }
this.setSelectionWithoutErrorChecks(selection);
},

Powered by Google App Engine
This is Rietveld 408576698