WebPlatformTouchPoint() gets the screen position as a 3rd parameter and contents position as a fourth parameter, But we are passing the contents position as a 3rd parameter and screen position as a fourth parameter. So, I want to correct that.
Created attachment 175331 [details] Patch
This looks good to me. I think it would also help if we could consistently order the arguments everywhere so that screen/device position comes first and content position follows (e.g. in WebEventFactory::createWebTouchEvent(), NativeWebTouchEvent::NativeWebTouchEvent()). Kenneth, what do you think ?
Comment on attachment 175331 [details] Patch Could this be tested somehow? At least the content position part should be testable
Comment on attachment 175331 [details] Patch Clearing flags on attachment: 175331 Committed r135491: <http://trac.webkit.org/changeset/135491>
All reviewed patches have been landed. Closing bug.
(In reply to comment #3) > (From update of attachment 175331 [details]) > Could this be tested somehow? At least the content position part should be testable Actually, I found and tested this problem on Tizen. I agree with you and we need test cases to test positions.