serenity/Userland/Applications/SoundPlayer
Shannon Booth 65f4aae1d8 LibURL+Everywhere: Only percent decode URL paths when actually needed
Web specs do not return through javascript percent decoded URL path
components - but we were doing this in a number of places due to the
default behaviour of URL::serialize_path.

Since percent encoded URL paths may not contain valid UTF-8 - this was
resulting in us crashing in these places.

For example - on an HTMLAnchorElement when retrieving the pathname for
the URL of:

http://ladybird.org/foo%C2%91%91

To fix this make the URL class only return the percent encoded
serialized path, matching the URL spec. When the decoded path is
required instead explicitly call URL::percent_decode.

This fixes a crash running WPT URL tests for the anchor element on:

https://wpt.live/url/a-element.html
(cherry picked from commit cc557323326ba55514ef2a8a6e0efd7f09330f06;
amended heavily to call `URL::percent_decode()` on all results of
`url.serialize_path()` in the rest of serenity -- except in
LibGemini, where it looked incorrect, and in LibHTTP, where
LadybirdBrowser/ladybird#983 will add it.)
2024-11-21 17:47:14 -05:00
..
AlbumCoverVisualizationWidget.cpp
AlbumCoverVisualizationWidget.h
BarsVisualizationWidget.cpp
BarsVisualizationWidget.h
CMakeLists.txt
M3UParser.cpp
M3UParser.h
main.cpp
PlaybackManager.cpp
PlaybackManager.h
Player.cpp
Player.h
Playlist.cpp
Playlist.h
PlaylistWidget.cpp
PlaylistWidget.h
SampleWidget.cpp
SampleWidget.h
SoundPlayerWidget.cpp
SoundPlayerWidget.h
VisualizationWidget.h