Bug 234477 – Use smart pointers for WebCoreNSURLSessionDataTask ObjC members

• Component: New Bugs
• Status: RESOLVED FIXED

[Alex Christensen]

Use smart pointers for WebCoreNSURLSessionDataTask ObjC members

[Alex Christensen]

Created attachment 447521 [details]
Patch

[Alex Christensen]

<rdar://problem/79224869>

[Alex Christensen]

These properties are marked as (copy).  Will do copying and autoreleasing.

[Alex Christensen]

Created attachment 447528 [details]
Patch

[EWS]

Committed r287230 (245391@main): <https://commits.webkit.org/245391@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 447528 [details].

[Darin Adler]

Comment on attachment 447528 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=447528&action=review

> Source/WebCore/platform/network/cocoa/WebCoreNSURLSession.mm:803
> +- (NSURLRequest *)originalRequest
> +{
> +    return adoptNS([_originalRequest copy]).autorelease();
> +}
> +
> +- (NSURLRequest *)currentRequest
> +{
> +    return adoptNS([_currentRequest copy]).autorelease();
> +}
> +
> +- (NSError *)error
> +{
> +    return adoptNS([_error copy]).autorelease();
> +}
> +
> +- (NSString *)taskDescription
> +{
> +    return adoptNS([_taskDescription copy]).autorelease();
> +}
> +
> +- (void)setTaskDescription:(NSString *)description
> +{
> +    _taskDescription = adoptNS([description copy]);
> +}

Was the old code doing these copies and autoreleases? It’s not a given that they must be done just because a smart pointer is involved. They might be good changes in their own right, but need to explain why typically.