[webkit-reviews] review granted: [Bug 216301] Import selection web platform tests : [Attachment 408300] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Sep 8 23:03:08 PDT 2020


youenn fablet <youennf at gmail.com> has granted Alex Christensen
<achristensen at apple.com>'s request for review:
Bug 216301: Import selection web platform tests
https://bugs.webkit.org/show_bug.cgi?id=216301

Attachment 408300: Patch

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




--- Comment #6 from youenn fablet <youennf at gmail.com> ---
Comment on attachment 408300
  --> https://bugs.webkit.org/attachment.cgi?id=408300
Patch

Skipping timing out tests is always preferable than running them.
It is also better to import all tests in a folder: we can fix them
progressively and we do not have to spend the time to rediscover they are
timing out on each reimport.

Looking at some of these tests, they end up with more than 5000 subtests...
Alex, please verify these tests pass in debug.


More information about the webkit-reviews mailing list