<rdar://problem/75505210>
Created attachment 423767 [details] Patch
Comment on attachment 423767 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=423767&action=review r=me > Source/WebCore/html/HTMLElement.cpp:1240 > + if (auto host = node.shadowHost(); is<HTMLElement>(host) && downcast<HTMLElement>(*host).hasImageOverlay()) > + return makeRefPtr(downcast<HTMLElement>(*host)); NIT: Can we avoid the double dereference-`downcast`?
Comment on attachment 423767 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=423767&action=review >> Source/WebCore/html/HTMLElement.cpp:1240 >> + return makeRefPtr(downcast<HTMLElement>(*host)); > > NIT: Can we avoid the double dereference-`downcast`? Yes, fixed!
Created attachment 423783 [details] Patch
Committed r274772: <https://commits.webkit.org/r274772> All reviewed patches have been landed. Closing bug and clearing flags on attachment 423783 [details].