TextField doesn't allow backspace removal of last character [165956313]

Fixed

Bug

Status Update

No update yet.

Description

se...@google.com

created issue #1

Aug 22, 2020 04:45AM

dev 17
kotlin 1.4

1-function repro :)

@Preview
@Composable
fun PreviewTextField() {
    val (text, setText) = remember { mutableStateOf("") }
    TextField(value = text, onValueChange = setText, label = { })
}

TextField works as expected until user presses backspace to remove last character. setText is not called, but the cursor position is modified.

Selecting the text and using cut does cause setText to be cleared.

Thanks!

Comments

ma...@google.com <ma...@google.com> #2Aug 24, 2020 10:43AM

Have same problem

so...@google.com <so...@google.com> #3Aug 24, 2020 10:48AM

java.util.ConcurrentModificationException
at java.util.WeakHashMap$HashIterator.next(WeakHashMap.java:165)
at androidx.work.impl.utils.WakeLocks.checkWakeLocks(WakeLocks.java:70)
at androidx.work.impl.background.systemalarm.SystemAlarmService.onAllCommandsCompleted(SystemAlarmService.java:69)
at androidx.work.impl.background.systemalarm.SystemAlarmDispatcher.dequeueAndCheckForCompletion(SystemAlarmDispatcher.java:217)
at androidx.work.impl.background.systemalarm.SystemAlarmDispatcher$DequeueAndCheckForCompletion.run(SystemAlarmDispatcher.java:317)
at android.os.Handler.handleCallback(Handler.java:615)
at android.os.Handler.dispatchMessage(Handler.java:92)
at android.os.Looper.loop(Looper.java:137)
at android.app.ActivityThread.main(ActivityThread.java:4960)
at java.lang.reflect.Method.invokeNative(Method.java)
at java.lang.reflect.Method.invoke(Method.java:511)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1038)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:805)
at dalvik.system.NativeStart.main(NativeStart.java)

ma...@google.com <ma...@google.com> #4Aug 24, 2020 10:51AM

https://android-review.googlesource.com/c/platform/frameworks/support/+/857873

ma...@google.com <ma...@google.com> #5Aug 24, 2020 12:08PM

Will this be released in the next beta? Thanks

so...@google.com <so...@google.com> #6Aug 24, 2020 01:25PM

Yes, it will.

so...@google.com <so...@google.com> #7Aug 24, 2020 01:31PM

Seems like it's still not fixed:

Fatal Exception: java.util.ConcurrentModificationException
at java.util.WeakHashMap$HashIterator.next(WeakHashMap.java:165)
at androidx.work.impl.utils.WakeLocks.checkWakeLocks(WakeLocks.java:78)
at androidx.work.impl.background.systemalarm.SystemAlarmService.onAllCommandsCompleted(SystemAlarmService.java:69)
at androidx.work.impl.background.systemalarm.SystemAlarmDispatcher.dequeueAndCheckForCompletion(SystemAlarmDispatcher.java:217)
at androidx.work.impl.background.systemalarm.SystemAlarmDispatcher$DequeueAndCheckForCompletion.run(SystemAlarmDispatcher.java:317)
at android.os.Handler.handleCallback(Handler.java:810)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:189)
at android.app.ActivityThread.main(ActivityThread.java:5532)
at java.lang.reflect.Method.invoke(Method.java)
at java.lang.reflect.Method.invoke(Method.java:372)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:950)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:745)

Message last modified on Aug 24, 2020 01:31PM

si...@google.com <si...@google.com> Aug 24, 2020 05:43PM

Reassigned to so...@google.com.

ap...@google.com <ap...@google.com> #8Aug 26, 2020 08:58AM

Marked as fixed.

Still happened on beta02, Android 5.0.2

vi...@airbnb.com <vi...@airbnb.com> #9Aug 27, 2020 05:45AM

Sample app here:

https://issuetracker.google.com/issues/121345393

It's hard to reproduce with SystemAlarmService.onAllCommandsCompleted. Only managed to do it once but it was just luck. The sample app crashes more often though.

so...@google.com <so...@google.com> #10Aug 27, 2020 09:48AM

I meant this url:

https://github.com/rubensousa/WorkManagerConcurrent