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

Unified Diff: tracing/tracing/ui/tracks/chart_series_y_axis.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/ui/tracks/chart_series_y_axis.html
diff --git a/tracing/tracing/ui/tracks/chart_series_y_axis.html b/tracing/tracing/ui/tracks/chart_series_y_axis.html
index 84d31348f648421a9888c14cc5bfd72c0dfb366b..5254db10df9029c9dfb110993fd830e5fe5ab385 100644
--- a/tracing/tracing/ui/tracks/chart_series_y_axis.html
+++ b/tracing/tracing/ui/tracks/chart_series_y_axis.html
@@ -28,10 +28,8 @@ tr.exportTo('tr.ui.tracks', function() {
function ChartSeriesYAxis(opt_min, opt_max) {
this.guid_ = tr.b.GUID.allocateSimple();
this.bounds = new tr.b.math.Range();
- if (opt_min !== undefined)
- this.bounds.addValue(opt_min);
- if (opt_max !== undefined)
- this.bounds.addValue(opt_max);
+ if (opt_min !== undefined) this.bounds.addValue(opt_min);
+ if (opt_max !== undefined) this.bounds.addValue(opt_max);
}
ChartSeriesYAxis.prototype = {
@@ -40,11 +38,11 @@ tr.exportTo('tr.ui.tracks', function() {
},
valueToUnitRange: function(value) {
- if (this.bounds.isEmpty)
+ if (this.bounds.isEmpty) {
throw new Error('Chart series y-axis bounds are empty');
+ }
var bounds = this.bounds;
- if (bounds.range === 0)
- return 0;
+ if (bounds.range === 0) return 0;
return (value - bounds.min) / bounds.range;
},
@@ -95,8 +93,7 @@ tr.exportTo('tr.ui.tracks', function() {
* false), this method will only modify the y-axis bounds if they are empty.
*/
autoSetFromRange: function(range, opt_config) {
- if (range.isEmpty)
- return;
+ if (range.isEmpty) return;
var bounds = this.bounds;
if (bounds.isEmpty) {
@@ -104,8 +101,7 @@ tr.exportTo('tr.ui.tracks', function() {
return;
}
- if (!opt_config)
- return;
+ if (!opt_config) return;
var useRangeMin = (opt_config.expandMin && range.min < bounds.min ||
opt_config.shrinkMin && range.min > bounds.min);
@@ -113,8 +109,7 @@ tr.exportTo('tr.ui.tracks', function() {
opt_config.shrinkMax && range.max < bounds.max);
// Neither bound is modified.
- if (!useRangeMin && !useRangeMax)
- return;
+ if (!useRangeMin && !useRangeMax) return;
// Both bounds are modified. Assuming the range argument is a valid
// range, no extra checks are necessary.
« no previous file with comments | « tracing/tracing/ui/tracks/chart_series_test.html ('k') | tracing/tracing/ui/tracks/chart_series_y_axis_test.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698