1650607 - Remove the / → : replacement File constructor

Status: RESOLVED FIXED

(Core :: DOM: File, task)

Description

[Timothy Gu [:TimothyGu]]

See discussion in https://github.com/w3c/FileAPI/issues/41.

Comment 1

[Andrea Marchesini [:baku]]

Subhamoy, do you want to work on this?

Comment 2

[Jens Stutte [:jstutte]]

Hi baku, how should we proceed here?

Comment 3

[Tom S [:evilpie]]

I can take this. We just need to backout bug 1321534.

Comment 4

[Tom S [:evilpie]]

Seems like the test for this wasn't changed yet? https://searchfox.org/mozilla-central/source/testing/web-platform/tests/FileAPI/file/File-constructor.html#87

Comment 5

[Andrea Marchesini [:baku]]

Correct! Thanks for taking this bug.

Comment 6

[Tom S [:evilpie]]

Attachment: Bug 1650607 - Do not replace / with : in File constructor. r?baku

Basically a backout of https://hg.mozilla.org/mozilla-central/rev/1857e763332c.
Pending spec change: https://github.com/w3c/FileAPI/issues/41

Comment 7

[Tom S [:evilpie]]

Attachment: Bug 1650607 - Change WPT test for File constructor replacement. r?baku

Change the test to follow the proposed change in https://github.com/w3c/FileAPI/issues/41,
now that all browsers agree to not replace /.

Depends on D86981

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5e87f862c24b
Do not replace / with : in File constructor. r=baku
https://hg.mozilla.org/integration/autoland/rev/365ae38a5bca
Change WPT test for File constructor replacement. r=baku

Comment 9

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/25046 for changes under testing/web-platform/tests

Comment 10

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/5e87f862c24b
https://hg.mozilla.org/mozilla-central/rev/365ae38a5bca

Comment 11

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot