Calling internals from the performance test runner prevents manual running
authorbenjamin@webkit.org <benjamin@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 8 Aug 2012 21:48:38 +0000 (21:48 +0000)
committerbenjamin@webkit.org <benjamin@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 8 Aug 2012 21:48:38 +0000 (21:48 +0000)
https://bugs.webkit.org/show_bug.cgi?id=93527

Reviewed by Ryosuke Niwa.

Since r125065, it is impossible to run the Performance tests manually. The problem is
"internals" is not defined when running outside the test runner, so the whole script
evaluation fails.

* resources/runner.js: Verify is window.internals is available before invoking something on it.

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

PerformanceTests/ChangeLog
PerformanceTests/resources/runner.js

index ba69e17..58fbfd1 100644 (file)
@@ -1,3 +1,16 @@
+2012-08-08  Benjamin Poulain  <benjamin@webkit.org>
+
+        Calling internals from the performance test runner prevents manual running
+        https://bugs.webkit.org/show_bug.cgi?id=93527
+
+        Reviewed by Ryosuke Niwa.
+
+        Since r125065, it is impossible to run the Performance tests manually. The problem is
+        "internals" is not defined when running outside the test runner, so the whole script
+        evaluation fails.
+
+        * resources/runner.js: Verify is window.internals is available before invoking something on it.
+
 2012-08-08  Zoltan Horvath  <zoltan@webkit.org>
 
         Set access to MemoryInfo enabled for the performance tests
index 61e555c..d7dcfa0 100755 (executable)
@@ -1,7 +1,8 @@
 // There are tests for computeStatistics() located in LayoutTests/fast/harness/perftests
 
 // We need access to console.memory for the memory measurements
-internals.settings.setMemoryInfoEnabled(true);
+if (window.internals)
+     internals.settings.setMemoryInfoEnabled(true);
 
 var PerfTestRunner = {};