1507357 - DOM URL object doesn't allow setting pathname when underlying implementation is SimpleURI

Status: RESOLVED FIXED

(Core :: Networking, defect, P2)

Description

[Logan Smyth [:loganfsmyth]]

The 'pathname' setter on the URL constructor doesn't always work.

> var url = new URL("webpack:///foo");
> url.pathname = "/path";
> url.pathname;
> // "///foo"

The extra "//" in the unexpected result is covered by https://bugzilla.mozilla.org/show_bug.cgi?id=1507355 so that can be ignored, but the core of it is that the setter is a no-op. It appears to be unimplemented: https://hg.mozilla.org/mozilla-central/file/tip/netwerk/base/nsSimpleURI.cpp#l773

Comment 1

[Valentin Gosu [:valentin] (he/him)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=9f8dbab9a35212213ed3121a6f35f43b62342a4b

Comment 2

[Valentin Gosu [:valentin] (he/him)]

Attachment: Bug 1507357 - Implement nsSimpleURI::SetFilePath r=kershaw!

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/536e641e0d3f
Implement nsSimpleURI::SetFilePath r=kershaw

Comment 4

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/536e641e0d3f