Bug 205209

Summary: Help debugging flaky http/tests/cache-storage/page-cache-domcachestorage-pending-promise.html
Product: WebKit Reporter: youenn fablet <youennf>
Component: Service WorkersAssignee: youenn fablet <youennf>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, cdumez, cgarcia, commit-queue, ews-watchlist, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

Description youenn fablet 2019-12-13 09:21:41 PST
Help debugging flaky http/tests/cache-storage/page-cache-domcachestorage-pending-promise.html
Comment 1 youenn fablet 2019-12-13 09:24:54 PST
Created attachment 385610 [details]
Patch
Comment 2 youenn fablet 2019-12-13 09:46:59 PST
Created attachment 385612 [details]
Patch
Comment 3 WebKit Commit Bot 2019-12-13 10:58:11 PST
Comment on attachment 385612 [details]
Patch

Clearing flags on attachment: 385612

Committed r253487: <https://trac.webkit.org/changeset/253487>
Comment 4 WebKit Commit Bot 2019-12-13 10:58:13 PST
All reviewed patches have been landed.  Closing bug.
Comment 5 Radar WebKit Bug Importer 2019-12-13 10:59:23 PST
<rdar://problem/57917734>
Comment 6 Alexey Proskuryakov 2019-12-16 13:17:57 PST
Looks like it got a lot more flaky. One recent result:

@@ -1,3 +1,6 @@
+CONSOLE MESSAGE: line 31: { "path": "/var/folders/5x/tddp1dn97mjg69n9470bjmlh0000gn/T/WebKitTestRunnerApp.apps-boRyeU/CacheStorage", "origins": [
+{ "origin" : { "topOrigin" : "http://127.0.0.1:8000", "clientOrigin": "http://127.0.0.1:8000" }, "caches" : { "persistent": ["test"], "removed": []}
+}]}
 Tests that a page with pending DOMCacheStorage activity goes into the page cache.
 
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
@@ -7,9 +10,9 @@
 pagehide - entering cache
 pageshow - from cache
 PASS Page was restored from Page Cache
-PASS Opened the cache
-PASS !!restoredFromPageCache is true
+FAIL Timed out while waiting for the cache to open
 PASS successfullyParsed is true
+Some tests failed.