Bug 211393 - REGRESSION (r260228): Linker warning about limitsNavigationsToAppBoundDomains property overriding instance methods from class
Summary: REGRESSION (r260228): Linker warning about limitsNavigationsToAppBoundDomains...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: David Kilzer (:ddkilzer)
URL:
Keywords: InRadar
Depends on: 210620
Blocks:
  Show dependency treegraph
 
Reported: 2020-05-04 13:07 PDT by David Kilzer (:ddkilzer)
Modified: 2020-05-08 10:53 PDT (History)
5 users (show)

See Also:


Attachments
Patch v1 (2.28 KB, patch)
2020-05-07 17:25 PDT, David Kilzer (:ddkilzer)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Kilzer (:ddkilzer) 2020-05-04 13:07:11 PDT
Linker warning about limitsNavigationsToAppBoundDomains/setLimitsNavigationsToAppBoundDomains: overriding instance method from class when building WebKit:

ld: warning: instance method 'setLimitsNavigationsToAppBoundDomains:' in category from WebKit.build/Debug/WebKit.build/Objects-normal/x86_64/UnifiedSource24-mm.o overrides method from class in WebKit.build/Debug/WebKit.build/Objects-normal/x86_64/UnifiedSource24-mm.o
ld: warning: instance method 'limitsNavigationsToAppBoundDomains' in category from WebKit.build/Debug/WebKit.build/Objects-normal/x86_64/UnifiedSource24-mm.o overrides method from class in WebKit.build/Debug/WebKit.build/Objects-normal/x86_64/UnifiedSource24-mm.o

I think this was introduced 2-3 weeks ago.  (Might be the cause of flaky behavior if the wrong method is being called.)
Comment 1 David Kilzer (:ddkilzer) 2020-05-04 13:08:41 PDT
I first noticed this on Apr 17, 2020.
Comment 2 David Kilzer (:ddkilzer) 2020-05-07 17:08:39 PDT
This warning means that the @property was added to the public header:

@property (nonatomic) BOOL limitsNavigationsToAppBoundDomains WK_API_AVAILABLE(macos(WK_MAC_TBA), ios(WK_IOS_TBA));

But the implementation was added to a category:

@implementation WKWebViewConfiguration (WKPrivate)
[...]
- (BOOL)limitsNavigationsToAppBoundDomains
{
    return _pageConfiguration->limitsNavigationsToAppBoundDomains();
}

- (void)setLimitsNavigationsToAppBoundDomains:(BOOL)limitsToAppBoundDomains
{
    _pageConfiguration->setLimitsNavigationsToAppBoundDomains(limitsToAppBoundDomains);
}
[...]
@end
Comment 3 David Kilzer (:ddkilzer) 2020-05-07 17:25:15 PDT
Created attachment 398818 [details]
Patch v1
Comment 4 David Kilzer (:ddkilzer) 2020-05-07 17:25:53 PDT
(In reply to David Kilzer (:ddkilzer) from comment #3)
> Created attachment 398818 [details]
> Patch v1

(I got tired of seeing the warning when building. :)
Comment 5 Radar WebKit Bug Importer 2020-05-07 19:45:54 PDT
<rdar://problem/63004163>
Comment 6 Brent Fulgham 2020-05-08 10:19:30 PDT
Comment on attachment 398818 [details]
Patch v1

r=me
Comment 7 EWS 2020-05-08 10:53:27 PDT
Committed r261396: <https://trac.webkit.org/changeset/261396>

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