Bug 208165

Summary: [WK1] http/tests/cookies/document-cookie-after-showModalDialog.html is a flaky failure since introduction
Product: WebKit Reporter: Truitt Savell <tsavell>
Component: WebKit2Assignee: Nobody <webkit-unassigned>
Status: RESOLVED WONTFIX    
Severity: Normal CC: ahmad.saleem792, ap, bfulgham, cdumez, rniwa, webkit-bot-watchers-bugzilla, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=207593
https://bugs.webkit.org/show_bug.cgi?id=208126

Description Truitt Savell 2020-02-24 16:16:08 PST
http/tests/cookies/document-cookie-after-showModalDialog.html

This test has been a flaky failure sense introduction in https://trac.webkit.org/changeset/256820/webkit

history:
https://results.webkit.org/?suite=layout-tests&test=http%2Ftests%2Fcookies%2Fdocument-cookie-after-showModalDialog.html

Diff:
--- /Volumes/Data/slave/catalina-release-tests-wk1/build/layout-test-results/http/tests/cookies/document-cookie-after-showModalDialog-expected.txt
+++ /Volumes/Data/slave/catalina-release-tests-wk1/build/layout-test-results/http/tests/cookies/document-cookie-after-showModalDialog-actual.txt
@@ -4,7 +4,7 @@
 
 
 PASS normalizeCookie(document.cookie) is "testKey=testValue"
-PASS normalizeCookie(document.cookie) is "foo=bar; testKey=testValue"
+FAIL normalizeCookie(document.cookie) should be foo=bar; testKey=testValue. Was testKey=testValue.
 PASS successfullyParsed is true
 
 TEST COMPLETE
Comment 1 Radar WebKit Bug Importer 2020-02-24 16:16:28 PST
<rdar://problem/59745648>
Comment 2 Ahmad Saleem 2022-08-05 15:11:30 PDT
Following commit stop exposing "stopModalDialog" to web and these tests are now marked to run with webkit-test-runner when showModalDialog is true.

https://github.com/WebKit/WebKit/commit/c4bb4d1af89e1ef5f57ef1b1cf7b65ce62c30ba5

I think these flaky tests will not fixed. Hence, I am going to mark this as "RESOLVED WONTFIX", please reopen if my selection is wrong and there is further needed from this bug. Thanks!