Bug 237099

Summary: Adopt more widely the new URL constructor that takes in a String
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: WebCore Misc.Assignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, darin, ggaren, sam, webkit-bug-importer, youennf
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 237068    
Bug Blocks:    
Attachments:
Description Flags
Patch
none
Patch none

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.