Speedometer 2.0: Add the capability to run a specific suite
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 23 Aug 2017 23:54:40 +0000 (23:54 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 23 Aug 2017 23:54:40 +0000 (23:54 +0000)
https://bugs.webkit.org/show_bug.cgi?id=175908

Address Joe's review comments.

* Speedometer/resources/main.js:
(startBenchmark):

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

PerformanceTests/ChangeLog
PerformanceTests/Speedometer/resources/main.js

index 6d4d58c..b5d4bd5 100644 (file)
@@ -3,6 +3,16 @@
         Speedometer 2.0: Add the capability to run a specific suite
         https://bugs.webkit.org/show_bug.cgi?id=175908
 
+        Address Joe's review comments.
+
+        * Speedometer/resources/main.js:
+        (startBenchmark):
+
+2017-08-23  Ryosuke Niwa  <rniwa@webkit.org>
+
+        Speedometer 2.0: Add the capability to run a specific suite
+        https://bugs.webkit.org/show_bug.cgi?id=175908
+
         Reviewed by Saam Barati.
 
         Added ?suite=X query parameter to specify a specific suite (e.g. React-TodoMVC) to run.
index 0e48e33..59dc7e1 100644 (file)
@@ -165,8 +165,6 @@ function enableOneSuite(suites, suiteToEnable)
 }
 
 function startBenchmark() {
-    var enabledSuites = Suites.filter(function (suite) { return !suite.disabled; });
-
     if (location.search.length > 1) {
         var parts = location.search.substring(1).split('&');
         for (var i = 0; i < parts.length; i++) {
@@ -189,7 +187,7 @@ function startBenchmark() {
                 break;
             case 'suite':
                 if (!enableOneSuite(Suites, value)) {
-                    alert('No tests to run');
+                    alert('Suite "' + value + '" does not exist. No tests to run.');
                     return false;
                 }
                 break;