819741 - Please support ArrayBufferView instances as parameters to XMLHttpRequest.send()

Status: RESOLVED FIXED

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

Description

[Victor Costan]

User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.21 (KHTML, like Gecko) Chrome/25.0.1349.2 Safari/537.21

Steps to reproduce:

arrayBufferView = new Uint8Array(32);
var xhr = new XMLHttpRequest();
xhr.open('http://test_backend.local', 'POST', false);
xhr.send(arrayBufferView);



Actual results:

[Exception... "Illegal value"  nsresult: "0x80070057 (NS_ERROR_ILLEGAL_VALUE)"  location: "JS frame :: Web Console :: <TOP_LEVEL> :: line 3"  data: no]


Expected results:

The send() call should go through.

Comment 1

[Josh Matthews [:jdm]]

The problem isn't the send call; the method and target of the open call are reversed. It should be |xhr.open('POST', 'http://test_backend.local', false);|

Comment 2

[Boris Zbarsky [:bzbarsky]]

The test code given is buggy, but we do not in fact support passing in an ArrayBufferView in a useful way, and it looks like this has been added to the XHR2 spec.

Fix coming up.  send() of a DataView from a worker won't work until bug 819838, I suspect, but the rest should work.  Hard to tell, since I can't find existing worker tests for sending things other than null and a string.

Comment 3

[Boris Zbarsky [:bzbarsky]]

Attachment: Add support for passing an ArrayBufferView to XHR.send().

Ben, please let me know if there are existing worker tests for sending non-strings that I just totally missed that I should add this to

Comment 4

[Ben Turner (not reading bugmail, use the needinfo flag!)]

Comment on attachment 690270 [details] [diff] [review]
Add support for passing an ArrayBufferView to XHR.send().

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

Looks good to me. We don't really have good xhr-post tests for workers, unfortunately. Some day we need to run the main thread xhr tests in a worker.

Comment 5

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Comment on attachment 690270 [details] [diff] [review]
Add support for passing an ArrayBufferView to XHR.send().

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

r=me assuming the answer to the below question is "bytes" :)

::: content/base/src/nsXMLHttpRequest.cpp
@@ +2668,5 @@
> +    }
> +    case nsXMLHttpRequest::RequestBody::ArrayBufferView:
> +    {
> +      return ::GetRequestBody(value.mArrayBufferView->Data(),
> +                              value.mArrayBufferView->Length(), aResult,

Does Length() return the size in bytes or in units?

Comment 6

[Boris Zbarsky [:bzbarsky]]

> Does Length() return the size in bytes or in units?

For a dom::ArrayBufferView, in bytes.

In general, Length() on the DOM typed array types returns the length in sizeof(*Data()), and Data() for an ArrayBufferView returns uint8_t*, for lack of anything better to do.

Comment 7

[Victor Costan]

bz: thank you very much, and apologies for the bad test case!

Comment 8

[Boris Zbarsky [:bzbarsky]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/42213ca59024

Victor, you're welcome, and thank you for filing the bug!

Comment 9

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/42213ca59024