1188062 - Unship Request.context

Status: RESOLVED FIXED

(Core :: DOM: Service Workers, defect)

Description

[(no longer active)]

According to our IRC conversations today, we decided that we are not yet ready to ship Request.context.  See <https://github.com/whatwg/fetch/issues/93#issuecomment-125289983> which is my feedback on the spec.

Comment 1

[(no longer active)]

Attachment: Unship Request.context

Comment 2

[Andrea Marchesini [:baku]]

Comment on attachment 8639547 [details] [diff] [review]
Unship Request.context

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

::: dom/fetch/Request.cpp
@@ +14,5 @@
>  #include "mozilla/dom/Fetch.h"
>  #include "mozilla/dom/Promise.h"
>  #include "mozilla/dom/URL.h"
>  #include "mozilla/dom/workers/bindings/URL.h"
> +#include "mozilla/dom/WorkerPrivate.h"

alphabetic order.

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/73828c770d56

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/73828c770d56

Comment 5

[Chris Mills [:cmills]]

I've added a footnote to the browser compat table to make this clear:

https://developer.mozilla.org/en-US/docs/Web/API/Request/context

The page has been marked deprecated too, as the feature is no longer in the spec.

I've also checked, and a note has been included in the Firefox release notes:

https://developer.mozilla.org/en-US/Firefox/Releases/42

Comment 6

[Kohei Yoshino]

The site compat doc is here:

https://www.fxsitecompat.com/en-US/docs/2015/request-context-has-been-removed/