Bug 63073

Summary: Adding version of fast/events/offsetX-offsetY.html without text run or line height deps
Product: WebKit Reporter: Matthew Delaney <mdelaney7>
Component: Tools / TestsAssignee: Matthew Delaney <mdelaney7>
Status: RESOLVED FIXED    
Severity: Normal CC: joepeck, simon.fraser
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch simon.fraser: review+

Matthew Delaney
Reported 2011-06-21 08:54:30 PDT
Adding in new version of fast/events/offsetX-offsetY.html that does not depend on text runs and line-height for the position of the targets. Since other ports may still have a use for the original test, I'll probably just leave it alone in this patch.
Attachments
Patch (10.11 KB, patch)
2011-06-21 09:11 PDT, Matthew Delaney
no flags
Patch (9.54 KB, patch)
2011-06-21 10:33 PDT, Matthew Delaney
simon.fraser: review+
Matthew Delaney
Comment 1 2011-06-21 09:11:56 PDT
Simon Fraser (smfr)
Comment 2 2011-06-21 10:14:13 PDT
Comment on attachment 97994 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=97994&action=review > LayoutTests/fast/events/document-elementFromPoint.html:216 > + document.body.ontouchstart = function(e) { > + var touch = e.touches[0]; > + var resultBox = document.getElementById('mouse-position'); > + var clientRect = e.target.getBoundingClientRect(); > + var offsets = 'element id: ' + e.target.id + '<br>'; > + offsets += 'clientX: ' + touch.clientX + ' clientY: ' + touch.clientY + '<br>'; > + offsets += '~offsetX: ' + (clientRect.left - touch.clientX) + ' ~offsetY: ' + (clientRect.top - touch.clientY); > + resultBox.innerHTML = offsets; > + // Do not prevent default, so we can scroll. > + } Remove this.
Matthew Delaney
Comment 3 2011-06-21 10:33:29 PDT
Matthew Delaney
Comment 4 2011-06-21 10:47:58 PDT
Note You need to log in before you can comment on or make changes to this bug.