NEW 166790
Add a test to ensure that preventDefault on touchmove prevents scrolling
https://bugs.webkit.org/show_bug.cgi?id=166790
Summary Add a test to ensure that preventDefault on touchmove prevents scrolling
Beth Dakin
Reported 2017-01-06 18:02:34 PST
Add a test to ensure that preventDefault on touchmove prevents scrolling rdar://problem/28693432
Attachments
Test (4.58 KB, patch)
2017-01-06 18:04 PST, Beth Dakin
simon.fraser: review+
Beth Dakin
Comment 1 2017-01-06 18:04:27 PST
Simon Fraser (smfr)
Comment 2 2017-01-06 18:09:44 PST
Comment on attachment 298240 [details] Test View in context: https://bugs.webkit.org/attachment.cgi?id=298240&action=review > LayoutTests/fast/events/touch/ios/iphone7/prevent-default-touchmove-prevents-scrolling.html:17 > +#scroller { > + border: 2px solid black; > + height: 400px; > + overflow: scroll; > + position:absolute; > +} > + > +#big { > + width:50px; > + height:2000px; > + background-color:purple; > + border: 15px solid green; > +} Would be nice to be consistent about spaces after colons. > LayoutTests/fast/events/touch/ios/iphone7/prevent-default-touchmove-prevents-scrolling.html:69 > + { > + interpolate : "linear", > + timestep: 0.025, > + startEvent : { > + inputType : "hand", > + timeOffset : 0, > + touches : [ > + { > + inputType : "finger", > + phase : "began", > + id : 1, > + x : ${startX}, > + y : ${startY}, > + pressure : 0 > + } > + ] > + }, > + endEvent : { > + inputType : "hand", > + timeOffset : 3.0, > + touches : [ > + { > + inputType : "finger", > + phase : "stationary", > + id : 1, > + x : ${endX}, > + y : ${endY}, > + pressure : 500 > + } > + ] > + } > + } > + ] > + }; The indentation is funky. > LayoutTests/fast/events/touch/ios/iphone7/prevent-default-touchmove-prevents-scrolling.html:73 > + uiController.uiScriptComplete(); > + }); Here too. > LayoutTests/fast/events/touch/ios/iphone7/prevent-default-touchmove-prevents-scrolling.html:91 > + shouldBe("document.getElementById('scroller').scrollTop", "0"); > + finishJSTest(); > + }); Here too.
Note You need to log in before you can comment on or make changes to this bug.