WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
170216
Add some tests that dump the touch event regions with various content configurations
https://bugs.webkit.org/show_bug.cgi?id=170216
Summary
Add some tests that dump the touch event regions with various content configu...
Simon Fraser (smfr)
Reported
2017-03-28 17:11:37 PDT
Add some tests that dump the touch event regions with various content configurations
Attachments
Patch
(17.08 KB, patch)
2017-03-28 17:13 PDT
,
Simon Fraser (smfr)
dino
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Simon Fraser (smfr)
Comment 1
2017-03-28 17:13:00 PDT
Created
attachment 305678
[details]
Patch
Dean Jackson
Comment 2
2017-03-28 17:18:39 PDT
Comment on
attachment 305678
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=305678&action=review
> LayoutTests/fast/events/touch/ios/touch-event-regions/resources/touch-regions-helper.js:11 > + result += rect.left + ', ' + rect.top + ' - ' + rect.right + ', ' + rect.bottom;
result =+ `${rect.left}, ${rect.left} - ${rect.right}, ${rect.bottom}`; and maybe result =+ `${i ? '\n'}${rect.left}, ${rect.left} - ${rect.right}, ${rect.bottom}`; and then maybe you should just do a map and join rects.map((rect) => `${rect.left}, ${rect.left} - ${rect.right}, ${rect.bottom}`).join('\n')
Simon Fraser (smfr)
Comment 3
2017-03-29 18:32:27 PDT
https://trac.webkit.org/r214590
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug