971202 - crash in java.lang.IllegalArgumentException: newEnd does not match text: # vs #-1 at org.mozilla.gecko.GeckoEditable.onTextChange(GeckoEditable.java)

Status: RESOLVED DUPLICATE of bug 1129457

(Firefox for Android Graveyard :: General, defect)

Description

[Aaron Train [:aaronmt]]

This bug was filed from the Socorro interface and is 
report bp-95136bc8-c34a-4c57-97da-359932140209.
=============================================================

java.lang.IllegalArgumentException: newEnd does not match text: 667 vs 666
	at org.mozilla.gecko.GeckoEditable.onTextChange(GeckoEditable.java:854)
	at org.mozilla.gecko.GeckoAppShell.notifyIMEChange(GeckoAppShell.java:453)
	at org.mozilla.gecko.mozglue.GeckoLoader.nativeRun(Native Method)
	at org.mozilla.gecko.mozglue.GeckoLoader.nativeRun(Native Method)
	at org.mozilla.gecko.GeckoAppShell.runGecko(GeckoAppShell.java:359)
	at org.mozilla.gecko.GeckoThread.run(GeckoThread.java:177)

User comment: "Pressing the return key in the comment field at this site had crashed the browser three times now."

Comment 1

[Mark Capella [:capella]]

This has my interest, for possible tie in to 969929 where I report (non-crashing) STR text/data loss in a GechoEditable (which of course affects length).

Comment 2

[Kevin Brosnan [Ex-Mozilla]]

This crash will be hard to track as the signature will depend on the length of the text run. Adding the signatures we know about currently.

Comment 3

[liam]

Although I can't guarantee the signature is the same, a crash occurs on the same site when hitting backspace as well (there's a bug submitted for that).

Comment 4

[Mark Capella [:capella]]

liam: I've tried to repro this on an N7 w/Mobile 30.0a1, but failed. Can you provide a screen shot of the page before you press Enter / Backspace ? Also, can you say which bug you submitted that you refer to in comment #3?

Comment 5

[liam]

Attachment: Screenshot just prior to crash induced by Return key.

Hi mark,

As requested here's a screenshot. 
If you like, I can spin up adb and grab a trace.
As for the other bug, I don't know the number since it was submitted automatically (I believe I left a comment in the field to the effect that the crash was occurred after pressing backspace).

Best/Liam

Comment 6

[Mark Capella [:capella]]

Trace might help, I still can't repro this ... and I'm using the same page, same keyboard IME (google). I've got a current m-c repo with no/custom patches, etc.

I've trying to press both backspace, and enter, while the cursor is positioned in either the title field, and in the HTML editor / comments field.

My N7 is on 4.4.2 build KOT49H, and the google keyboard version is 2.0.19123.914326a

I've also tried but failed to crash using the Jellybean keyboard and the Swift keyboard, and in various combinations of autoCorrect / autoSuggest...

?!?!?   :-(

Comment 7

[liam]

Attachment: logcat for gecko

Hi Mark,

Fault starts at line 1435.
I used the method described at https://wiki.mozilla.org/Android#Using_logcat. Turns out, to do a trace I'd need to make a change to the manifest and re-compile (incidentally, it'd be nice if Mozilla generated such apks automatically for those willing to do the trace). If there's another method I could use let me know.
BTW, I'm running the google keyboard (so, stock). I've also installed lastpass plugin but disabling it didn't help, so I left it on.