[webkit-reviews] review granted: [Bug 162814] Web Inspector: TestSuite test cases should have their own timeout to ensure tests fail with output instead of timeout by test runner : [Attachment 366824] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Apr 5 15:38:01 PDT 2019
Brian Burg <bburg at apple.com> has granted review:
Bug 162814: Web Inspector: TestSuite test cases should have their own timeout
to ensure tests fail with output instead of timeout by test runner
https://bugs.webkit.org/show_bug.cgi?id=162814
Attachment 366824: Patch
https://bugs.webkit.org/attachment.cgi?id=366824&action=review
--- Comment #22 from Brian Burg <bburg at apple.com> ---
Comment on attachment 366824
--> https://bugs.webkit.org/attachment.cgi?id=366824
Patch
View in context: https://bugs.webkit.org/attachment.cgi?id=366824&action=review
r=me
>> Source/WebInspectorUI/ChangeLog:15
>> + This change also relaxes the expectation that any individual test
case failure will stop the
>
> This patch is trying to do too many things. Given that it's making a risky
infrastructure change that's likely to trip up the bumbling bots, I think you
should split out the "continue on failure" part out. That part seems to be
responsible for most of the test and result diffs, yet it seems less
contentious and less likely to be rolled out and back in repeatedly than the
"add a timeout" part of the patch.
OTOH, maybe it will work. If it has to be rolled out, let's be more cautious in
followup attempts.
> LayoutTests/inspector/canvas/resources/shaderProgram-utilities.js:78
> + whenProgramAdded((program) => {
Nice.
More information about the webkit-reviews
mailing list