Make UIEventAttribution tests less platform dependent
Created attachment 435943 [details]
TestWebKitAPI.EventAttribution.Basic is timing out on iOS.
Comment on attachment 435943 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=435943&action=review
r=me as long as the broken iOS test is fixed.
> +- (void)_addEventAttributionWithSourceID:(uint8_t)sourceID destinationURL:(NSURL *)destination sourceDescription:(NSString *)sourceDescription purchaser:(NSString *)purchaser reportEndpoint:(NSURL *)reportEndpoint nonce:(NSString *)nonce
Should we call the nonce parameter optionalNonce?
> + [webView _addEventAttributionWithSourceID:42 destinationURL:exampleURL() sourceDescription:@"test source description" purchaser:@"test purchaser" reportEndpoint:serverURL nonce:@"ABCDEFabcdef0123456789"];
Where did 42 come from? Did it come from MockEventAttribution before?
This test seems to have been timing out since r277447, which was refactored significantly in r279514
Created attachment 436034 [details]
Comment on attachment 436034 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=436034&action=review
> +#if HAVE(UI_EVENT_ATTRIBUTION)
I verified on Monterey that this is not needed. The test is no longer dependent on UIEventAttribution.