766566 - Text-selection on a hyperlink makes selection handles jump to seemingly random positions on page

Status: RESOLVED INVALID

(Firefox for Android Graveyard :: Text Selection, defect)

Description

[Aaron Train [:aaronmt]]

Currently when one initiates a text selection on a hyperlink and drags the text selection handles onto the text, the selection handles will jump to different positions on the page.

See video: http://www.youtube.com/watch?v=Aj6zmJSD7Go&hd=1

--
Nightly (06.20)
Galaxy Nexus (Android 4.0.4)

Comment 1

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

Since we are basically using the same mechanism as we did in XUL Fennec, I wonder if the bug happened in XUL Fennec too.

In fact, we could look at any still-open text selection bugs we had in XUL Fennec and test for those situations.

Comment 2

[Aaron Train [:aaronmt]]

Take your pick https://bugzilla.mozilla.org/showdependencytree.cgi?id=661388&hide_resolved=1

I remember bug 677896 also caused weird behaviour on hyperlinks but it's not reproducible on Native.

Comment 3

[Wesley Johnston (:wesj)]

How are we showing the handles on a link? In XUL we let context menus win over text selection:

http://mxr.mozilla.org/mozilla-central/source/mobile/xul/chrome/content/browser.js#1892

Comment 4

[:Margaret Leibovic]

(In reply to Wesley Johnston (:wesj) from comment #3)
> How are we showing the handles on a link? In XUL we let context menus win
> over text selection:
> 
> http://mxr.mozilla.org/mozilla-central/source/mobile/xul/chrome/content/
> browser.js#1892

We do let context menus win over text selection:
http://mxr.mozilla.org/mozilla-central/source/mobile/android/chrome/content/browser.js#1252

So, yeah, I wonder how we're initiating text selection on a hyperlink.

Comment 5

[Paul Feher]

I can reproduce this issue on Firefox Mobile 15.0b6(2012-08-22) using Samsung Galaxy Tab (Android 3.1). Also in some cases when one handler is positioned over a link, the position is kept but the highlighted area of the text selected is dismissed.

Comment 6

[Mark Capella [:capella]]

Pruning some old bugs obviated by new Core/Layout AccessibleCaret implementation, or being no longer observable.

(Please re-open if appropriate.)