Allow modern decoding of URLs
Created attachment 323690 [details] Patch
Comment on attachment 323690 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=323690&action=review r=me with suggestion. > Source/WebCore/platform/URL.h:299 > +std::optional<URL> URL::decode(Decoder& decoder) When you do this sort of things, could we rewrite the legacy decoder to use the modern one to avoid duplication? e.g. template <class Decoder> bool URL::decode(Decoder& decoder, URL& url) { auto optionalURL = URL::decode(decoder); if (!optionalURL) return false; url = optionalURL.value(); return true; }
Created attachment 323929 [details] Patch
Comment on attachment 323929 [details] Patch Clearing flags on attachment: 323929 Committed r223446: <https://trac.webkit.org/changeset/223446>
All reviewed patches have been landed. Closing bug.
<rdar://problem/35018342>