Content filter blocking shield is not displayed on macOS for blocked URLs.
<rdar://91586177>
Created attachment 458966 [details] Patch
Comment on attachment 458966 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=458966&action=review r=me > Source/WebCore/platform/cocoa/ContentFilterUnblockHandlerCocoa.mm:185 > + auto unblocked = false; > + callOnMainThread([decisionHandler, unblocked] { > + decisionHandler(unblocked); > + }); Can unblocked be a local variable rather than a captured variable here?
(In reply to Geoffrey Garen from comment #3) > Comment on attachment 458966 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=458966&action=review > > r=me > > > Source/WebCore/platform/cocoa/ContentFilterUnblockHandlerCocoa.mm:185 > > + auto unblocked = false; > > + callOnMainThread([decisionHandler, unblocked] { > > + decisionHandler(unblocked); > > + }); > > Can unblocked be a local variable rather than a captured variable here? Yes, that is a good point, will fix. Thanks for reviewing!
Created attachment 458976 [details] Patch
Committed r293931 (250378@main): <https://commits.webkit.org/250378@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 458976 [details].