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

Unified Diff: tracing/tracing/ui/base/timing_tool.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/ui/base/table_test.html ('k') | tracing/tracing/ui/base/ui.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tracing/tracing/ui/base/timing_tool.html
diff --git a/tracing/tracing/ui/base/timing_tool.html b/tracing/tracing/ui/base/timing_tool.html
index 66652a8b970221901f44c0d85afe12711ff2d594..517712b29e52886ee564a634a5d358fc461d8479 100644
--- a/tracing/tracing/ui/base/timing_tool.html
+++ b/tracing/tracing/ui/base/timing_tool.html
@@ -42,8 +42,9 @@ tr.exportTo('tr.ui.b', function() {
},
onBeginTiming: function(e) {
- if (!this.isTouchPointInsideTrackBounds_(e.clientX, e.clientY))
+ if (!this.isTouchPointInsideTrackBounds_(e.clientX, e.clientY)) {
return;
+ }
var pt = this.getSnappedToEventPosition_(e);
this.mouseDownAt_(pt.x, pt.y);
@@ -52,13 +53,15 @@ tr.exportTo('tr.ui.b', function() {
},
updateSnapIndicators_: function(pt) {
- if (!pt.snapped)
- return;
+ if (!pt.snapped) return;
+
var ir = this.viewport_.interestRange;
- if (ir.min === pt.x)
+ if (ir.min === pt.x) {
ir.leftSnapIndicator = new tr.ui.SnapIndicator(pt.y, pt.height);
- if (ir.max === pt.x)
+ }
+ if (ir.max === pt.x) {
ir.rightSnapIndicator = new tr.ui.SnapIndicator(pt.y, pt.height);
+ }
},
onUpdateTiming: function(e) {
@@ -77,8 +80,8 @@ tr.exportTo('tr.ui.b', function() {
},
onMouseMove_: function(e) {
- if (e.button)
- return;
+ if (e.button) return;
+
var worldX = this.getWorldXFromEvent_(e);
this.mouseMoveAt_(worldX, e.clientY, false);
},
@@ -92,14 +95,16 @@ tr.exportTo('tr.ui.b', function() {
isTouchPointInsideTrackBounds_: function(clientX, clientY) {
if (!this.viewport_ ||
!this.viewport_.modelTrackContainer ||
- !this.viewport_.modelTrackContainer.canvas)
+ !this.viewport_.modelTrackContainer.canvas) {
return false;
+ }
var canvas = this.viewport_.modelTrackContainer.canvas;
var canvasRect = canvas.getBoundingClientRect();
if (clientX >= canvasRect.left && clientX <= canvasRect.right &&
- clientY >= canvasRect.top && clientY <= canvasRect.bottom)
+ clientY >= canvasRect.top && clientY <= canvasRect.bottom) {
return true;
+ }
return false;
},
@@ -177,15 +182,17 @@ tr.exportTo('tr.ui.b', function() {
var ir = this.viewport_.interestRange;
var a = ir.min;
var b = ir.max;
- if (this.isMovingLeftEdge_)
+ if (this.isMovingLeftEdge_) {
a = newWorldX;
- else
+ } else {
b = newWorldX;
+ }
- if (a <= b)
+ if (a <= b) {
ir.setMinAndMax(a, b);
- else
+ } else {
ir.setMinAndMax(b, a);
+ }
if (ir.min === newWorldX) {
this.isMovingLeftEdge_ = true;
@@ -207,8 +214,9 @@ tr.exportTo('tr.ui.b', function() {
var pixelRatio = window.devicePixelRatio || 1;
var minWidthValue = dt.xViewVectorToWorld(2 * pixelRatio);
- if (ir.range < minWidthValue)
+ if (ir.range < minWidthValue) {
ir.reset();
+ }
},
getWorldXFromEvent_: function(e) {
« no previous file with comments | « tracing/tracing/ui/base/table_test.html ('k') | tracing/tracing/ui/base/ui.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698