V2 UI fails to show the data for the very first point in charts
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 6 Oct 2015 21:21:44 +0000 (21:21 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 6 Oct 2015 21:21:44 +0000 (21:21 +0000)
https://bugs.webkit.org/show_bug.cgi?id=149857

Reviewed by Chris Dumez.

The bug was caused by seriesBetweenPoints returning null for when point.seriesIndex is 0.
Explicitly check the type of this property instead.

* public/v2/data.js:
(TimeSeries.prototype.seriesBetweenPoints):

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

Websites/perf.webkit.org/ChangeLog
Websites/perf.webkit.org/public/v2/data.js

index 86da492..c3dbbca 100644 (file)
@@ -1,5 +1,18 @@
 2015-10-06  Ryosuke Niwa  <rniwa@webkit.org>
 
+        V2 UI fails to show the data for the very first point in charts
+        https://bugs.webkit.org/show_bug.cgi?id=149857
+
+        Reviewed by Chris Dumez.
+
+        The bug was caused by seriesBetweenPoints returning null for when point.seriesIndex is 0.
+        Explicitly check the type of this property instead.
+
+        * public/v2/data.js:
+        (TimeSeries.prototype.seriesBetweenPoints):
+
+2015-10-06  Ryosuke Niwa  <rniwa@webkit.org>
+
         Perf dashboard should have the capability to test local UI with production data
         https://bugs.webkit.org/show_bug.cgi?id=149834
 
index 5e2e185..665e395 100644 (file)
@@ -477,7 +477,7 @@ TimeSeries.prototype.findPointAfterTime = function (time)
 
 TimeSeries.prototype.seriesBetweenPoints = function (startPoint, endPoint)
 {
-    if (!startPoint.seriesIndex || !endPoint.seriesIndex)
+    if (typeof(startPoint.seriesIndex) != "number" || typeof(endPoint.seriesIndex) != "number")
         return null;
     return this._series.slice(startPoint.seriesIndex, endPoint.seriesIndex + 1);
 }