Simplify PCM::Client interface
Created attachment 437303 [details] Patch
Comment on attachment 437303 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=437303&action=review r=me > Source/WebKit/NetworkProcess/PrivateClickMeasurement/PrivateClickMeasurementNetworkLoader.cpp:50 > + request.setHTTPBody(WebCore::FormData::create(jsonPayload->toJSONString().utf8())); You dropped a .data() here, was that intentional?
Yes, I think that removed an unnecessary CString copy.
Committed r282031 (241335@main): <https://commits.webkit.org/241335@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 437303 [details].
<rdar://problem/82741931>
View in context: https://bugs.webkit.org/attachment.cgi?id=437303&action=review LGTM. See comment. > Source/WebKit/NetworkProcess/PrivateClickMeasurement/PrivateClickMeasurementNetworkLoader.cpp:41 > +static NetworkLoadParameters generateNetworkLoadParameters(URL&& url, RefPtr<JSON::Object>&& jsonPayload, PrivateClickMeasurement::PcmDataCarried pcmDataCarried) This function does the exact same thing as the three deleted ones above, right? Except we lost the UNLIKELY(isDebugModeEnabled) which I don't know how much it gave us.
Right.