They should be in viewToContents/contentsToView functions.
Created attachment 359678 [details] patch
Created attachment 359679 [details] patch
Comment on attachment 359679 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=359679&action=review This LGTMs, thanks! > Source/WebCore/platform/ScrollView.cpp:-864 > - This is reverting what was added in bug 186956. I guess your patch is the right way to fix it. > Source/WebCore/platform/ScrollView.cpp:-877 > - I'm curious if this will fix bug 184297 as the patch I uploaded there does similar thing.
Comment on attachment 359679 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=359679&action=review >> Source/WebCore/platform/ScrollView.cpp:-864 >> - > > This is reverting what was added in bug 186956. I guess your patch is the right way to fix it. Yeah, the result should be the same.
Comment on attachment 359679 [details] patch Clearing flags on attachment: 359679 Committed r240232: <https://trac.webkit.org/changeset/240232>
All reviewed patches have been landed. Closing bug.
<rdar://problem/47427108>