Bug 238642 - REGRESSION(r292127): [ BigSur+ wk1 ] imported/w3c/web-platform-tests/css/css-contain/contain-body-overflow-002.html is a constant image failure
Summary: REGRESSION(r292127): [ BigSur+ wk1 ] imported/w3c/web-platform-tests/css/css-...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Tim Nguyen (:ntim)
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-03-31 17:48 PDT by Dawn Morningstar
Modified: 2022-11-15 10:14 PST (History)
4 users (show)

See Also:


Attachments
/imported/w3c/web-platform-tests/css/css-contain/contain-body-overflow-002-diffs.html DIFF images (8.33 KB, application/zip)
2022-03-31 17:51 PDT, Dawn Morningstar
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Dawn Morningstar 2022-03-31 17:48:12 PDT
imported/w3c/web-platform-tests/css/css-contain/contain-body-overflow-002.html
Is a constant image failure on wk1 following the CSS changes in r292127


HISTORY:
https://results.webkit.org/?suite=layout-tests&test=imported%2Fw3c%2Fweb-platform-tests%2Fcss%2Fcss-contain%2Fcontain-body-overflow-002.html

DIFFS Will be attached.

DIFF-URL:
https://build.webkit.org/results/Apple-Monterey-Debug-WK1-Tests/r292164%20(1636)/imported/w3c/web-platform-tests/css/css-contain/contain-body-overflow-002-diffs.html
Comment 1 Radar WebKit Bug Importer 2022-03-31 17:49:00 PDT
<rdar://problem/91137177>
Comment 2 Dawn Morningstar 2022-03-31 17:51:12 PDT
Created attachment 456300 [details]
/imported/w3c/web-platform-tests/css/css-contain/contain-body-overflow-002-diffs.html DIFF images
Comment 3 Robert Jenner 2022-03-31 18:14:11 PDT
Added test expectations as [ Failure ] for Mac-wk1 here:

https://commits.webkit.org/249095@main
Comment 4 zalan 2022-04-07 08:52:03 PDT
this has something to do with our WK1 focus behavior.

@@ -28,6 +28,10 @@ body {
 <p tabindex=1 id=target>Test passes if there is no red.
 <script>
 window.onload = function() {
-    document.getElementById("target").focus()
+    testRunner.waitUntilDone();
+    document.getElementById("target").focus();
+    setTimeout(function() {
+        testRunner.notifyDone();
+    }, 0);
 }
 </script>

^^PASS.
Comment 5 Tim Nguyen (:ntim) 2022-11-15 00:11:44 PST
Pull request: https://github.com/WebKit/WebKit/pull/6508
Comment 6 Tim Nguyen (:ntim) 2022-11-15 00:13:31 PST
Submitted web-platform-tests pull request: https://github.com/web-platform-tests/wpt/pull/36968
Comment 7 EWS 2022-11-15 10:14:08 PST
Committed 256701@main (300a05321aff): <https://commits.webkit.org/256701@main>

Reviewed commits have been landed. Closing PR #6508 and removing active labels.