Closed Bug 1377749 Opened 5 years ago Closed 5 years ago

Duplicate characters when changing the text case value on input event

Categories

(Firefox for Android Graveyard :: Keyboards and IME, defect)

54 Branch
defect
Not set
normal

Tracking

(firefox57 fixed)

RESOLVED FIXED
Firefox 57
Tracking Status
firefox57 --- fixed

People

(Reporter: magic, Assigned: jchen)

Details

Attachments

(2 files)

User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/59.0.3071.104 Safari/537.36

Steps to reproduce:

* Open the attachment
* Type 'a'
* Type 'b'




Actual results:

You get 'AAB'


Expected results:

You should get 'AB'
The bug seems occur when the value is transformed (with toUpperCase or a String.replace).
JChen would you take a look at this?
Flags: needinfo?(nchen)
Assignee: nobody → nchen
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Flags: needinfo?(nchen)
Normally, we keep the Java composition when composition is committed on
the Gecko side. However, if we somehow lose the composition on the Java
side (e.g. due to script changing the text), we should reset the
keyboard so it's aware of the change.
Attachment #8897068 - Flags: review?(esawin)
Attachment #8897068 - Flags: review?(esawin) → review+
Pushed by nchen@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/91aeee278ee2
Reset keyboard when composition is lost; r=esawin
https://hg.mozilla.org/mozilla-central/rev/91aeee278ee2
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Flags: needinfo?(nchen)
Pushed by nchen@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/59d0f9b5458b
Follow-up to add null check for editable; r=me
Flags: needinfo?(nchen)
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.