In case when the visible content already had renderers it's not sufficient to check the "newly visible" element only since it might just be the container for the clickable content.
Created attachment 364068 [details]
Created attachment 364069 [details]
Comment on attachment 364069 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=364069&action=review
> + In StyleChangeScope we try to figure out whether a newly visible content should stick (menu panes etc) by checking if it is clickable.
"newly visible content"?
> + if (!m_wasHidden || isConsideredHidden())
Would be clearer as if (changedFromHiddenToVisible())
Created attachment 364073 [details]
Comment on attachment 364073 [details]
Clearing flags on attachment: 364073
Committed r242661: <https://trac.webkit.org/changeset/242661>
All reviewed patches have been landed. Closing bug.