There's unused code there and it can be refactored a bit
Created attachment 419573 [details] Patch
Comment on attachment 419573 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=419573&action=review > Source/WebCore/platform/network/soup/ResourceRequestSoup.cpp:63 > + GRefPtr<SoupMessage> soupMessage = adoptGRef(soup_message_new_from_uri(httpMethod().ascii().data(), uri.get())); I would have used “auto” here because the “adoptGRef()” + “soup_message_new_foo()” calls already make it quite clear that the inferred type will be “GRefPtr<SoupMessage>” No strong preference, though. > Source/WebCore/platform/network/soup/ResourceRequestSoup.cpp:-174 > - Nice to see some code go away \o/
Committed r272641: <https://trac.webkit.org/changeset/272641>
<rdar://problem/74180921>