946222 - Touch events don't work on gaia without gaia addon

Status: RESOLVED FIXED

(DevTools :: Responsive Design Mode, defect)

Description

[Alexandre Poirot [:ochameau]]

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.

Comment 1

[Alexandre Poirot [:ochameau]]

Attachment: Listen on browser, not content window

https://tbpl.mozilla.org/?tree=Try&rev=1c2aeb39ca84

Comment 2

[Paul Rouget [: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;

Comment 3

[Paul Rouget [:paul]]

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.

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/fx-team/rev/3b6eb557374c

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/3b6eb557374c