Keep WeakPtrs instead of raw pointers on WebCoreResourceHandleAsOperationQueueDelegate
Created attachment 387708 [details] Patch
Created attachment 387710 [details] Patch
Created attachment 387711 [details] Patch
Created attachment 387714 [details] Patch
Created attachment 387717 [details] Patch
Created attachment 387748 [details] Patch
Created attachment 387749 [details] Patch
<rdar://problem/57562592>
Comment on attachment 387749 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=387749&action=review r=me if code in CurlRequest.h is okay. > Source/WebCore/platform/network/curl/CurlRequest.h:67 > + static Ref<CurlRequest> create(const ResourceRequest& request, CurlRequestClient& client, ShouldSuspend shouldSuspend = ShouldSuspend::No, EnableMultipart enableMultipart = EnableMultipart::No, CaptureNetworkLoadMetrics captureMetrics = CaptureNetworkLoadMetrics::Basic, RefPtr<SynchronousLoaderMessageQueue>&& messageQueue = nullptr) Is assigning `nullptr` to the default value of a RefPtr<...>&& type allowed?
Created attachment 387824 [details] Patch
Created attachment 387862 [details] Patch
(In reply to David Kilzer (:ddkilzer) from comment #9) > Is assigning `nullptr` to the default value of a RefPtr<...>&& type allowed? It is!
Comment on attachment 387862 [details] Patch Clearing flags on attachment: 387862 Committed r254659: <https://trac.webkit.org/changeset/254659>
All reviewed patches have been landed. Closing bug.
Committed r254673: <https://trac.webkit.org/changeset/254673>