New issue
Advanced search Search tips

Issue 652994 link

Starred by 3 users

Issue metadata

Status: Assigned
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug

Blocked on:
issue 836697
issue 376039
issue 453190

Sign in to add a comment

Remove FetchEvent.isReload

Project Member Reported by, Oct 5 2016

Issue description

FetchEvent.isReload is being removed in favor of Request.cache. See

This bug is blocked on some things though:
(1)  bug 453190 : Implement Request.cache
(2)  bug 376039 : UseCounter for SW (we'll likely need this for intent to remove)
Labels: -Pri-3 M-56 Pri-2
Labels: -M-56 Needs-Review
Let's take a step back on that one.
It seems that isReload provides value over the fetch cache attribute (i.e. more resilient and ergonomic)

Comment 3 by, May 17 2017

> It seems that isReload provides value over the fetch cache attribute (i.e. more resilient and ergonomic)

If we are going to keep it, then its behavior for sub-resources needs to be defined.  The spec was previously quite vague about it.

I proposed one way to clarify the behavior here:

Comment 4 by, May 17 2017

Status: ExternalDependency (was: Available)
ExternalDependency since the spec discussions are not settled.

Note that FetchEvent.isReload is currently not implemented well. See  issue 648761 ,  issue 627481 ,  issue 579866 . If we keep it we need to improve the implementation.
Project Member

Comment 5 by, Jun 28 2017

The following revision refers to this bug:

commit 900dcae2261ada401d6244235def48dedb976619
Author: Matt Falkenhagen <>
Date: Wed Jun 28 10:13:22 2017

service worker: Add UseCounter for FetchEvent#isReload

This feature is at-risk and we need to measure usage.

This patch also fixes a bug where use counter for a page wasn't updated for
features used by the service worker during the main resource load.

Bug: 652994, 737355 
Change-Id: Ib62e6c3ea4d1f39345a978c6c8c6c752fc1bd022
Reviewed-by: Hiroki Nakagawa <>
Reviewed-by: Kentaro Hara <>
Reviewed-by: Kinuko Yasuda <>
Commit-Queue: Matt Falkenhagen <>
Cr-Commit-Position: refs/heads/master@{#482935}

Comment 6 by, Jun 12 2018

Blockedon: 836697
Labels: -Needs-Review
Status: Assigned (was: ExternalDependency)
We can start deprecating this since isReloadNavigation is landing soon.

Sign in to add a comment