Add argument types to `URL.createObjectURL`

[olyop]

The function URL.createObjectURL currently has an argument of type any. However, this is not in line with the types it would like to be parsed.

I suggest changing the argument from type object: any to object: File | Blob | MediaSource.

This would make make the type more helpful and make it inline with the MDN docs.

I might be missing a type of null or undefined on the union type but I'm not sure if there is a use case for this.

[ulrichstark]

Whoops. I was too fast.

Seems like the type should be changed here: microsoft/TSJS-lib-generator

[andrewbranch]

This seems reasonable to me. N.B., lib.webworker.d.ts must exclude MediaSource.

[olyop]

I plan to make a pull request, however I'm not sure which files to edit. Could you point me the right direction?

[andrewbranch]

I’ve actually never made a lib edit myself, but you can check the Contribution Guidelines here.

[jakebailey]

This was fixed back in #46080.

[isocroft]

Hello @jakebailey . Please, contrary to your statement, this wasn't fixed in #46080 .

It ought to be URL.createObjectURL(obj: File | Blob | MediaSource) and not URL.createObjectURL(obj: Blob | MediaSource).

See here: https://github.com/microsoft/TypeScript/pull/46080/files#diff-e498d298468fe172c74aca00f59e8626b4c1586a331ad8b62574b207e7f643bcR56

I will graciously like to re-open this issue which @olyop opened

[jakebailey]

File extends Blob; there is no reason to include File in the union as it would be reduced away.

[isocroft]

Ok 👍🏾