1260957 - CodeMirror backspace: crash in java.lang.IllegalArgumentException: invalid selection notification range at org.mozilla.gecko.GeckoEditable.onSelectionChange(GeckoEditable.java)

Status: RESOLVED WORKSFORME

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

Description

[Devin Hussey]

This bug was filed from the Socorro interface and is 
report bp-b8708ead-2fae-4ad2-8978-712e72160330.
=============================================================

This may be a dupe of bugs like bug 1051556. But I can reproduce it in 47.0a2 and 45.0.1, despite all of the similar ones being verified fixed. 

CodeMirror crashes Fennec when you backspace on the beginning of a line.

*How to reproduce:*
1. Open up a CodeMirror field. The demo on http://codemirror.org will do.
2. Place your cursor right at the beginning of the line or on an empty line.
3. Press backspace.

*What happened:*

Fennec freezes for a moment and crashes.

Proper behavior happens on WebKit-based browsers.

*What should happen:*

You press backspace and the line break is deleted, and the cursor goes to the end of the previous line. Like any other text editor.

I don't care about the other backspace bugs (https://github.com/codemirror/CodeMirror/issues/3508). 

Firefox 45.0.1; Aurora 47.0a2
Samsung Galaxy S3 (Qualcomm) SGH-I747
KitKat TouchWiz 4.4.2 rooted w/Xposed (nothing directly modifying Firefox. Shouldn't make a difference)

Comment 1

[Kevin Brosnan [Ex-Mozilla]]

This works for me on nightly and aurora, so it looks like this fix will be in Firefox 48. Suspect some of the text selection work Capella has done resolved this. Note we still have a lesser bug that you cannot delete a [cr] or maybe [cr][lf]. I'll file a new bug for the backspace should delete the control characters issue.