RESOLVED FIXED 182088
Make run-perf-tests work with Speedometer 2.0 and re-enable the tests
https://bugs.webkit.org/show_bug.cgi?id=182088
Summary Make run-perf-tests work with Speedometer 2.0 and re-enable the tests
Ryosuke Niwa
Reported 2018-01-25 00:59:02 PST
Make the new score computation reflected in run-perf-tests and unskip the test.
Attachments
Patch (6.27 KB, patch)
2018-01-25 01:05 PST, Ryosuke Niwa
koivisto: review+
commit-queue: commit-queue-
Ryosuke Niwa
Comment 1 2018-01-25 01:05:47 PST
Ryosuke Niwa
Comment 2 2018-01-25 01:06:10 PST
Unfortunately, we can't land this patch until we fix perf dashboard as well.
Ryosuke Niwa
Comment 3 2018-01-25 01:08:13 PST
Comment on attachment 332237 [details] Patch As such, we can't land this patch. Landing this patch would prevent all test results from being processed in the perf dashboard.
Radar WebKit Bug Importer
Comment 4 2018-01-25 01:15:10 PST
Ryosuke Niwa
Comment 5 2018-01-29 17:38:33 PST
We actually need this fix for StyleBench as well.
Ryan Haddad
Comment 6 2018-02-05 21:24:29 PST
*** Bug 182079 has been marked as a duplicate of this bug. ***
WebKit Commit Bot
Comment 7 2018-02-09 21:22:51 PST
Comment on attachment 332237 [details] Patch Rejecting attachment 332237 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-02', 'apply-attachment', '--no-update', '--non-interactive', 332237, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: ile PerformanceTests/resources/results-template.html Hunk #1 FAILED at 227. 1 out of 1 hunk FAILED -- saving rejects to file PerformanceTests/resources/results-template.html.rej patching file PerformanceTests/resources/runner.js Hunk #1 FAILED at 223. 1 out of 1 hunk FAILED -- saving rejects to file PerformanceTests/resources/runner.js.rej Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Antti Koivisto']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Full output: http://webkit-queues.webkit.org/results/6442748
Ryosuke Niwa
Comment 8 2018-02-09 22:14:54 PST
Note You need to log in before you can comment on or make changes to this bug.