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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: tracing/tracing/metrics/v8/gc_metric.html

Issue 2776653002: [ESLint] Fix violations when enabling curly rule in eslint. (Closed)
Patch Set: Fix test 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
« dashboard/dashboard/elements/alerts-table.html ('K') | « tracing/tracing/metrics/system_health/webview_startup_metric.html ('k') | tracing/tracing/metrics/v8/utils.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tracing/tracing/metrics/v8/gc_metric.html
diff --git a/tracing/tracing/metrics/v8/gc_metric.html b/tracing/tracing/metrics/v8/gc_metric.html
index 377997a095c6f47d4dad000260099b4e2e151809..dcff1522789b9ea58b702862d4614f553a5b6cf8 100644
--- a/tracing/tracing/metrics/v8/gc_metric.html
+++ b/tracing/tracing/metrics/v8/gc_metric.html
@@ -89,10 +89,11 @@ tr.exportTo('tr.metrics.v8', function() {
function createPercentage(name, numerator, denominator, unit) {
var hist = new tr.v.Histogram(name, unit);
- if (denominator === 0)
+ if (denominator === 0) {
hist.addSample(0);
- else
+ } else {
hist.addSample(numerator / denominator);
+ }
hist.customizeSummaryOptions({
avg: true,
count: false,
« dashboard/dashboard/elements/alerts-table.html ('K') | « tracing/tracing/metrics/system_health/webview_startup_metric.html ('k') | tracing/tracing/metrics/v8/utils.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698