Bug 34162

Summary: [Android] [Qt] Touch event page co-ordinates are incorrect when touch is received in an iframe.
Product: WebKit Reporter: Ben Murdoch <benm>
Component: WebCore Misc.Assignee: Ben Murdoch <benm>
Status: RESOLVED FIXED    
Severity: Normal CC: android-webkit-unforking, aurelio.santos, commit-queue, hausmann
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Android   
OS: Android   
Bug Depends on:    
Bug Blocks: 32485    
Attachments:
Description Flags
Proposed patch and updated test.
none
Proposed patch and updated test. hausmann: review+, commit-queue: commit-queue-

Description Ben Murdoch 2010-01-26 05:05:17 PST
The pageX/pageY co-ordinates in a touch should be relative to the containing frame. If the target element is in an iframe, the code currently does not readjust the page co-ordinates to be relative to that iframe but leaves them relative to the main frame.

Patch and test to follow. The patch also renames a variable in the Android specific code to more clearly reflect it's contents.
Comment 1 Ben Murdoch 2010-01-26 05:15:50 PST
Created attachment 47405 [details]
Proposed patch and updated test.
Comment 2 Ben Murdoch 2010-01-26 10:20:22 PST
Created attachment 47420 [details]
Proposed patch and updated test.

This patch additionally removes some dead code.
Comment 3 WebKit Commit Bot 2010-01-28 02:29:22 PST
Comment on attachment 47420 [details]
Proposed patch and updated test.

Rejecting patch 47420 from commit-queue.

Failed to run "['git', 'svn', 'dcommit']" exit_code: 1
Last 500 characters of output:
int.h
	M	WebCore/platform/android/PlatformTouchEventAndroid.cpp
	M	WebCore/platform/android/PlatformTouchPointAndroid.cpp
A repository hook failed: MERGE request failed on '/repository/webkit/trunk': Commit blocked by pre-commit hook (exit code 1) with output:
svnlook: Can't write to stream: Broken pipe

    The following ChangeLog files contain OOPS:

        trunk/LayoutTests/ChangeLog

    Please don't ever say "OOPS" in a ChangeLog file.
 at /usr/local/git/libexec/git-core/git-svn line 558


Full output: http://webkit-commit-queue.appspot.com/results/215684
Comment 4 Ben Murdoch 2010-01-28 02:50:46 PST
My bad, I have a duplicate entry in the change log. Will remove and land manually.
Comment 5 Ben Murdoch 2010-01-28 03:31:41 PST
Landed as r53994.
Comment 6 Aurélio Santos 2013-10-18 08:28:42 PDT
This problem is still occurring to me in some webkit browsers when a CSS transition is applied to the iframe element. The browsers that I tested and saw this problem still happening are:

iOS5 Safari Mobile. Webkit version: 534.46
iOS6 Safari Mobile. Webkit version: 536.26
Android 4.0.3 stock browser. Webkit version: 534.30
Tizen Browser. Webkit version: 537.3

Could there have been a regression?
Comment 7 Aurélio Santos 2013-10-18 09:16:09 PDT
I have created a simple test to show this problem happening on the specified browsers:

http://jsfiddle.net/aurhe/kQ8P4/3/embedded/result/