Closed
Bug 743325
Opened 13 years ago
Closed 13 years ago
Touch events are broken in XUL Fennec
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox13+ fixed, firefox14 fixed)
VERIFIED
FIXED
Firefox 15
People
(Reporter: wesj, Assigned: mbrubeck)
References
Details
(Keywords: regression)
Attachments
(2 files)
(deleted),
patch
|
peterv
:
review+
akeybl
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
peterv
:
review+
akeybl
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
Reported from Kairo in IRC that touch events (even single touch ones) are broken in XUL Fennec. We need some testing to verify.
We're unlikely to ship another release build of XUL Fennec. We may end up pushing this to the Beta channel if Native slips again (although no one has tested Aurora yet to know if it has this problem).
Reporter | ||
Comment 1•13 years ago
|
||
Adding Kairo. I don't think this will be a priority for the team, but want to track the regression. Metro Firefox should also be seeing this (although they'll eventually want to turn off this silly Parent Process MouseEvents->Child process touch events trickery we did.
Assignee | ||
Comment 2•13 years ago
|
||
The tests at http://limpet.net/w3/touchevents/single-touch.html fail with exception "Permission denied for <http://limpet.net> to create wrapper for object of class UnnamedClass."
Keywords: regression,
regressionwindow-wanted
Version: Trunk → Firefox 14
Comment 3•13 years ago
|
||
Also add romaxa, who is doing the N9 builds, that are XUL as well and show that bug as well (I only tried Nightly versions there).
Comment 4•13 years ago
|
||
OK, I did some regression testing, the bug was introduced between the 2012-03-12 and 2012-03-13 13.0a1 mozilla-central builds. This means that Aurora is affected.
FYI, sites to use for testing are http://lantea.kairo.at/ (panning the map - which is what I used for regression testing), http://mandelbrot.kairo.at/ (zooming into the set via dragging over a rendered image), http://browserquest.mozilla.org/ (moving the character via tapping on the playing field). Of course, all those are single-touch events, as XUL Fennec doesn't support multitouch.
Here's more info on the regression range:
Last good build:
http://ftp.mozilla.org/pub/mozilla.org/mobile/nightly/2012-03-12-03-11-36-mozilla-central-android-xul/
20120312031136
http://hg.mozilla.org/mozilla-central/rev/5ec9524de1af
First bad build:
http://ftp.mozilla.org/pub/mozilla.org/mobile/nightly/2012-03-13-03-11-12-mozilla-central-android-xul/
20120313031112
http://hg.mozilla.org/mozilla-central/rev/1ca7a94573f2
Checkin range:
hg.mozilla.org/mozilla-central/pushloghtml?fromchange=5ec9524de1af&tochange=1ca7a94573f2
I searched for "touch" in the pushlog and found this one:
http://hg.mozilla.org/mozilla-central/rev/554ab0a2f470
Peter Van der Beken — Fix for bug 734503 (Add new DOM binding for TouchList). r=jst.
Could that be what triggers this bug?
Reporter | ||
Comment 5•13 years ago
|
||
Looks likely to me.
Comment 6•13 years ago
|
||
Can someone try setting the dom.new_bindings pref to false and check if that fixes it?
Comment 7•13 years ago
|
||
(In reply to Peter Van der Beken [:peterv] from comment #6)
> Can someone try setting the dom.new_bindings pref to false and check if that
> fixes it?
Unfortunately, that doesn't change anything. :(
Comment 8•13 years ago
|
||
yep, with disabled new dom bindings nothing has changed
Comment 9•13 years ago
|
||
Has anyone tried backing out bug 734503 to see if it fixes the issue?
Comment 10•13 years ago
|
||
(In reply to Jim Mathies [:jimm] from comment #9)
> Has anyone tried backing out bug 734503 to see if it fixes the issue?
I don't do my own builds, but if you get around to doing a try build with it backed out, I can test that.
Comment 11•13 years ago
|
||
(In reply to Jim Mathies [:jimm] from comment #9)
> Has anyone tried backing out bug 734503 to see if it fixes the issue?
Tested with that patch backed out, and single touch events works ok,
Sand-trap gape is usable without that patch.
Assignee | ||
Comment 12•13 years ago
|
||
Requesting tracking-firefox13 because this is a regression in XUL Fennec 13 that breaks web content, including popular sites like Google Maps. (We aren't shipping XUL Fennec 13 to the release channel, but we should fix this before it reaches beta.)
Peter, would it be reasonable to back out bug 734503 on Aurora until a better fix is found? Would the backout have any risks or side effects?
Comment 13•13 years ago
|
||
No, I don't think backing out would have much risk.
Assignee | ||
Comment 14•13 years ago
|
||
Assignee | ||
Updated•13 years ago
|
Attachment #617536 -
Attachment description: patch for trunk → backout patch for trunk
Assignee | ||
Comment 15•13 years ago
|
||
[Approval Request Comment]
Regression caused by (bug #): 734503
User impact if declined: Touch events broken in XUL Fennec
Testing completed (on m-c, etc.): Not yet landed on m-c. Existing automated tests cover the code touched by this patch.
Risk to taking this patch (and alternatives if risky): Low-risk backout that reverts the TouchList class from using new-style binding to old-style bindings.
String changes made by this patch: None.
Attachment #617539 -
Flags: review?(peterv)
Attachment #617539 -
Flags: approval-mozilla-aurora?
Updated•13 years ago
|
Attachment #617539 -
Flags: review?(peterv) → review+
Updated•13 years ago
|
Attachment #617536 -
Flags: review?(peterv) → review+
Assignee | ||
Updated•13 years ago
|
Attachment #617536 -
Attachment description: backout patch for trunk → backout patch for trunk and aurora 14
Attachment #617536 -
Flags: approval-mozilla-aurora?
Assignee | ||
Updated•13 years ago
|
Attachment #617539 -
Attachment description: backout patch for aurora → backout patch for beta 13
Attachment #617539 -
Flags: approval-mozilla-aurora? → approval-mozilla-beta?
Assignee | ||
Comment 16•13 years ago
|
||
Target Milestone: --- → Firefox 15
Comment 17•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•13 years ago
|
Comment 18•13 years ago
|
||
Comment on attachment 617536 [details] [diff] [review]
backout patch for trunk and aurora 14
[Triage Comment]
Approved for Aurora 14.
Attachment #617536 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 19•13 years ago
|
||
Comment on attachment 617539 [details] [diff] [review]
backout patch for beta 13
and Beta 13
Attachment #617539 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Assignee | ||
Comment 20•13 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/064758fe4026
https://hg.mozilla.org/releases/mozilla-beta/rev/7e8c20249a4f
This did *not* land in time for 13.0b1 build 1. I'm not sure whether we should hold the build or respin for this...
Comment 21•13 years ago
|
||
Verified that touch events work again in 15.0a1 XUL Nightly from today.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•