Last Comment Bug 722965 - Remove MOZ_TOUCH ifdefs
: Remove MOZ_TOUCH ifdefs
Product: Core
Classification: Components
Component: Layout (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla14
Assigned To: Wesley Johnston (:wesj)
: Jet Villegas (:jet)
Depends on:
  Show dependency treegraph
Reported: 2012-01-31 19:54 PST by Wesley Johnston (:wesj)
Modified: 2012-03-20 03:52 PDT (History)
3 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Patch (15.38 KB, patch)
2012-03-01 15:26 PST, Wesley Johnston (:wesj)
bugs: review+
Details | Diff | Splinter Review

Description User image Wesley Johnston (:wesj) 2012-01-31 19:54:42 PST
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...)
Comment 1 User image Wesley Johnston (:wesj) 2012-03-01 15:26:04 PST
Created attachment 602153 [details] [diff] [review]

Lets see how this fares on try:
Comment 2 User image Mozilla RelEng Bot 2012-03-01 19:31:08 PST
Try run for 0aeeb6abc514 is complete.
Detailed breakdown of the results available here:
Results (out of 219 total builds):
    success: 178
    warnings: 39
    failure: 2
Builds (or logs if builds failed) available at:
Comment 3 User image Wesley Johnston (:wesj) 2012-03-01 19:51:58 PST
Comment on attachment 602153 [details] [diff] [review]

Looks good in tests! I didn't pull the MOZ_TOUCH stuff from gonk.
Comment 4 User image Timothy Nikkel (:tnikkel) 2012-03-03 13:26:26 PST
I landed bug 664707 which touches both GetEventCoordinatesRelativeTo in the same way, so when you land you'll have to account for that.
Comment 6 User image Mounir Lamouri (:mounir) 2012-03-20 03:52:40 PDT

Note You need to log in before you can comment on or make changes to this bug.