RESOLVED FIXED 63073
Adding version of fast/events/offsetX-offsetY.html without text run or line height deps
https://bugs.webkit.org/show_bug.cgi?id=63073
Summary Adding version of fast/events/offsetX-offsetY.html without text run or line h...
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.