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.

Sign up for GitHub

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

Jump to bottom

"// close() method inherited from WorkerGlobalScope should not be accessible." #865

Closed
annevk opened this issue Apr 2, 2016 · 1 comment
Closed

"// close() method inherited from WorkerGlobalScope should not be accessible." #865

annevk opened this issue Apr 2, 2016 · 1 comment
Labels
decided spec detail
Milestone
Version 1

Comments

@annevk
Copy link
Member

annevk commented Apr 2, 2016

This makes little sense. Could you please upstream this to the HTML Standard by moving close() to dedicated and shared workers?
@jakearchibald jakearchibald added this to the Version 1 milestone Apr 10, 2016
jungkees added a commit to jungkees/html that referenced this issue Apr 25, 2016
@jungkees
Move worker's close() to dedicated worker and shared worker
535e285 
As WorkerGlobalScope's close() method should not be exposed to one of
its derived interfaces, ServiceWorkerGlobalScope, close() needs to be
moved to the derived interfaces which explicitly requires it. This
commit moves close() to DedicatedWorkerGlobalScope and
SharedWorkerGlobalScope.

Related issue: w3c/ServiceWorker#865
@jungkees jungkees mentioned this issue Apr 25, 2016
Merged
annevk pushed a commit to whatwg/html that referenced this issue Apr 27, 2016
@jungkees @annevk
Move worker's close() to dedicated and shared worker
4081f12 
As WorkerGlobalScope's close() method should not be exposed to one of its derived interfaces, ServiceWorkerGlobalScope, close() needs to be moved to the derived interfaces which explicitly require it. This commit moves close() to DedicatedWorkerGlobalScope and SharedWorkerGlobalScope.

Related issue: w3c/ServiceWorker#865.
@nhiroki
Copy link
Contributor

nhiroki commented May 16, 2016

Tracked in http://crbug.com/611640 for Blink

@oliverdunk oliverdunk mentioned this issue Oct 31, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
decided spec detail
Projects
None yet
Milestone
Version 1
Development

No branches or pull requests
4 participants
@jakearchibald @slightlyoff @nhiroki @annevk