We put these in so we could move touch events forward on mobile with minimum changes to desktop. We should remove them now (assuming the tests pass...)
Created attachment 602153 [details] [diff] [review] Patch Lets see how this fares on try: https://tbpl.mozilla.org/?tree=Try&rev=0aeeb6abc514
Try run for 0aeeb6abc514 is complete. Detailed breakdown of the results available here: https://tbpl.mozilla.org/?tree=Try&rev=0aeeb6abc514 Results (out of 219 total builds): success: 178 warnings: 39 failure: 2 Builds (or logs if builds failed) available at: http://firstname.lastname@example.org
Comment on attachment 602153 [details] [diff] [review] Patch Looks good in tests! I didn't pull the MOZ_TOUCH stuff from gonk.
I landed bug 664707 which touches both GetEventCoordinatesRelativeTo in the same way, so when you land you'll have to account for that.
5 years ago