Closed Bug 640025 Opened 9 years ago Closed 8 years ago

Clear text function using ezKeyboard does not work

Categories

(Firefox for Android Graveyard :: General, defect, P4)

ARM
Android
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: nhirata, Assigned: alexp)

Details

(Whiteboard: [vkb])

Attachments

(1 file)

Mozilla/5.0 (Android; Linux armv71; rv2.0b13pre) Gecko/20110308 Firefox/4.0b13pre Fennec/4.0b6pre
Device: Droid 2 
OS: Android 2.2

1. install and use ezKeyboard
2. launch fennec and on a desktop web browser connect to the ip address given on the phone
3. double click on the awesome bar on the android device
4. click on the "clear text on phone" on the web browser on the desktop

Expected: the field will clear
Actual: the field does not clear

Note:
1. works on the android default browser
Whiteboard: [vkb]
Priority: -- → P4
Attached patch FixSplinter Review
Fix for the bug 664364 introduced this problem. ezKeyboard is using commitText() with an empty string to delete the selected text, but with bug 664364 such condition is simply ignored now, which is not correct. Added an additional check: if there is some selection, ask Gecko to delete it to fulfill the IME request.
Assignee: nobody → alexp
Status: NEW → ASSIGNED
Attachment #570065 - Flags: review?(blassey.bugs)
Asked Naoki to test a build with this change to make sure it doesn't regress bug 664364.
Attachment #570065 - Flags: review?(blassey.bugs) → review+
http://hg.mozilla.org/projects/birch/rev/edcfe04cb71a
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Verified fixed Nightly Birch:
Mozilla/5.0 (Android; Linux armv7I; rv:10.0a1) Gecko/20111028 Firefox/10.0a1 Fennec/10.0a1
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.