Closed Bug 1570836 Opened 3 years ago Closed 3 years ago

[wpt-sync] Sync PR 18246 - Removed 'fetch' method from ServiceWorkerGlobalScope

Categories

(Testing :: web-platform-tests, task, P4)

Product:
Testing
Component:
web-platform-tests
Type:
task

Tracking

(firefox70 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla70
Tracking Flags:
Tracking Status
firefox70 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 18246 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/18246
Details from upstream follow.

Artem Herasymenko <arherasy@microsoft.com> wrote:

Removed 'fetch' method from ServiceWorkerGlobalScope

ServiceWorkerGlobalScope does not have 'fetch' method in the spec:
https://w3c.github.io/ServiceWorker/#serviceworkerglobalscope-interface
It should inherit the method from WorkerGlobalScope which is exposed
via partial interface WorkerGlobalScope:
https://cs.chromium.org/chromium/src/third_party/blink/renderer/core/fetch/worker_fetch.idl?q=worker_fetch.&sq=package:chromium&g=0&l=1

This change also adds a new test 'fetch-on-the-right-interface.https.html' to make sure that
'fetch' method is on the right interface.

Bug: 697147
Change-Id: I4d1bd653424561e5a18cc4104fa61ebf48658500

Reviewed-on: https://chromium-review.googlesource.com/1730499
WPT-Export-Revision: 20b01f169856dbe20e87005f88b35fbdfb4e9387

Component: web-platform-tests → DOM: Service Workers
Product: Testing → Core
Component: DOM: Service Workers → web-platform-tests
Product: Core → Testing
Ran 1 tests and 2 subtests
OK     : 1
PASS   : 2
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1cebcb175639
[wpt PR 18246] - Removed 'fetch' method from ServiceWorkerGlobalScope, a=testonly

https://hg.mozilla.org/mozilla-central/rev/1cebcb175639

Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox70: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
You need to log in before you can comment on or make changes to this bug.