1697089 - Web Manifest: Remove query & fragment components from scope URL

Status: RESOLVED FIXED

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

Description

[marcos]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_6) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/14.0.3 Safari/605.1.15

Steps to reproduce:

As per spec, manifest processing needs to remove query & fragment components from scope URL.

Actual results:

It currently doesn't.

Expected results:

It should remove query & fragment components from scope URL during processing.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The Bugbug bot thinks this bug should belong to the 'Core::DOM: Core & HTML' component, and is moving the bug to that component. Please revert this change in case you think the bot is wrong.

Comment 2

[marcos]

Attachment: 0001-Web-manifest-Remove-query-fragment-components-from-s.patch

moz-phab is not working for me, so doing this the old fashion way.

Comment 3

[Kagami Rosylight 🏳️‍🌈🏳️‍⚧️ [:saschanaz] (they/them)]

I don't think the old fashion way is accepted anymore... What's blocking you on moz-phab? 👀

Comment 4

[marcos]

Attachment: Web manifest: Remove query & fragment components from scope URL

Comment 5

[marcos]

Did a git remote update and now it works again ::::::P

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5379b7e00612
Web manifest: Remove query & fragment components from scope URL r=saschanaz,annevk

Comment 7

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/5379b7e00612

Comment 8

[marcos]

Thanks for merging it, @saschanaz!

Comment 9

[Kagami Rosylight 🏳️‍🌈🏳️‍⚧️ [:saschanaz] (they/them)]

Thanks for writing it, @marcosc 😊