[webkit-reviews] review granted: [Bug 195107] Flaky API Test: TestWebKitAPI.ProcessSwap.PageZoomLevelAfterSwap : [Attachment 363226] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Feb 28 10:19:37 PST 2019
Alex Christensen <achristensen at apple.com> has granted Chris Dumez
<cdumez at apple.com>'s request for review:
Bug 195107: Flaky API Test: TestWebKitAPI.ProcessSwap.PageZoomLevelAfterSwap
https://bugs.webkit.org/show_bug.cgi?id=195107
Attachment 363226: Patch
https://bugs.webkit.org/attachment.cgi?id=363226&action=review
--- Comment #4 from Alex Christensen <achristensen at apple.com> ---
Comment on attachment 363226
--> https://bugs.webkit.org/attachment.cgi?id=363226
Patch
View in context: https://bugs.webkit.org/attachment.cgi?id=363226&action=review
> Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:2788
> + } while (clientWidth != expectedClientWidth);
Having a maximum number of tries would help us debug any future problems with
this, turning a timeout into an eventual failure. Right now the maximum is
effectively 1, and this patch makes it infinite. Somewhere between the two
ought to be good.
More information about the webkit-reviews
mailing list