652958 - FormHelper does not work well with IME compositions

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Matt Brubeck (:mbrubeck)]

Attachment: screenshot

Steps to reproduce:
1. Use an input method that offers word suggestions, e.g. "Android keyboard."
2. Open a page with a text field, e.g. http://people.mozilla.com/~mbrubeck/test/form.html
3. Tap on the text field.
4. Type a letter that matches the first letter of a suggestion.
5. Tap the suggestion.

Expected results: The suggestion appears in the text field.

Actual results: The typed character is removed, the other letters of the suggestion appear in the text field, and the first letters in the text field are underlined to indicate they are part of a composition.

I can reproduce this bug on T-Mobile G2 (Android 2.2) in Firefox 4.0, 5.0a2, and trunk.

Comment 1

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

Vivien - is this a Form Helper bug or IME?

Comment 2

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Attachment: Patch

Force a composition end when using the autocomplete mechanism

Comment 3

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

http://hg.mozilla.org/mozilla-central/rev/4db7b12df9ae

Comment 4

[Andreea Pod]

Mozilla /5.0 (Android;Linux armv7l;rv:6.0a1) Gecko/20110516 Firefox/6.0a1 Fennec/6.0a1
Mozilla /5.0 (Android;Linux armv7l;rv:5.0a2) Gecko/20110516 Firefox/5.0a2 Fennec/5.0a2

Device: Samsung Captivate (Android 2.1 update 1)

I just verified this on both above builds and on nightly it is fixed but on aurora I can still see the problem. Should I reopen this bug?

Comment 5

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

> I just verified this on both above builds and on nightly it is fixed but on
> aurora I can still see the problem. Should I reopen this bug?

No, the bug is fixed.

Comment 6

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

> I just verified this on both above builds and on nightly it is fixed but on
> aurora I can still see the problem. Should I reopen this bug?

The bug is "tracking-fennec: 6+" which means it is only planned to land in Fx6. If we wanted the bug fixed in Fx 5, that would require landing in mozilla-aurora too.

Comment 7

[Andreea Pod]

Marking it as verified fixed.