Bug 153985 - Remove __weak from WKAirPlayRoutePicker.mm to fix build warning.
Summary: Remove __weak from WKAirPlayRoutePicker.mm to fix build warning.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jeremy Jones
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2016-02-08 07:39 PST by Jeremy Jones
Modified: 2017-11-02 11:34 PDT (History)
2 users (show)

See Also:

Attachments
Patch (1.55 KB, patch)
2016-02-08 07:42 PST, Jeremy Jones 		darin: review+
Details | Formatted Diff | Diff
Patch for landing. (1.51 KB, patch)
2016-02-08 10:28 PST, Jeremy Jones 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jeremy Jones 2016-02-08 07:39:18 PST 
Remove __weak from WKAirPlayRoutePicker.mm to fix build warning.
Comment 1 Jeremy Jones 2016-02-08 07:40:30 PST 
rdar://problem/24485348
Comment 2 Jeremy Jones 2016-02-08 07:42:55 PST 
Created attachment 270854 [details]
Patch
Comment 3 Darin Adler 2016-02-08 08:54:19 PST 
Comment on attachment 270854 [details]
Patch

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

> Source/WebKit2/UIProcess/ios/forms/WKAirPlayRoutePicker.mm:53
> -    __weak WKContentView* _view;   // Weak reference.
> +    __unsafe_unretained WKContentView* _view; // Weak reference.

Since this code is not ARC, we should not have any modifier at all. Is there any value to stating __unsafe_unretained here?
Comment 4 Darin Adler 2016-02-08 08:54:43 PST 
Comment on attachment 270854 [details]
Patch

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

>> Source/WebKit2/UIProcess/ios/forms/WKAirPlayRoutePicker.mm:53
>> +    __unsafe_unretained WKContentView* _view; // Weak reference.
> 
> Since this code is not ARC, we should not have any modifier at all. Is there any value to stating __unsafe_unretained here?

I suggest removing __weak and not replacing it with anything.
Comment 5 Jeremy Jones 2016-02-08 10:25:21 PST 
(In reply to comment #4)
> Comment on attachment 270854 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=270854&action=review
> 
> >> Source/WebKit2/UIProcess/ios/forms/WKAirPlayRoutePicker.mm:53
> >> +    __unsafe_unretained WKContentView* _view; // Weak reference.
> > 
> > Since this code is not ARC, we should not have any modifier at all. Is there any value to stating __unsafe_unretained here?
> 
> I suggest removing __weak and not replacing it with anything.

If this were a header that might get included in an ARC code, it would be necessary. Since it is in an .mm file, the only value is in annotating the fact that this is an unretained reference.  I'll remove it.
Comment 6 Jeremy Jones 2016-02-08 10:28:41 PST 
Created attachment 270863 [details]
Patch for landing.
Comment 7 WebKit Commit Bot 2016-02-08 11:27:03 PST 
Comment on attachment 270863 [details]
Patch for landing.

Clearing flags on attachment: 270863

Committed r196260: <http://trac.webkit.org/changeset/196260>