.
Created attachment 421515 [details] Patch
Comment on attachment 421515 [details] Patch LGTM with a nit and a comment :) View in context: https://bugs.webkit.org/attachment.cgi?id=421515&action=review > Source/WebCore/ChangeLog:8 > + Use new session getters and setters instead og g_object_get/set. Typo: og → of > Source/WebCore/platform/network/soup/SoupNetworkSession.cpp:113 > + m_soupSession = adoptGRef(soup_session_new_with_options( I guess the idea to move the initialization of “m_soupSession” here was because now it's longer but I don't see any other reason to not keep using the constructor initializer list syntax (and also not needing to move the “maxConnections” and “maxConnectionsPerHost” constants, I guess?) In this case it's not much of an issue because a GRefPtr<> is cheap to construct with the default value, then assign to it inside the constructor body, so feel free to land as-is =)
Committed r273545 (234608@main): <https://commits.webkit.org/234608@main>