Currently, the app-bound domains check returns true for special protocols without checking for the WKAppBoundDomains key.
Created attachment 398521 [details]
I confirmed this fix works manually. I was unable to write a test case for the empty WKAppBoundDomains case because TestWebKitAPI has the key in its Info.plist for other tests.
Comment on attachment 398521 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=398521&action=review
Looks good. r=me.
> + // Must check for both an empty app bound domains and an empty key before returning nullopt
"both an empty app bound domains ..." maybe add set? or array? -- the plural domains with the 'an' is throwing me off.
Created attachment 398529 [details]
Patch for landing
Committed r261187: <https://trac.webkit.org/changeset/261187>
All reviewed patches have been landed. Closing bug and clearing flags on attachment 398529 [details].