Title: [159421] trunk/PerformanceTests
- Revision
- 159421
- Author
- [email protected]
- Date
- 2013-11-18 07:28:47 -0800 (Mon, 18 Nov 2013)
Log Message
[Performance tests] Interactive/SelectAll.html throws an exception
https://bugs.webkit.org/show_bug.cgi?id=124495
Reviewed by Antti Koivisto
Return a boolean indicating whether more values are needed or not in
PerfTestRunner.measureValueAsync so that runTest can terminate gracefully.
* Interactive/SelectAll.html:
(runTest): Don't schedule a timer for runTest if we've got enough values.
* resources/runner.js:
(PerfTestRunner.measureValueAsync): Returns true iff more values are needed.
Modified Paths
Diff
Modified: trunk/PerformanceTests/ChangeLog (159420 => 159421)
--- trunk/PerformanceTests/ChangeLog 2013-11-18 14:52:13 UTC (rev 159420)
+++ trunk/PerformanceTests/ChangeLog 2013-11-18 15:28:47 UTC (rev 159421)
@@ -1,3 +1,18 @@
+2013-11-18 Ryosuke Niwa <[email protected]>
+
+ [Performance tests] Interactive/SelectAll.html throws an exception
+ https://bugs.webkit.org/show_bug.cgi?id=124495
+
+ Reviewed by Antti Koivisto
+
+ Return a boolean indicating whether more values are needed or not in
+ PerfTestRunner.measureValueAsync so that runTest can terminate gracefully.
+
+ * Interactive/SelectAll.html:
+ (runTest): Don't schedule a timer for runTest if we've got enough values.
+ * resources/runner.js:
+ (PerfTestRunner.measureValueAsync): Returns true iff more values are needed.
+
2013-11-13 Antti Koivisto <[email protected]>
This was supposed to test overflow-wrap:break-word instead of word-break:break-all.
Modified: trunk/PerformanceTests/Interactive/SelectAll.html (159420 => 159421)
--- trunk/PerformanceTests/Interactive/SelectAll.html 2013-11-18 14:52:13 UTC (rev 159420)
+++ trunk/PerformanceTests/Interactive/SelectAll.html 2013-11-18 15:28:47 UTC (rev 159421)
@@ -22,10 +22,11 @@
iframe.contentDocument.execCommand('SelectAll');
iframe.contentDocument.body.offsetTop;
setTimeout(function () {
- PerfTestRunner.measureValueAsync(PerfTestRunner.now() - startTime);
+ if (!PerfTestRunner.measureValueAsync(PerfTestRunner.now() - startTime))
+ return;
PerfTestRunner.gc();
setTimeout(runTest, 0);
- }, 0);
+ }, 0);
}, 0);
}
Modified: trunk/PerformanceTests/resources/runner.js (159420 => 159421)
--- trunk/PerformanceTests/resources/runner.js 2013-11-18 14:52:13 UTC (rev 159420)
+++ trunk/PerformanceTests/resources/runner.js 2013-11-18 15:28:47 UTC (rev 159421)
@@ -236,11 +236,15 @@
ignoreWarmUpAndLog(measuredValue);
} catch (exception) {
logFatalError("Got an exception while logging the result with name=" + exception.name + ", message=" + exception.message);
- return;
+ return false;
}
- if (completedIterations >= iterationCount)
+ if (completedIterations >= iterationCount) {
finish();
+ return false;
+ }
+
+ return true;
}
PerfTestRunner.measureTime = function (test) {
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
