Bug 217741 - 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
Summary: REGRESSION (r268178): [iOS] imported/w3c/web-platform-tests/html/semantics/em...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Images (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-10-14 16:52 PDT by Ryan Haddad
Modified: 2020-10-30 18:36 PDT (History)
7 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ryan Haddad 2020-10-14 16:52:00 PDT
imported/w3c/web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-multiple-times.html is a flaky failure on iOS bots after https://trac.webkit.org/changeset/268178 landed

--- /Volumes/Data/slave/ios-simulator-14-release-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-multiple-times-expected.txt
+++ /Volumes/Data/slave/ios-simulator-14-release-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-multiple-times-actual.txt
@@ -1,4 +1,4 @@
 
 
-PASS Images with loading='lazy' can be lazy loaded multiple times
+FAIL Images with loading='lazy' can be lazy loaded multiple times assert_unreached: The loading=lazy below-viewport image should lazily load its second image, and not load it eagerly when the `src` attribute is changed Reached unreachable code
 
https://results.webkit.org/?suite=layout-tests&test=imported/w3c/web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-multiple-times.html
Comment 1 Radar WebKit Bug Importer 2020-10-14 16:52:44 PDT
<rdar://problem/70313927>
Comment 2 Karl Rackler 2020-10-15 08:18:29 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 ]
Comment 3 Rob Buis 2020-10-15 08:29:43 PDT
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.
Comment 4 Aakash Jain 2020-10-15 08:35:51 PDT
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
Comment 5 Karl Rackler 2020-10-15 08:44:11 PDT
I have marked this test as failing while this issue is investigated.
https://trac.webkit.org/changeset/268524/webkit
Comment 6 Karl Rackler 2020-10-30 18:36:50 PDT
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.