Test results indicate that test output is sometimes written twice for the same test, causing flakiness.
Created attachment 299243 [details] Patch
Comment on attachment 299243 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=299243&action=review r=me > Tools/DumpRenderTree/win/DumpRenderTree.cpp:685 > + It would be good to understand why this happens. Do we need a similar protection in macOS and other platforms? Or does Windows call dump() in more places?
Comment on attachment 299243 [details] Patch Clearing flags on attachment: 299243 Committed r210965: <http://trac.webkit.org/changeset/210965>
All reviewed patches have been landed. Closing bug.
(In reply to comment #2) > Comment on attachment 299243 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=299243&action=review > > r=me > > > Tools/DumpRenderTree/win/DumpRenderTree.cpp:685 > > + > > It would be good to understand why this happens. Do we need a similar > protection in macOS and other platforms? Or does Windows call dump() in more > places? macOS will not process messages after dump() is called, while Windows will process a few messages before receiving the WM_QUIT message. This opens up for dump() to be called again on Windows. This might happen if the test performs a reload, and calls notifyDone() a second time. Thanks for reviewing!