Closed Bug 747792 Opened 8 years ago Closed 8 years ago
Preventdefault doesn't work for touch events from inside iframe
See url, inside the black bordered box inside the iframes, just make some swipe movements. Notice how the page moves inside Fennec. That should not be happening. Also pinch zoom gestures shouldn't have any effect other than that black lines are drawn inside the black bordered box. Tested on the Samsung Galaxy Nexus. This works at least in a 2012-03-15 trunk build. Also, notice how drawing lines in the top iframe are not bound by the iframe box, which it should. But this is a different bug. I filed bug 747791 for that.
Video here: http://www.youtube.com/watch?v=qzZivRrCo6s
Last good build: 03/09 Bad build: 03/10 Possible regression range: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=9ca66ce2672f&tochange=7702bca6b64d -- Device: HTC Desire Z OS: Android 2.3.3
tracking-fennec: --- → ?
blocking-fennec1.0: --- → ?
Kats - If you think this shouldn't block, just renom and comment
Assignee: nobody → bugmail.mozilla
blocking-fennec1.0: ? → +
Assignee: bugmail.mozilla → wjohnston
Attachment #621173 - Flags: review?(mark.finkle)
To explain the problem, we weren't notifing java of the listeners, so it allowed panning. With our new touch handler, once we've started giving events to the touch handler we don't give the page another chance to stop prevent panning. We did that on purpose because trying to start and stop panning after the page responds leads to confusing situations for the user (and our backend)
Comment on attachment 621173 [details] [diff] [review] Patch I can r+ this, I suspected it was something like this but hadn't started looking for the relevant line of code yet :)
Attachment #621173 - Flags: review?(mark.finkle) → review+
Comment on attachment 621173 [details] [diff] [review] Patch [Approval Request Comment] Regression caused by (bug #): bug 742019 made it visible. always present User impact if declined: iframes will not be able to prevent panning. Testing completed (on m-c, etc.): landed on mc March 4 Risk to taking this patch (and alternatives if risky): Very low risk. Samll change for correctness, and I haven't seen this on real sites ever. String changes made by this patch: None.
Attachment #621173 - Flags: approval-mozilla-aurora?
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 15
Comment on attachment 621173 [details] [diff] [review] Patch [Triage Comment] Very low risk and blocking FN1.0. Approved for Aurora 14.
Attachment #621173 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.