Closed Bug 764025 Opened 12 years ago Closed 12 years ago

Move composition commit from onTextChanged() to afterTextChanged()

Categories

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

Product:
Firefox for Android Graveyard
Component:
Keyboards and IME
Platform:
ARM
Android
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: cpeterson, Assigned: cpeterson)

References

Details

When we commit the composition string from onTextChanged(), Gecko callbacks can reenter Java with text change notifications when Android IME does not permit the Editable text to be modified. Deferring composition commit to afterTextChange() allows Gecko's text change notifications to be processed safely.
Blocks: 751513
This work should not be necessary after bug 769520 fixes the real cause of these re-entrant text change events.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Depends on: 769520
Resolution: --- → WONTFIX
Reopening because of the backout of bug 769250.
Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---
This work is no longer necessary after jchen's IME rewrite.
Status: REOPENED → RESOLVED
Closed: 12 years ago → 12 years ago
Resolution: --- → WONTFIX
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.