Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Client.reserved references wrong context state? #1030

Closed
wanderview opened this issue Dec 15, 2016 · 3 comments
Closed

Client.reserved references wrong context state? #1030

wanderview opened this issue Dec 15, 2016 · 3 comments

Comments

@wanderview
Copy link
Member

Section 4.2.4 Client.reserved states:

The reserved attribute must return the context object’s associated reserved state.

At:

https://w3c.github.io/ServiceWorker/#client-reserved

But things like caches.matchAll() reference "execution ready flag" instead of "reserved state". See step 2.2.4.1:

https://w3c.github.io/ServiceWorker/#dom-clients-matchall

Should client.reserved also use execution ready flag?

@jungkees
Copy link
Collaborator

A Client is a snapshot captured when Create Client is invoked. So, client.reserved's getter just returns the value captured in the creation time as such instead of the execution ready flag that can dynamically change.

@jakearchibald
Copy link
Contributor

F2F: @wanderview to look at it again.

@jakearchibald
Copy link
Contributor

We aren't going to expose reserved clients, so this is no longer an issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants