Closed
Bug 818715
Opened 12 years ago
Closed 12 years ago
Moving cursor handle in input field causes keyboard to lock up
Categories
(Firefox for Android Graveyard :: Keyboards and IME, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
Firefox 20
People
(Reporter: bnicholson, Assigned: jchen)
Details
Attachments
(1 file)
(deleted),
patch
|
cpeterson
:
review+
|
Details | Diff | Splinter Review |
STR:
1) Go to http://people.mozilla.com/~bnicholson/test/form.html
2) Click inside of the textarea to make the handle appear
3) Drag the handle around in circles for a few seconds
4) Try to type something
The keyboard is completely locked up and does not respond. I added logging to notifyIME() in GeckoAppShell, and it looks like we get loads of notifyIME messages sent when dragging the handle. Each time we move the handle, we generate a click in the text field, so we probably want to suppress IME notifications while we're positioning the cursor.
Updated•12 years ago
|
Assignee: nobody → nchen
Assignee | ||
Comment 1•12 years ago
|
||
We call restartInput on the InputMethodManager during each of the many notifyIME calls, and it tends to overwhelm the InputMethodManager. This patch coalesces the restartInput calls into 200ms bunches, so only one call is made within a 200ms period. I don't think delaying the call for 200ms would have big side effects, since the call is asynchronous anyways.
Attachment #691844 -
Flags: review?(cpeterson)
Comment 2•12 years ago
|
||
Comment on attachment 691844 [details] [diff] [review]
Coalesce restartInput calls in GeckoInputConnection (v1)
Review of attachment 691844 [details] [diff] [review]:
-----------------------------------------------------------------
LGTM
Attachment #691844 -
Flags: review?(cpeterson) → review+
Assignee | ||
Comment 3•12 years ago
|
||
Status: NEW → ASSIGNED
Flags: in-testsuite-
Target Milestone: --- → Firefox 20
Comment 4•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 5•12 years ago
|
||
Firefox 20.0a1 (2012-12-17)
Device: Galaxy Nexus
OS: Android 4.1.1
Using the following STR:
1) Go to http://people.mozilla.com/~bnicholson/test/form.html
2) Click inside of the textarea to make the handle appear
3) Drag the handle around in circles for a few seconds
4) Try to type something
I am not able to reproduce i. Marking bug as VERIFIED FIXED.
Status: RESOLVED → VERIFIED
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•