Address Antti's review comment.

* public/v3/models/analysis-results.js:
(AnalysisResults.prototype.containsTest):

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

diff --git a/Websites/perf.webkit.org/ChangeLog b/Websites/perf.webkit.org/ChangeLog
index ffbd679..5a6e90d 100644 (file)
--- a/Websites/perf.webkit.org/ChangeLog
+++ b/Websites/perf.webkit.org/ChangeLog
@@ -1,5 +1,12 @@
 2017-07-10  Ryosuke Niwa  <rniwa@webkit.org>
 
+        Address Antti's review comment.
+
+        * public/v3/models/analysis-results.js:
+        (AnalysisResults.prototype.containsTest):
+
+2017-07-10  Ryosuke Niwa  <rniwa@webkit.org>
+
         A/B testing results page show results for the top-level tests instead of the one being analyzed
         https://bugs.webkit.org/show_bug.cgi?id=174304
 

diff --git a/Websites/perf.webkit.org/public/v3/models/analysis-results.js b/Websites/perf.webkit.org/public/v3/models/analysis-results.js
index bc395af..5104aa5 100644 (file)
--- a/Websites/perf.webkit.org/public/v3/models/analysis-results.js
+++ b/Websites/perf.webkit.org/public/v3/models/analysis-results.js
@@ -20,7 +20,7 @@ class AnalysisResults {
     containsTest(test)
     {
         console.assert(test instanceof Test);
-        for (let metric of test.metrics()) {
+        for (const metric of test.metrics()) {
             if (metric.id() in this._metricToBuildMap)
                 return true;
         }