<rdar://problem/54410263>
Created attachment 379645 [details] Patch
Comment on attachment 379645 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=379645&action=review > Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm:2191 > + I’m guessing that the unobscuredArea check is for not doing a divide by zero, but if there is no unobscured area, should we be toggling or clearing? Clearing might make sense on this case as well?
Comment on attachment 379645 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=379645&action=review Thanks for the review! >> Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm:2191 >> + > > I’m guessing that the unobscuredArea check is for not doing a divide by zero, but if there is no unobscured area, should we be toggling or clearing? Clearing might make sense on this case as well? Good call — changed this check to `!unobscuredArea || …`.
Created attachment 379660 [details] Patch for landing
Comment on attachment 379660 [details] Patch for landing Clearing flags on attachment: 379660 Committed r250392: <https://trac.webkit.org/changeset/250392>