Changeset 179765 in webkit

Timestamp:

Feb 6, 2015, 2:55:57 PM (12 years ago)

Author:

rniwa@webkit.org

Message:

Unreviewed build fix. currentPoint wasn't defined when selectedPoints was used to find points.

• public/v2/app.js:

(App.PaneController._updateDetails):

Location:

trunk/Websites/perf.webkit.org

Files:

• ChangeLog (modified) (1 diff)
• public/v2/app.js (modified) (2 diffs)

trunk/Websites/perf.webkit.org/ChangeLog

@@ +1 @@
+2015-02-06  Ryosuke Niwa  <rniwa@webkit.org>
+
+        Unreviewed build fix. currentPoint wasn't defined when selectedPoints was used to find points.
+
+        * public/v2/app.js:
+        (App.PaneController._updateDetails):
+
 2015-02-06  Ryosuke Niwa  <rniwa@webkit.org>
 

trunk/Websites/perf.webkit.org/public/v2/app.js

@@ -804 +804 @@
         var currentMeasurement;
         var previousPoint;
-        if (currentPoint) {
-            currentMeasurement = currentPoint.measurement;
+        if (!selectedPoints)
             previousPoint = currentPoint.series.previousPoint(currentPoint);
-        } else {
-            currentMeasurement = selectedPoints[selectedPoints.length - 1].measurement;
+        else {
+            currentPoint = selectedPoints[selectedPoints.length - 1];
             previousPoint = selectedPoints[0];
         }
+        var currentMeasurement = currentPoint.measurement;
         var oldMeasurement = previousPoint ? previousPoint.measurement : null;
 
@@ -828 +828 @@
         var buildNumber = null;
         var buildURL = null;
-        if (currentPoint) {
+        if (!selectedPoints) {
             buildNumber = currentMeasurement.buildNumber();
             var builder = App.Manifest.builder(currentMeasurement.builderId());