From 6a42daa6a190d3f8e0a41eb3ea037b982b1f0536 Mon Sep 17 00:00:00 2001 From: sisidovski Date: Sat, 27 Jan 2024 18:00:42 +0000 Subject: [PATCH] Bug 1876466 [wpt PR 44190] - Add test for the no fetch handler case, a=testonly Automatic update from web-platform-tests Add test for the no fetch handler case This adds the test for crrev.com/c/5119144/. Even if there is no fetch handler in SW, this test confirms the router is always evaluated. Bug: 1519727, 1511459 Change-Id: I20353cf9b076e3d225047f42df1557c1fbd4f5ce Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5233868 Commit-Queue: Shunya Shishido Reviewed-by: Yoshisato Yanagisawa Reviewed-by: Minoru Chikamune Cr-Commit-Position: refs/heads/main@{#1252109} -- wpt-commits: 46c16b1a5ef923ddc62fb7c93bbfd5d9212f5627 wpt-pr: 44190 --- .../resources/static-router-no-fetch-handler-sw.js | 35 ++++++++++++++++++++++ .../static-router-no-fetch-handler.https.html | 29 ++++++++++++++++++ 2 files changed, 64 insertions(+) create mode 100644 testing/web-platform/tests/service-workers/service-worker/tentative/static-router/resources/static-router-no-fetch-handler-sw.js create mode 100644 testing/web-platform/tests/service-workers/service-worker/tentative/static-router/static-router-no-fetch-handler.https.html diff --git a/testing/web-platform/tests/service-workers/service-worker/tentative/static-router/resources/static-router-no-fetch-handler-sw.js b/testing/web-platform/tests/service-workers/service-worker/tentative/static-router/resources/static-router-no-fetch-handler-sw.js new file mode 100644 index 000000000000..1ba5fd7d463b --- /dev/null +++ b/testing/web-platform/tests/service-workers/service-worker/tentative/static-router/resources/static-router-no-fetch-handler-sw.js @@ -0,0 +1,35 @@ +'use strict'; + +import {routerRules} from './router-rules.js'; +import { + recordError, + getRecords, + resetRecords } from './static-router-sw.sub.js'; + +self.addEventListener('install', async e => { + e.waitUntil(caches.open('v1').then( + cache => {cache.put('cache.txt', new Response('From cache'))})); + + const params = new URLSearchParams(location.search); + const key = params.get('key'); + try { + await e.addRoutes(routerRules[key]); + } catch (e) { + recordError(e); + } + self.skipWaiting(); +}); + +self.addEventListener('activate', e => { + e.waitUntil(clients.claim()); +}); + +self.addEventListener('message', function(event) { + if (event.data.reset) { + resetRecords(); + } + if (event.data.port) { + const {requests, errors} = getRecords(); + event.data.port.postMessage({requests, errors}); + } +}); diff --git a/testing/web-platform/tests/service-workers/service-worker/tentative/static-router/static-router-no-fetch-handler.https.html b/testing/web-platform/tests/service-workers/service-worker/tentative/static-router/static-router-no-fetch-handler.https.html new file mode 100644 index 000000000000..64678eea9d92 --- /dev/null +++ b/testing/web-platform/tests/service-workers/service-worker/tentative/static-router/static-router-no-fetch-handler.https.html @@ -0,0 +1,29 @@ + + + + Static Router: routers are evaluated when there is no fetch handler. + + + + + + + + + -- 2.11.4.GIT