RESOLVED FIXED 237099
Adopt more widely the new URL constructor that takes in a String 
https://bugs.webkit.org/show_bug.cgi?id=237099
Summary Adopt more widely the new URL constructor that takes in a String
Chris Dumez
Reported 2022-02-23 11:50:51 PST
Adopt more widely the new URL constructor that takes in a String.
Attachments
Patch (167.98 KB, patch)
2022-02-23 11:57 PST, Chris Dumez 		no flags
DetailsFormatted DiffDiff
Patch (170.42 KB, patch)
2022-02-23 14:34 PST, Chris Dumez 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2022-02-23 11:57:15 PST
Created attachment 453008 [details] Patch
Darin Adler
Comment 2 2022-02-23 13:56:25 PST
Comment on attachment 453008 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=453008&action=review > Source/WebCore/platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:881 > - URL url(URL(), uriString); > + URL url { uriString }; > return url; I would have merged these into one line. > Source/WebCore/platform/win/PasteboardWin.cpp:614 > + URL kurl { url }; "kurl", ha ha ha
Chris Dumez
Comment 3 2022-02-23 14:34:18 PST
Created attachment 453029 [details] Patch
EWS
Comment 4 2022-02-23 15:19:46 PST
Committed r290394 (247708@main): <https://commits.webkit.org/247708@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 453029 [details].
Radar WebKit Bug Importer
Comment 5 2022-02-23 15:20:26 PST
<rdar://problem/89381477>
Note You need to log in before you can comment on or make changes to this bug.