Bug 188398 - Use fewer Strings as URLs
Summary: Use fewer Strings as URLs
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Alex Christensen
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-08-07 17:48 PDT by Alex Christensen
Modified: 2018-08-08 09:33 PDT (History)
6 users (show)

See Also:


Attachments
Patch (19.40 KB, patch)
2018-08-07 17:49 PDT, Alex Christensen
no flags Details | Formatted Diff | Diff
Patch (20.58 KB, patch)
2018-08-07 18:35 PDT, Alex Christensen
mcatanzaro: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alex Christensen 2018-08-07 17:48:40 PDT
Use fewer Strings as URLs
Comment 1 Alex Christensen 2018-08-07 17:49:08 PDT
Created attachment 346746 [details]
Patch
Comment 2 Alex Christensen 2018-08-07 18:35:42 PDT
Created attachment 346749 [details]
Patch
Comment 3 EWS Watchlist 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
Comment 4 Darin Adler 2018-08-08 09:33:26 PDT
Why?

URL objects are larger, so I’d like to understand the rationale for using them more.