Bug 226640 – FileReader.readAsArrayBuffer misses checking for the type of blob param

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[[e-mail]]

Steps to reproduce:

(1) Open a tab and navigate to any URL;
(2) Run the following code in the Console of Devtools:
```
new FileReader().readAsArrayBuffer(undefined);
```
(3) Then this code would be evaluated successfully without throwing any exception.

Actual results:

This code is evaluated successfully without throwing any exception.

Expected results:

FileReader.readAsArrayBuffer asks a Blob type object as its parameter, but Webkit doesn't check the type of the param in this code. A TypeError should be thrown in this code.

BTW, both Chrome and Firefox throw exceptions:
Chrome: `VM1217:1 Uncaught TypeError: Failed to execute 'readAsArrayBuffer' on 'FileReader': parameter 1 is not of type 'Blob'.`
Firefox: `Uncaught TypeError: FileReader.readAsArrayBuffer: Argument 1 is not an object.`

[Michael Catanzaro]

Hi, please don't use the WebKitGTK component for these general web content bugs. It's for known or suspected platform-specific issues only. You can use the Hardware field to indicate that you're using Linux.

[Radar WebKit Bug Importer]

<rdar://problem/79193198>

[Chris Dumez]

Created attachment 431567 [details]
WIP Patch

[Chris Dumez]

Created attachment 431583 [details]
Patch

[youenn fablet]

Comment on attachment 431583 [details]
Patch

Strange that we do not have any WPT test

[Chris Dumez]

(In reply to youenn fablet from comment #5)
> Comment on attachment 431583 [details]
> Patch
> 
> Strange that we do not have any WPT test

Indeed. I'll resync the WPT tests in a follow-up as we seem a bit out of sync.

[EWS]

Committed r278951 (238880@main): <https://commits.webkit.org/238880@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 431583 [details].