perftestsrunner can call printer.write() after printer.cleanup()
PerfTestsRunner.run() calls printer.cleanup(), and you should not call any methods on printer after cleanup(). In the unittests, we call run() twice in a row (in test_run_with_upload_json), and the second time attempts to write something. We should either not call cleanup() during run(), or reinitialize the printer during run(), or not call run() twice. I will post a patch that does the first thing, but I'm not sure if you're supposed to be able to call run() twice in a row or not.
Created attachment 129391 [details] Patch
Comment on attachment 129391 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=129391&action=review > Tools/ChangeLog:13 > + It was a fluke that this happened to work, but it is still a bug > + that should be fixed either by not calling cleanup() during > + run() or by not calling run() twice. This patch does the former. Can we take the latter approach instead? We shouldn't be calling run() twice.
Sure, I'll rework it.
Created attachment 129392 [details] don't call run() twice instead
Comment on attachment 129392 [details] don't call run() twice instead please take another look?
Committed r109235: <http://trac.webkit.org/changeset/109235>