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

Unified Diff: tracing/tracing/extras/vsync/vsync_auditor.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/extras/vsync/vsync_auditor.html
diff --git a/tracing/tracing/extras/vsync/vsync_auditor.html b/tracing/tracing/extras/vsync/vsync_auditor.html
index 8076cf48b2b15892c09f161ca7cc26fb4848e80d..68a4aa8ef043d0ec8c422289649f7e05d25c8446 100644
--- a/tracing/tracing/extras/vsync/vsync_auditor.html
+++ b/tracing/tracing/extras/vsync/vsync_auditor.html
@@ -75,11 +75,9 @@ tr.exportTo('tr.e.audits', function() {
function useInstead(title, precisions) {
var precision = precisions[title];
- if (precision === undefined)
- return false;
+ if (precision === undefined) return false;
- if (title === maxTitle)
- return true;
+ if (title === maxTitle) return true;
if (precision <= maxPrecision) {
if (precision === maxPrecision) {
@@ -118,14 +116,16 @@ tr.exportTo('tr.e.audits', function() {
var thread = process.threads[tid];
for (var i = 0; i < thread.sliceGroup.slices.length; i++) {
var slice = thread.sliceGroup.slices[i];
- if (useInstead(slice.title, VSYNC_SLICE_PRECISIONS))
+ if (useInstead(slice.title, VSYNC_SLICE_PRECISIONS)) {
times.push(slice.start);
- // We need to check not only that we have a Scheduler::BeginFrame
- // event, but also that we have one that has a frame time associated
- // with it. Older versions of Scheduler::BeginFrame don't have one.
- else if (useInstead(slice.title, BEGIN_FRAME_SLICE_PRECISION) &&
- slice.args.args && slice.args.args.frame_time_us)
+ } else if (useInstead(slice.title, BEGIN_FRAME_SLICE_PRECISION) &&
+ slice.args.args && slice.args.args.frame_time_us) {
+ // We need to check not only that we have a Scheduler::BeginFrame
+ // event, but also that we have one that has a frame time
+ // associated with it.
+ // Older versions of Scheduler::BeginFrame don't have one.
times.push(slice.args.args.frame_time_us / 1000.0);
+ }
}
}
}
« no previous file with comments | « tracing/tracing/extras/tquery/tquery_test.html ('k') | tracing/tracing/extras/vsync/vsync_auditor_test.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698