[webkit-reviews] review granted: [Bug 196898] TestRunner::notifyDone() should not be reentrant : [Attachment 367427] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Apr 15 11:03:50 PDT 2019


Darin Adler <darin at apple.com> has granted Per Arne Vollan <pvollan at apple.com>'s
request for review:
Bug 196898: TestRunner::notifyDone() should not be reentrant
https://bugs.webkit.org/show_bug.cgi?id=196898

Attachment 367427: Patch

https://bugs.webkit.org/attachment.cgi?id=367427&action=review




--- Comment #8 from Darin Adler <darin at apple.com> ---
Comment on attachment 367427
  --> https://bugs.webkit.org/attachment.cgi?id=367427
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=367427&action=review

Guess it’s OK. Lets give it a try.

> Tools/DumpRenderTree/mac/TestRunnerMac.mm:301
> +	   fprintf(stderr, "TestRunner::notifyDone() called unexpectedly.");

So I guess it’s OK to do this since it won’t make tests fail? But it will fill
our test results with "stderr output"; if a reentrant call is *not* actually
unexpected.


More information about the webkit-reviews mailing list