1250985 - Prevent "copy constructing" a Request object with navigate mode if a RequestInit member is present

Status: RESOLVED FIXED

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

Description

[(no longer active)]

This is about implementing 12.1 under <https://fetch.spec.whatwg.org/#dom-request>.

Comment 1

[(no longer active)]

Attachment: Part 1: Add a way to identify whether a WebIDL dictionary has any members present

Comment 2

[(no longer active)]

Attachment: Part 2: Prevent copy constructing a Request object with navigate mode if a RequestInit member is present

Comment 3

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 8723147 [details] [diff] [review]
Part 1: Add a way to identify whether a WebIDL dictionary has any members present

This will get the indent wrong for the "has default value" case, no?

You really do want to do this appending right after the conversionReplacements dictionary is set up (or _as_ it's set up, really) and before the if/elif/else bit messes with the indentation of conversionReplacements["convert"].  And then you don't need to add your own indent().

r=me with that fixed.

Comment 4

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 8723148 [details] [diff] [review]
Part 2: Prevent copy constructing a Request object with navigate mode if a RequestInit member is present

Perhaps add a test that passing undefined, null, {}, or { someUnrelatedProperty: 5 } as the second arg all allow the Request constructor to succeed in this case?

I guess if we don't do the respondWith thing we'll just get a page (a 404? or not?) which has a falsy .test_result?

r=me

Comment 5

[(no longer active)]

(In reply to Boris Zbarsky [:bz] from comment #4)
> I guess if we don't do the respondWith thing we'll just get a page (a 404?
> or not?) which has a falsy .test_result?

Yeah if we don't do a respondWith(), the request goes to the network, which turns into a 404 as you suspected.

Will address the rest of the comments when landing.

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/b9fce5b8b83c
https://hg.mozilla.org/integration/mozilla-inbound/rev/97636b12d458

Comment 7

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/b9fce5b8b83c
https://hg.mozilla.org/mozilla-central/rev/97636b12d458