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

Description Chris Dumez 2022-02-23 11:50:51 PST
Adopt more widely the new URL constructor that takes in a String.
Comment 1 Chris Dumez 2022-02-23 11:57:15 PST
Created attachment 453008 [details]
Patch
Comment 2 Darin Adler 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
Comment 3 Chris Dumez 2022-02-23 14:34:18 PST
Created attachment 453029 [details]
Patch
Comment 4 EWS 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].
Comment 5 Radar WebKit Bug Importer 2022-02-23 15:20:26 PST
<rdar://problem/89381477>