Summary: | REGRESSION (r268178): [iOS] imported/w3c/web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-multiple-times.html is a flaky failure | ||
---|---|---|---|
Product: | WebKit | Reporter: | Ryan Haddad <ryanhaddad> |
Component: | Images | Assignee: | Nobody <webkit-unassigned> |
Status: | NEW --- | ||
Severity: | Normal | CC: | aakash_jain, rackler, rbuis, rwlbuis, webkit-bot-watchers-bugzilla, webkit-bug-importer, youennf |
Priority: | P2 | Keywords: | InRadar |
Version: | WebKit Nightly Build | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=216979 |
Description
Ryan Haddad
2020-10-14 16:52:00 PDT
REPRODUCTION STEPS I can reproduce this on 268217, but unable to reproduce on 268154. Commit r268178 has to do with changing the test expectation, which could have introduced the flakiness. The change was introduced here https://trac.webkit.org/changeset/268178/webkit. Command: run-webkit-tests --ios-simulator --iterations 50 --exit-after-n-failures 1 --exit-after-n-crashes-or-timeouts 1 imported/w3c/web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-multiple-times.html Result: Regressions: Unexpected text-only failures (1) imported/w3c/web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-multiple-times.html [ Failure ] This is not unexpected, as the logic is tricky, but I wonder if the same problem also happens on OS X. I will have a look tomorrow. Interesting that https://ews-build.webkit.org/#/builders/24/builds/27659 also says: Found flaky test: imported/w3c/web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-multiple-times.html I have marked this test as failing while this issue is investigated. https://trac.webkit.org/changeset/268524/webkit This reverts commit r268178 because it caused a test failure. https://trac.webkit.org/changeset/268619/webkit https://trac.webkit.org/changeset/269217/webkit Remove test expectation after revert of r268178. |