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

Unified Diff: tracing/tracing/model/clock_sync_manager.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/model/async_slice_group.html ('k') | tracing/tracing/model/container_memory_dump.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tracing/tracing/model/clock_sync_manager.html
diff --git a/tracing/tracing/model/clock_sync_manager.html b/tracing/tracing/model/clock_sync_manager.html
index 89e31e6ce6b1f91d5d61338503d7076a6ab4558b..68e0a63e753a096cb221805a3ba123b065419bbf 100644
--- a/tracing/tracing/model/clock_sync_manager.html
+++ b/tracing/tracing/model/clock_sync_manager.html
@@ -118,8 +118,9 @@ tr.exportTo('tr.model', function() {
'complete - cannot add a third clock sync marker to it.');
}
- if (markers[0].domainId === domainId)
+ if (markers[0].domainId === domainId) {
throw new Error('A clock domain cannot sync with itself.');
+ }
markers.push(marker);
this.onSyncCompleted_(markers[0], marker);
@@ -150,8 +151,9 @@ tr.exportTo('tr.model', function() {
getModelTimeTransformer: function(domainId) {
this.onDomainSeen_(domainId);
- if (!this.modelDomainId_)
+ if (!this.modelDomainId_) {
this.selectModelDomainId_();
+ }
return this.getTimeTransformerRaw_(domainId, this.modelDomainId_).fn;
},
@@ -201,18 +203,19 @@ tr.exportTo('tr.model', function() {
var current = queue.shift();
- if (current.domainId === toDomainId)
+ if (current.domainId === toDomainId) {
return current.transformer;
+ }
- if (visitedDomainIds.has(current.domainId))
+ if (visitedDomainIds.has(current.domainId)) {
continue;
+ }
visitedDomainIds.add(current.domainId);
var outgoingTransformers =
this.transformerMapByDomainId_[current.domainId];
- if (!outgoingTransformers)
- continue;
+ if (!outgoingTransformers) continue;
// Add all nodes that are directly connected to this one to the queue.
for (var outgoingDomainId in outgoingTransformers) {
@@ -295,8 +298,9 @@ tr.exportTo('tr.model', function() {
// one attached to UNKNOWN_CHROME_LEGACY and the other attached to the
// real Chrome clock domain.
for (var chromeDomainId of POSSIBLE_CHROME_CLOCK_DOMAINS) {
- if (chromeDomainId === ClockDomainId.UNKNOWN_CHROME_LEGACY)
+ if (chromeDomainId === ClockDomainId.UNKNOWN_CHROME_LEGACY) {
continue;
+ }
this.collapseDomains_(
ClockDomainId.UNKNOWN_CHROME_LEGACY, chromeDomainId);
@@ -337,8 +341,9 @@ tr.exportTo('tr.model', function() {
* how to transform timestamps between directly connected clock domains.
*/
getOrCreateTransformerMap_: function(domainId) {
- if (!this.transformerMapByDomainId_[domainId])
+ if (!this.transformerMapByDomainId_[domainId]) {
this.transformerMapByDomainId_[domainId] = {};
+ }
return this.transformerMapByDomainId_[domainId];
},
« no previous file with comments | « tracing/tracing/model/async_slice_group.html ('k') | tracing/tracing/model/container_memory_dump.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698