[webkit-reviews] review granted: [Bug 203652] Flaky API Test TestWebKitAPI.WebKit.UploadDirectory : [Attachment 382741] Patch

[bugzilla-daemon at webkit.org]

Alexey Proskuryakov <ap at webkit.org> has granted Alex Christensen
<achristensen at apple.com>'s request for review:
Bug 203652: Flaky API Test TestWebKitAPI.WebKit.UploadDirectory
https://bugs.webkit.org/show_bug.cgi?id=203652

Attachment 382741: Patch

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




--- Comment #12 from Alexey Proskuryakov <ap at webkit.org> ---
Comment on attachment 382741
  --> https://bugs.webkit.org/attachment.cgi?id=382741
Patch

I would slightly prefer checking for the error or asserting after deleting.
Knowing that deletion succeeded will make it easier to reason about future
failures, if any occur.

r=me with or without such changes.