RESOLVED FIXED 97870
REGRESSION (r129824): fast/harness/perftests/runs-per-second-iterations.html, fast/harness/perftests/runs-per-second-log.html failing 
https://bugs.webkit.org/show_bug.cgi?id=97870
Summary REGRESSION (r129824): fast/harness/perftests/runs-per-second-iterations.html,...
Zan Dobersek
Reported 2012-09-27 23:56:49 PDT
The following two tests started failing after r129824[1]: fast/harness/perftests/runs-per-second-iterations.html fast/harness/perftests/runs-per-second-log.html Diffs: --- /home/slave/webkitgtk/gtk-linux-64-debug/build/layout-test-results/fast/harness/perftests/runs-per-second-iterations-expected.txt +++ /home/slave/webkitgtk/gtk-linux-64-debug/build/layout-test-results/fast/harness/perftests/runs-per-second-iterations-actual.txt @@ -12,5 +12,5 @@ PASS callsInIterations[5] is callsInIterations[6] PASS callsInIterations[6] is callsInIterations[7] PASS callsInIterations[7] is callsInIterations[8] -PASS callsInIterations[9] is undefined. +FAIL callsInIterations[9] should be undefined. Was 40 --- /home/slave/webkitgtk/gtk-linux-64-debug/build/layout-test-results/fast/harness/perftests/runs-per-second-log-expected.txt +++ /home/slave/webkitgtk/gtk-linux-64-debug/build/layout-test-results/fast/harness/perftests/runs-per-second-log-actual.txt @@ -1,18 +1,33 @@ This test verifies PerfTestRunner.runPerSecond() outputs log as expected. -Running 5 times +Running 20 times Ignoring warm-up run (0 runs/s) 1 runs/s 2 runs/s 3 runs/s 4 runs/s 5 runs/s +6 runs/s +7 runs/s +8 runs/s +9 runs/s +10 runs/s +11.000000000000002 runs/s +12.000000000000002 runs/s +13 runs/s +13.999999999999998 runs/s +14.999999999999998 runs/s +16 runs/s +17 runs/s +18 runs/s +18.999999999999996 runs/s +20 runs/s Time: -values 1, 2, 3, 4, 5 runs/s -avg 3 runs/s -median 3 runs/s -stdev 1.41 runs/s +values 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11.000000000000002, 12.000000000000002, 13, 13.999999999999998, 14.999999999999998, 16, 17, 18, 18.999999999999996, 20 runs/s +avg 10.5 runs/s +median 10.5 runs/s +stdev 5.77 runs/s min 1 runs/s -max 5 runs/s +max 20 runs/s [1] http://trac.webkit.org/changeset/129824
Attachments
Add attachment proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1 2012-09-28 00:35:24 PDT
Ugh... test.runCount is used in these tests :(
Ryosuke Niwa
Comment 2 2012-09-28 00:38:00 PDT
Fixed by http://trac.webkit.org/changeset/129858.
Note You need to log in before you can comment on or make changes to this bug.