Bug 217408 - REGRESSION(iOS 14): http/tests/cache-storage/page-cache-domcache-pending-promise.html is a flaky failure
Summary: REGRESSION(iOS 14): http/tests/cache-storage/page-cache-domcache-pending-prom...
Status: RESOLVED CONFIGURATION CHANGED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-10-06 15:13 PDT by Hector Lopez
Modified: 2021-04-19 17:08 PDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Hector Lopez 2020-10-06 15:13:09 PDT
http/tests/cache-storage/page-cache-domcache-pending-promise.html

Test is a flaky failure according to history on iOS. First occurrence of failure coincide with the start of testing on iOS 14. 

History:

https://results.webkit.org/?suite=layout-tests&test=http%2Ftests%2Fcache-storage%2Fpage-cache-domcache-pending-promise.html&platform=ios

Diff:

--- /Volumes/Data/slave/ios-simulator-14-release-tests-wk2/build/layout-test-results/http/tests/cache-storage/page-cache-domcache-pending-promise-expected.txt
+++ /Volumes/Data/slave/ios-simulator-14-release-tests-wk2/build/layout-test-results/http/tests/cache-storage/page-cache-domcache-pending-promise-actual.txt
@@ -7,8 +7,9 @@
 pagehide - entering cache
 pageshow - from cache
 PASS Page was restored from Page Cache
-PASS Cache.add() succeeded after restoring from the cache
+FAIL Page restored from cache but the promise was not resolved.
 PASS successfullyParsed is true
+Some tests failed.
 
 TEST COMPLETE
Comment 1 Radar WebKit Bug Importer 2020-10-06 15:13:29 PDT
<rdar://problem/70019463>
Comment 2 Hector Lopez 2020-10-06 15:21:53 PDT
Test expectation while investigated:

https://trac.webkit.org/changeset/268078/webkit
Comment 3 Ryan Haddad 2020-12-18 15:46:48 PST
Added DumpJSConsoleLogInStdErr back to the test expectation with r270992 entry because its removal in r268078 made it fail consistently.
Comment 4 Ryan Haddad 2021-04-19 17:08:21 PDT
It looks like this is passing now.