New issue
Advanced search Search tips

Issue 865839 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 678905



Sign in to add a comment

update-after-navigation-redirect.https.html times out

Project Member Reported by falken@chromium.org, Jul 20

Issue description

This test times out because Chrome doesn't currently update intermediate service workers in the redirect chain on navigation.

It will be fixed in S13nServiceWorker and probably wontfix'd for non-S13nSW.
 
Components: Blink>ServiceWorker
Project Member

Comment 2 by bugdroid1@chromium.org, Jul 20

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/97f692d32ef358b8fee1a02387a4846c86fed931

commit 97f692d32ef358b8fee1a02387a4846c86fed931
Author: Matt Falkenhagen <falken@chromium.org>
Date: Fri Jul 20 07:33:28 2018

WPT: service worker: Add test for updating due to redirects in navigations.

Handle Fetch algorithm:
  If request is a non-subresource request...invoke Soft Update algorithm
  with registration.
https://w3c.github.io/ServiceWorker/#on-fetch-request-algorithm


Bug: 865839
Change-Id: I56eb447313171eebd2da4b4eb79771e8b89e212b
Reviewed-on: https://chromium-review.googlesource.com/1144584
Commit-Queue: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: Kenichi Ishibashi <bashi@chromium.org>
Reviewed-by: Makoto Shimazu <shimazu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#576805}
[modify] https://crrev.com/97f692d32ef358b8fee1a02387a4846c86fed931/third_party/WebKit/LayoutTests/TestExpectations
[add] https://crrev.com/97f692d32ef358b8fee1a02387a4846c86fed931/third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/update-after-navigation-redirect.https.html

Sign in to add a comment