Currently we define the webidl for RequestCache, but don't implement it. We should remove this webidl until its fully implemented so sites can feature detect it. This should be uplifted to beta/aurora so that they can reasonably feature detect for the next few months until we release the real implementation.

Note, this is just to hide the Request.cache property. We should keep the RequestCache enum since we already set it properly on the Request and plumb it all down into Cache. We just should hide the property so people can feature detect where its useful for or not. Once FetchDriver looks at we can expose the property.

This should probably be behind a pref like RequestContext to avoid removing all our tests and then re-adding them later.

Any chance you have the bandwidth for this? The window is closing before the next merge, and I have some other bugs I'm working on that also need to happen before merge.

Sure!

Comment on attachment 8678152 [details] [diff] [review] Unship Request.cache until the implementation is finished This diff is missing test_request_cache.html. r=me with it added, but I'd like to see that test file.

Sorry for the missing file!

Comment on attachment 8678248 [details] [diff] [review] Unship Request.cache until the implementation is finished Approval Request Comment [Feature/regressing bug #]: We accidentally exposed this attribute without having a full implementation. I'd like to backport this patch that hides it so that web developers can feature detect it. [User impact if declined]: Web developers won't be able to feature detect whether we support Request.cache, and would have to look at the UA string version number of other terrible hacks like that. [Describe test coverage new/current, TreeHerder]: Locally. [Risks and why]: This is pretty harmless. [String/UUID change made/needed]: None.

Comment on attachment 8678248 [details] [diff] [review] Unship Request.cache until the implementation is finished This is really late :/ Taking it because we don't have much choice. Should be in 42 rc1.

https://hg.mozilla.org/mozilla-central/rev/f4bed10ca33f https://hg.mozilla.org/mozilla-central/rev/95f191207f8d

https://hg.mozilla.org/releases/mozilla-aurora/rev/78f1c3caa414 https://hg.mozilla.org/releases/mozilla-aurora/rev/326f764f4148

https://hg.mozilla.org/releases/mozilla-beta/rev/5c768aafbaa7 https://hg.mozilla.org/releases/mozilla-beta/rev/3f2ff85b2f16