Remove MOZ_TOUCH ifdefs

RESOLVED FIXED in mozilla14

Status

()

Core
Layout
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: wesj, Assigned: wesj)

Tracking

Trunk
mozilla14
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
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...)
(Assignee)

Comment 1

5 years ago
Created attachment 602153 [details] [diff] [review]
Patch

Lets see how this fares on try:
https://tbpl.mozilla.org/?tree=Try&rev=0aeeb6abc514
Assignee: nobody → wjohnston

Comment 2

5 years ago
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://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/wjohnston@mozilla.com-0aeeb6abc514
(Assignee)

Comment 3

5 years ago
Comment on attachment 602153 [details] [diff] [review]
Patch

Looks good in tests! I didn't pull the MOZ_TOUCH stuff from gonk.
Attachment #602153 - Flags: review?(bugs)
I landed bug 664707 which touches both GetEventCoordinatesRelativeTo in the same way, so when you land you'll have to account for that.

Updated

5 years ago
Attachment #602153 - Flags: review?(bugs) → review+
(Assignee)

Comment 5

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/b7d5d46662bc
https://hg.mozilla.org/mozilla-central/rev/b7d5d46662bc
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla14
Version: unspecified → Trunk
You need to log in before you can comment on or make changes to this bug.