Test freshness page should use build time instead of commit time to determine the...
authordewei_zhu@apple.com <dewei_zhu@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 27 Dec 2017 22:55:43 +0000 (22:55 +0000)
committerdewei_zhu@apple.com <dewei_zhu@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 27 Dec 2017 22:55:43 +0000 (22:55 +0000)
https://bugs.webkit.org/show_bug.cgi?id=181156

Reviewed by Alexey Proskuryakov.

Build time is a better data point freshness indicator. Test freshness page is designed to highlight test failures.
Using commit time will result in including the compiling and test running time which does not quite match the
definition of data point fresshness.

* public/v3/pages/test-freshness-page.js:
(TestFreshnessPage.prototype._fetchTestResults): Use build time instead of commit time.

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

Websites/perf.webkit.org/ChangeLog
Websites/perf.webkit.org/public/v3/pages/test-freshness-page.js

index 7fcb997..f66c437 100644 (file)
@@ -1,3 +1,17 @@
+2017-12-26  Dewei Zhu  <dewei_zhu@apple.com>
+
+        Test freshness page should use build time instead of commit time to determine the freshness of the data point.
+        https://bugs.webkit.org/show_bug.cgi?id=181156
+
+        Reviewed by Alexey Proskuryakov.
+
+        Build time is a better data point freshness indicator. Test freshness page is designed to highlight test failures.
+        Using commit time will result in including the compiling and test running time which does not quite match the
+        definition of data point fresshness.
+
+        * public/v3/pages/test-freshness-page.js:
+        (TestFreshnessPage.prototype._fetchTestResults): Use build time instead of commit time.
+
 2017-12-21  Dewei Zhu  <dewei_zhu@apple.com>
 
         Add UI for A/B testing on owned commits.
index 781c3f2..bd50697 100644 (file)
@@ -72,7 +72,7 @@ class TestFreshnessPage extends PageWithHeading {
 
                     let timeForLastDataPoint = startTime;
                     if (currentTimeSeries.lastPoint())
-                        timeForLastDataPoint = currentTimeSeries.lastPoint().time;
+                        timeForLastDataPoint = currentTimeSeries.lastPoint().build().buildTime();
 
                     lastDataPointByMetric.set(metric, {time: timeForLastDataPoint, hasCurrentDataPoint: !!currentTimeSeries.lastPoint()});
                     this.enqueueToRender();