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

Unified Diff: tracing/tracing/importer/proto_expectation.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
« no previous file with comments | « tracing/tracing/importer/importer.html ('k') | tracing/tracing/importer/simple_line_reader.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tracing/tracing/importer/proto_expectation.html
diff --git a/tracing/tracing/importer/proto_expectation.html b/tracing/tracing/importer/proto_expectation.html
index 939fad19553b4d41d3bfb6379335e8d9da580309..9c6ff51a3454e23b13767f3f12236de03421308f 100644
--- a/tracing/tracing/importer/proto_expectation.html
+++ b/tracing/tracing/importer/proto_expectation.html
@@ -57,8 +57,9 @@ tr.exportTo('tr.importer', function() {
];
function combineInitiatorTypes(title1, title2) {
- for (var item of INITIATOR_HIERARCHY)
+ for (var item of INITIATOR_HIERARCHY) {
if (title1 === item || title2 === item) return item;
+ }
throw new Error('Invalid titles in combineInitiatorTypes');
}
@@ -101,8 +102,7 @@ tr.exportTo('tr.importer', function() {
model, this.initiatorType, this.start, duration);
break;
}
- if (!ir)
- return undefined;
+ if (!ir) return undefined;
ir.sourceEvents.addEventSet(this.associatedEvents);
@@ -112,16 +112,18 @@ tr.exportTo('tr.importer', function() {
// |event| is either an InputLatencyAsyncSlice (which collects all of
// its associated events transitively) or a CSS Animation (which doesn't
// have any associated events). So this does not need to recurse.
- if (event.associatedEvents)
+ if (event.associatedEvents) {
ir.associatedEvents.addEventSet(event.associatedEvents);
+ }
}
this.associatedEvents.forEach(function(event) {
pushAssociatedEvents(event);
// Old-style InputLatencyAsyncSlices have subSlices.
- if (event.subSlices)
+ if (event.subSlices) {
event.subSlices.forEach(pushAssociatedEvents);
+ }
});
return ir;
@@ -139,8 +141,9 @@ tr.exportTo('tr.importer', function() {
this.associatedEvents.addEventSet(other.associatedEvents);
this.start = Math.min(this.start, other.start);
this.end = Math.max(this.end, other.end);
- if (other.isAnimationBegin)
+ if (other.isAnimationBegin) {
this.isAnimationBegin = true;
+ }
},
// Include |event| in this ProtoExpectation, expanding start/end to include
« no previous file with comments | « tracing/tracing/importer/importer.html ('k') | tracing/tracing/importer/simple_line_reader.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698