Changeset 100313 in webkit

Timestamp:

Nov 15, 2011 1:03:52 PM (12 years ago)

Author:

commit-queue@webkit.org

Message:

Extend sunspider driver to be able to run kraken
​https://bugs.webkit.org/show_bug.cgi?id=71799

Patch by Andy Wingo <​wingo@igalia.com> on 2011-11-15
Reviewed by Filip Pizlo.

• resources/sunspider-standalone-driver.js: Try to load a -data

file. If that succeeds, we have a kraken-like test, so we time
the test using load'. Otherwise fall back to using run'.

Location:

trunk/PerformanceTests/SunSpider

Files:

• ChangeLog (modified) (1 diff)
• resources/sunspider-standalone-driver.js (modified) (1 diff)

trunk/PerformanceTests/SunSpider/ChangeLog

@@ +1 @@
+2011-11-15  Andy Wingo  <wingo@igalia.com>
+
+        Extend sunspider driver to be able to run kraken
+        https://bugs.webkit.org/show_bug.cgi?id=71799
+
+        Reviewed by Filip Pizlo.
+
+        * resources/sunspider-standalone-driver.js: Try to load a -data
+        file.  If that succeeds, we have a kraken-like test, so we time
+        the test using `load'.  Otherwise fall back to using `run'.
+
 2011-08-11  Mark Rowe  <mrowe@apple.com>
 

trunk/PerformanceTests/SunSpider/resources/sunspider-standalone-driver.js

@@ -33 +33 @@
 
 for (var j = 0; j < tests.length; j++) {
-    var testName = suitePath + "/" + tests[j] + ".js";
-    var startTime = new Date;
-    if (testName.indexOf('parse-only') >= 0)
+    var testBase = suitePath + "/" + tests[j];
+    var testName = testBase + ".js";
+    var testData = testBase + "-data.js";
+
+    if (testName.indexOf('parse-only') >= 0) {
         times[j] = checkSyntax(testName);
-    else
-        times[j] = run(testName);
+    } else {
+        // Tests may or may not have associated -data files whose loading
+        // should not be timed.
+        try {
+            load(testData);
+            // If a file does have test data, then we can't use the
+            // higher-precision `run' timer, because `run' uses a fresh
+            // global environment, so we fall back to `load'.
+            var startTime = new Date;
+            load(testName);
+            times[j] = new Date() - startTime;
+        } catch (e) {
+            // No test data, just use `run'.
+            times[j] = run(testName);
+        }
+    }
     gc();
 }