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

Unified Diff: tracing/tracing/extras/chrome/layout_object.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/extras/chrome/gpu/state.html ('k') | tracing/tracing/extras/chrome/slice_title_fixer.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tracing/tracing/extras/chrome/layout_object.html
diff --git a/tracing/tracing/extras/chrome/layout_object.html b/tracing/tracing/extras/chrome/layout_object.html
index e9528ab243288c9d8340d77263cff806860f8cd8..d18e925b8b6b73c0e0e56ec2e6939376a8bfa907 100644
--- a/tracing/tracing/extras/chrome/layout_object.html
+++ b/tracing/tracing/extras/chrome/layout_object.html
@@ -58,14 +58,18 @@ tr.exportTo('tr.e.chrome', function() {
this.htmlId_ = args.htmlId;
this.classNames_ = args.classNames;
this.needsLayoutReasons_ = [];
- if (args.selfNeeds)
+ if (args.selfNeeds) {
this.needsLayoutReasons_.push('self');
- if (args.childNeeds)
+ }
+ if (args.childNeeds) {
this.needsLayoutReasons_.push('child');
- if (args.posChildNeeds)
+ }
+ if (args.posChildNeeds) {
this.needsLayoutReasons_.push('positionedChild');
- if (args.positionedMovement)
+ }
+ if (args.positionedMovement) {
this.needsLayoutReasons_.push('positionedMovement');
+ }
this.tableRow_ = args.row;
this.tableCol_ = args.col;
this.tableRowSpan_ = args.rowSpan;
@@ -78,8 +82,9 @@ tr.exportTo('tr.e.chrome', function() {
}
for (var property in args) {
- if (!KNOWN_PROPERTIES[property])
+ if (!KNOWN_PROPERTIES[property]) {
this.otherProperties_[property] = args[property];
+ }
}
}
@@ -166,8 +171,7 @@ tr.exportTo('tr.e.chrome', function() {
traverseTree: function(cb, opt_this) {
cb.call(opt_this, this);
- if (!this.hasChildLayoutObjects)
- return;
+ if (!this.hasChildLayoutObjects) return;
this.childLayoutObjects.forEach(function(child) {
child.traverseTree(cb, opt_this);
});
@@ -186,14 +190,12 @@ tr.exportTo('tr.e.chrome', function() {
},
get previousSnapshotLayoutObject() {
- if (!this.snapshot.previousSnapshot)
- return undefined;
+ if (!this.snapshot.previousSnapshot) return undefined;
return this.snapshot.previousSnapshot.getLayoutObjectById(this.id);
},
get nextSnapshotLayoutObject() {
- if (!this.snapshot.nextSnapshot)
- return undefined;
+ if (!this.snapshot.nextSnapshot) return undefined;
return this.snapshot.nextSnapshot.getLayoutObjectById(this.id);
}
};
« no previous file with comments | « tracing/tracing/extras/chrome/gpu/state.html ('k') | tracing/tracing/extras/chrome/slice_title_fixer.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698