Address Antti's review comment.
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 11 Jul 2017 03:46:16 +0000 (03:46 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 11 Jul 2017 03:46:16 +0000 (03:46 +0000)
* 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

Websites/perf.webkit.org/ChangeLog
Websites/perf.webkit.org/public/v3/models/analysis-results.js

index ffbd679..5a6e90d 100644 (file)
@@ -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
 
index bc395af..5104aa5 100644 (file)
@@ -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;
         }