Add a WebShare policy quirk for Twitter
<rdar://87944391>
Created attachment 449793 [details] Patch
Created attachment 449797 [details] Patch
Comment on attachment 449797 [details] Patch win failure unrelated
Comment on attachment 449797 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=449797&action=review r=me > Source/WebCore/page/Quirks.cpp:89 > + return topPrivatelyControlledDomain(document.url().host().toString()) == "twitter.com"; Why not use RegistrableDomain instead of calling topPrivatelyControlledDomain() and having to deal with #ifdefs?
(In reply to Chris Dumez from comment #5) > Comment on attachment 449797 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=449797&action=review > > r=me > > > Source/WebCore/page/Quirks.cpp:89 > > + return topPrivatelyControlledDomain(document.url().host().toString()) == "twitter.com"; > > Why not use RegistrableDomain instead of calling > topPrivatelyControlledDomain() and having to deal with #ifdefs? RegistrableDomain constructor is doing a little bit more work. The point on #ifdef is good though.
Created attachment 449920 [details] Patch for landing
Committed r288546 (246378@main): <https://commits.webkit.org/246378@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 449920 [details].