Closed Bug 946222 Opened 6 years ago Closed 6 years ago

Touch events don't work on gaia without gaia addon

Categories

(DevTools :: Responsive Design Mode, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 28

People

(Reporter: ochameau, Assigned: ochameau)

References

Details

Attachments

(1 file)

For now, gaia instanciates TouchEventHandler on its own, via its helper addon and totally bypass Responsive mode touch event button/codepath.

I'd like to have only one codepath for touch event support on Firefox,
unfortunately, Reponsive mode codepath doesn't work yet with gaia.

That's because we should listen for mouse events on the frame (ie the <browser> element), not on the content window.
Assignee: nobody → poirot.alex
Attachment #8342376 - Flags: review?(paul)
Comment on attachment 8342376 [details] [diff] [review]
Listen on browser, not content window

What about using the chromeEventHandler in touch-events.js instead?

> let chromeEventHandler = window.QueryInterface(Ci.nsIInterfaceRequestor)
>                         .getInterface(Ci.nsIWebNavigation)
>                         .QueryInterface(Ci.nsIDocShell)
>                         .chromeEventHandler;
Attachment #8342376 - Flags: review?(paul) → review-
Blocks: 946231
Comment on attachment 8342376 [details] [diff] [review]
Listen on browser, not content window

Talked to Alex. It's up to the consumer to make sure we pass a chromeEventHandler.
Attachment #8342376 - Flags: review- → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/3b6eb557374c
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 28
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.