655212 - Hyperlinks are non-active on tap at maps.google.com

Status: VERIFIED WORKSFORME

(Firefox for Android Graveyard :: General, defect)

Description

[Anna (Waverley)]

Attachment: The "Classic" link is broken.

Build Id: Mozilla /5.0 (Android;Linux armv7l;rv:6.0a1) Gecko/20110505 Firefox/6.0a1 Fennec/6.0a1 

Devices: HTC Desire Z (Android 2.2), LG Optimus 2x (Android 2.2)

Steps to reproduce:
1. Go to maps.google.com.
2. Tap on the third button from left to right to enter the classic mode.
3. A new webpage appears; at the bottom of the page one should see a "Classic" link. Tap on it.

Actual results: 
Nothing happens. The "Classic" link is highlighted but it triggers no action. See the attached screenshot.

Expected results:
When tapping the "Classic" link the user should be directed to the classic version of the web page. 

Note: It works fine on the Android Native Browser.

Comment 1

[Aaron Train [:aaronmt]]

Actually none of those links are responding to a tap. If you tap and hold you can open them in a new tab.

This is a regression as they work normally in 4.0.1

Comment 2

[Brad Lassey [:blassey] (use needinfo?)]

perhaps a regression from touch events landing?

Comment 3

[Wesley Johnston (:wesj)]

Google is calling prevent default on the touch events we fire on the page, which according to the current spec, should also cancel the mouseclick. I wonder if we're hitting some difference between the Android default browser and the iOS one?

Comment 4

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

breaks on 4/30 build, works on 4/29 build.

Mozilla/5.0 (Android; Linux armv71; rv6.0a1) Gecko/20110430 Firefox/6.0a1 Fennec/6.0a1
Device: Thunderbolt
OS: Android 2.2

Comment 5

[Mark Finkle (:mfinkle) (use needinfo?)]

Wes / Matt - what's the plan here? How critical is this bug? I assume it only affects touch event sites, but given that, how bad is it for such sites?

Comment 6

[Wesley Johnston (:wesj)]

This will only affect sites the are calling preventDefault on touchEvents. I spent awhile a week ago trying various changes to our touch event code to see what it takes to make Google recognize the clicks here, with no success. I think I am going to have to take apart the page to make more progress.

Comment 7

[Wesley Johnston (:wesj)]

I've dug into the js-source on maps today. It looks like Google intercepts the touch events and then refires mouse events on the document. If some set of conditions are met (its a single touch, only one finger has been active on the map, they haven't received any touchmove events, etc), they will fire a click.

All of this seems to work fine in a reduced test page for Fennec though. Still need to do more digging to see what's failing and whether its our fault or not.

Comment 8

[Mark Finkle (:mfinkle) (use needinfo?)]

Assigning to Wes so we don't let this drop off radar

Comment 9

[Aaron Train [:aaronmt]]

This works for me on latest-Nightly (07/28) as well as latest-aurora (07/28). Close as WFM?

Comment 10

[Mark Finkle (:mfinkle) (use needinfo?)]

Yep