Unreviewed build fixes in TestGroupResultsViewer.
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 18 Jan 2018 06:45:31 +0000 (06:45 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 18 Jan 2018 06:45:31 +0000 (06:45 +0000)
* public/v3/components/test-group-results-viewer.js:
(TestGroupResultsViewer.prototype.render): _renderCurrentMetricsLazily was never called.
(TestGroupResultsViewer.prototype._buildRowForMetric): Don't try to render null or NaN.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@227125 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Websites/perf.webkit.org/ChangeLog
Websites/perf.webkit.org/public/v3/components/test-group-results-viewer.js

index c554eea..0d9a170 100644 (file)
@@ -1,5 +1,13 @@
 2018-01-17  Ryosuke Niwa  <rniwa@webkit.org>
 
+        Unreviewed build fixes in TestGroupResultsViewer.
+
+        * public/v3/components/test-group-results-viewer.js:
+        (TestGroupResultsViewer.prototype.render): _renderCurrentMetricsLazily was never called.
+        (TestGroupResultsViewer.prototype._buildRowForMetric): Don't try to render null or NaN.
+
+2018-01-17  Ryosuke Niwa  <rniwa@webkit.org>
+
         Fix perf dashboard tests for node v8
         https://bugs.webkit.org/show_bug.cgi?id=181782
 
index 20c9607..4b2feac 100644 (file)
@@ -40,6 +40,7 @@ class TestGroupResultsViewer extends ComponentBase {
 
         this._renderResultsTableLazily.evaluate(this._testGroup, this._expandedTests,
             ...this._analysisResults.topLevelTestsForTestGroup(this._testGroup));
+        this._renderCurrentMetricsLazily.evaluate(this._currentMetric);
     }
 
     _renderResultsTable(testGroup, expandedTests, ...tests)
@@ -99,7 +100,10 @@ class TestGroupResultsViewer extends ComponentBase {
         const deltaFormatter = metric.makeFormatter(2, false);
         const formatValue = (value, interval) => {
             const delta = interval ? (interval[1] - interval[0]) / 2 : null;
-            return value == null || isNaN(value) ? '-' : `${formatter(value)} \u00b1 ${deltaFormatter(delta)}`;
+            let result = value == null || isNaN(value) ? '-' : formatter(value);
+            if (delta != null && !isNaN(delta))
+                result += ` \u00b1 ${deltaFormatter(delta)}`;
+            return result;
         }
 
         const barGroup = new BarGraphGroup();