IllegalArgumentException while selecting text in A

Question

I encountered a crash while deselecting text in a selectable Android TextView. This happens when I make a TextView selectable and set LinkMovementMethod.

IllegalArgumentException in TextView

Seems it's a bug inside Android.

java.lang.IllegalArgumentException: Invalid offset: -1. Valid range is [0, 10562]
    at android.text.method.WordIterator.checkOffsetIsValid(WordIterator.java:380)
    at android.text.method.WordIterator.isBoundary(WordIterator.java:101)
    at android.widget.Editor$SelectionStartHandleView.positionAtCursorOffset(Editor.java:4260)
    at android.widget.Editor$HandleView.updatePosition(Editor.java:3708)
    at android.widget.Editor$PositionListener.onPreDraw(Editor.java:2507)
    at android.view.ViewTreeObserver.dispatchOnPreDraw(ViewTreeObserver.java:944)
    at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:2055)
    at android.view.ViewRootImpl.doTraversal(ViewRootImpl.java:1107)
    at android.view.ViewRootImpl$TraversalRunnable.run(ViewRootImpl.java:6013)
    at android.view.Choreographer$CallbackRecord.run(Choreographer.java:858)
    at android.view.Choreographer.doCallbacks(Choreographer.java:670)
    at android.view.Choreographer.doFrame(Choreographer.java:606)
    at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:844)
    at android.os.Handler.handleCallback(Handler.java:739)
    at android.os.Handler.dispatchMessage(Handler.java:95)
    at android.os.Looper.loop(Looper.java:148)
    at android.app.ActivityThread.main(ActivityThread.java:5417)
    at java.lang.reflect.Method.invoke(Native Method)
    at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:726)
    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:616)

Answer 1

Thanks mariotaku for the answer. For some reason it wouldn't me me actually select the text in my TextView. I tweaked it a little to only trigger the work around when some characters are selected. It seems to be working great now!

@Override
public boolean dispatchTouchEvent(MotionEvent event) {
    // FIXME simple workaround to https://code.google.com/p/android/issues/detail?id=191430
    int startSelection = getSelectionStart();
    int endSelection = getSelectionEnd();
    if (startSelection != endSelection) {
        if (event.getActionMasked() == MotionEvent.ACTION_DOWN) {
            final CharSequence text = getText();
            setText(null);
            setText(text);
        }
    }
    return super.dispatchTouchEvent(event);
}

Answer 2

I found Android will clear TextView selection when calling setText, so here's a simple workaround to this problem.

public class FixedTextView extends TextView {

    public FixedTextView(final Context context) {
        super(context);
    }

    public FixedTextView(final Context context, final AttributeSet attrs) {
        super(context, attrs);
    }

    public FixedTextView(final Context context, final AttributeSet attrs, final int defStyle) {
        super(context, attrs, defStyle);
    }

    @Override
    public boolean dispatchTouchEvent(MotionEvent event) {
        // FIXME simple workaround to https://code.google.com/p/android/issues/detail?id=191430
        if (event.getActionMasked() == MotionEvent.ACTION_DOWN) {
            final CharSequence text = getText();
            setText(null);
            setText(text);
        }
        return super.dispatchTouchEvent(event);
    }

}

Answer 3

Here is a simple perfectly working workaround. You should put this into your Activity class:

@Override
public void onActionModeStarted(ActionMode mode) {
    super.onActionModeStarted(mode);
    yourTextView.setMovementMethod(ArrowKeyMovementMethod.getInstance());
}

@Override
public void onActionModeFinished(ActionMode mode) {
    super.onActionModeFinished(mode);
    yourTextView.setMovementMethod(LinkMovementMethod.getInstance());
}