1147367 - Add a test for checking the Request.context attribute for FetchEvents generated for fetch()

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Comment 1

[(no longer active)]

Attachment: Add a test for checking the Request.context attribute for FetchEvents generated for fetch()

Comment 2

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

Comment on attachment 8583054 [details] [diff] [review]
Add a test for checking the Request.context attribute for FetchEvents generated for fetch()

Review of attachment 8583054 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/workers/test/serviceworkers/fetch/context/context_test.js
@@ +2,5 @@
> +  if (event.request.url.indexOf("fetch.txt") >= 0) {
> +    var body = event.request.context == "fetch" ?
> +               "so fetch" : "so unfetch";
> +    event.respondWith(new Response(body));
> +  }

Could you add a event.respondWith(Promise.reject()); after the if block so that we never make this sw handle other fetches without finding out about it.

Comment 3

[(no longer active)]

Done!

https://hg.mozilla.org/integration/mozilla-inbound/rev/568da0ccb3ad

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/568da0ccb3ad