776539 - Move FormData to Paris bindings

Status: RESOLVED FIXED

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

Description

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Attachment: WIP

This doesn't build, because of workers.

Comment 1

[Boris Zbarsky [:bzbarsky]]

What's the build failure, exactly?

Comment 2

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Attachment: Patch v1

Comment 3

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 689701 [details] [diff] [review]
Patch v1

>+  "FormData interface constructor": true

Is there a bug covering this?

>+  // void append(DOMString name, Blob value, optional DOMString filename);

You might as well uncomment that and just have the implementation ignore the last arg, which is what would happen anyway right now, just in binding code.  Would make it simpler to later update things to have it work.

r=me with that.

Comment 4

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

(In reply to Boris Zbarsky (:bz) from comment #3)
> Comment on attachment 689701 [details] [diff] [review]
> Patch v1
> 
> >+  "FormData interface constructor": true
> 
> Is there a bug covering this?

Bug 793151, which I guess I should land.

Comment 5

[Boris Zbarsky [:bzbarsky]]

Yes, please!

Comment 6

[Boris Zbarsky [:bzbarsky]]

We should land this sooner rather than later, because until we do xhr send() is kinda broken if passed in a random object...

Comment 7

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

When bz asks...

https://hg.mozilla.org/integration/mozilla-inbound/rev/2fb469965c13

Comment 8

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/2fb469965c13