Bug 188398

Summary: Use fewer Strings as URLs
Product: WebKit Reporter: Alex Christensen <achristensen>
Component: New BugsAssignee: Alex Christensen <achristensen>
Status: NEW    
Severity: Normal CC: berto, cgarcia, darin, ews-watchlist, gustavo, mcatanzaro
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch mcatanzaro: review+

Alex Christensen
Reported 2018-08-07 17:48:40 PDT
Use fewer Strings as URLs
Attachments
Patch (19.40 KB, patch)
2018-08-07 17:49 PDT, Alex Christensen 		no flags
DetailsFormatted DiffDiff
Patch (20.58 KB, patch)
2018-08-07 18:35 PDT, Alex Christensen 		mcatanzaro: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Alex Christensen
Comment 1 2018-08-07 17:49:08 PDT
Created attachment 346746 [details] Patch
Alex Christensen
Comment 2 2018-08-07 18:35:42 PDT
Created attachment 346749 [details] Patch
EWS Watchlist
Comment 3 2018-08-07 18:38:22 PDT
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API
Darin Adler
Comment 4 2018-08-08 09:33:26 PDT
Why? URL objects are larger, so I’d like to understand the rationale for using them more.
Note You need to log in before you can comment on or make changes to this bug.