Bug 238642

Summary: REGRESSION(r292127): [ BigSur+ wk1 ] imported/w3c/web-platform-tests/css/css-contain/contain-body-overflow-002.html is a constant image failure
Product: WebKit Reporter: Dawn Morningstar <Morningstar>
Component: CSSAssignee: Tim Nguyen (:ntim) <ntim>
Status: RESOLVED FIXED    
Severity: Normal CC: jenner, webkit-bot-watchers-bugzilla, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://github.com/web-platform-tests/wpt/pull/36968
Attachments:
Description Flags
/imported/w3c/web-platform-tests/css/css-contain/contain-body-overflow-002-diffs.html DIFF images none

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.