Title: [295737] trunk/LayoutTests
- Revision
- 295737
- Author
- commit-qu...@webkit.org
- Date
- 2022-06-22 10:57:21 -0700 (Wed, 22 Jun 2022)
Log Message
REGRESSION (r293506): [ iOS ][ macOS ] imported/w3c/web-platform-tests/service-workers/service-worker/registration-updateviacache.https.html is a flaky failure
https://bugs.webkit.org/show_bug.cgi?id=240074
rdar://problem/92742526
Patch by Youenn Fablet <youe...@gmail.com> on 2022-06-22
Reviewed by Chris Dumez.
Before r293506, we were refreshing the imported scripts from network process only.
We are now correctly refreshing them from the caller of update.
This makes it possible to load the same scripts from various processes (web process, service worker process), which have different memory caches.
The failure happened due to the fact that different subtests were importing the same subscript with various options.
This triggered storing the subscript in different memory caches.
Some tests expect cache to kick in and expect in that case to have the same resource.
If the cache kicks in, but on different memory caches, we end up with different resources.
To prevent this in the test, we simply append the test name to each subscript import.
* LayoutTests/imported/w3c/web-platform-tests/service-workers/service-worker/resources/update-max-aged-worker.py:
(main):
* LayoutTests/platform/mac/TestExpectations:
Canonical link: https://commits.webkit.org/251742@main
Modified Paths
Diff
Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/service-workers/service-worker/resources/update-max-aged-worker.py (295736 => 295737)
--- trunk/LayoutTests/imported/w3c/web-platform-tests/service-workers/service-worker/resources/update-max-aged-worker.py 2022-06-22 17:23:25 UTC (rev 295736)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/service-workers/service-worker/resources/update-max-aged-worker.py 2022-06-22 17:57:21 UTC (rev 295737)
@@ -13,7 +13,7 @@
body = u'''
const mainTime = {time:8f};
const testName = {test};
- importScripts('update-max-aged-worker-imported-script.py');
+ importScripts('update-max-aged-worker-imported-script.py?test={test}');
addEventListener('message', event => {{
event.source.postMessage({{
Modified: trunk/LayoutTests/platform/mac/TestExpectations (295736 => 295737)
--- trunk/LayoutTests/platform/mac/TestExpectations 2022-06-22 17:23:25 UTC (rev 295736)
+++ trunk/LayoutTests/platform/mac/TestExpectations 2022-06-22 17:57:21 UTC (rev 295737)
@@ -2289,8 +2289,6 @@
webkit.org/b/239990 css3/calc/transitions-dependent.html [ Pass Failure ]
-webkit.org/b/240074 imported/w3c/web-platform-tests/service-workers/service-worker/registration-updateviacache.https.html [ Pass Failure ]
-
webkit.org/b/231298 accessibility/mac/selection-sync.html [ Skip ] # Timeout
webkit.org/b/227845 [ Debug ] webaudio/audioworket-out-of-memory.html [ Pass Timeout DumpJSConsoleLogInStdErr ]
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes