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 86da4922b06063d9b1450957ffdca61c34c03c17..c3dbbcae9a6b2ed3936462d64ddb081eda10015e 100644 (file)
@@ -1,3 +1,16 @@
+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
 2015-10-06  Ryosuke Niwa  <rniwa@webkit.org>
 
         Perf dashboard should have the capability to test local UI with production data
index 5e2e185ad87975311563a99c419a0b619ea64c55..665e395ac03aa2d3823b4e9dc0b31751e35574e0 100644 (file)
@@ -477,7 +477,7 @@ TimeSeries.prototype.findPointAfterTime = function (time)
 
 TimeSeries.prototype.seriesBetweenPoints = function (startPoint, endPoint)
 {
 
 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);
 }
         return null;
     return this._series.slice(startPoint.seriesIndex, endPoint.seriesIndex + 1);
 }