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

Unified Diff: tracing/tracing/model/helpers/chrome_process_helper.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/model/helpers/chrome_process_helper.html
diff --git a/tracing/tracing/model/helpers/chrome_process_helper.html b/tracing/tracing/model/helpers/chrome_process_helper.html
index 7f3736d0f9c7284099480ed2badcf5825f910ec2..dc5d93427d2073f9ceb534c1b425c11f5555bd13 100644
--- a/tracing/tracing/model/helpers/chrome_process_helper.html
+++ b/tracing/tracing/model/helpers/chrome_process_helper.html
@@ -27,8 +27,9 @@ tr.exportTo('tr.model.helpers', function() {
for (var i = 0; i < slices.length; i++) {
var slice = slices[i];
if (rangeOfInterest.intersectsExplicitRangeInclusive(
- slice.start, slice.end))
+ slice.start, slice.end)) {
slicesInFilterRange.push(slice);
+ }
}
return slicesInFilterRange;
}
@@ -79,10 +80,13 @@ tr.exportTo('tr.model.helpers', function() {
MAIN_RENDERING_STATS : IMPL_RENDERING_STATS);
var frameEvents = [];
- for (var event of this.process.getDescendantEvents())
- if (event.title === titleToGet)
- if (range.intersectsExplicitRangeInclusive(event.start, event.end))
+ for (var event of this.process.getDescendantEvents()) {
+ if (event.title === titleToGet) {
+ if (range.intersectsExplicitRangeInclusive(event.start, event.end)) {
frameEvents.push(event);
+ }
+ }
+ }
frameEvents.sort(function(a, b) {return a.start - b.start;});
return frameEvents;

Powered by Google App Engine
This is Rietveld 408576698