Updating bottom padding of PaddingValues and then passing it to LazyColumn(contentPadding = ...) sometimes does not recompose the lambda inside of rememberLazyListMeasurePolicy() [263402091]

Fixed

Bug

Status Update

No update yet.

Description

ke...@gmail.com

created issue #1

Dec 22, 2022 07:02AM

Jetpack Compose version: 1.4.0-alpha03

Jetpack Compose component(s) used: LazyColumn

Android Studio Build: Android Studio Electric Eel | 2022.1.1 RC 1

Kotlin version: 1.7.20

Originally, I asked this question on #compose slack channel but I was asked to report it here.

"Hi. I'm currently using Compose 1.4.0-alpha03 and I'm experiencing a weird behavior where updating bottom padding of PaddingValue which is then passed to LazyColumn(contentPadding = ...) doesn't actually use the new padding inside of the rememberLazyListMeasurePolicy. The bottom padding is updated with the keyboard height everytime it appears/disappears. I'm placing two breakpoints 1. Inside of the internal fun LazyList(...) function on state.placementAnimator = placementAnimator line. 2. Inside of the private fun rememberLazyListMeasurePolicy() function on checkScrollableContainerConstraints() line. Then I'm clicking a textfield to make the keyboard appear. Once the first breakpoint is hit I first check that the bottom padding is the value I'm expecting to see (~300dp) then I place a new breakpoint inside of PaddingValuesImpl.equals() method. Then I resume the app, PaddingValuesImpl.equals() is getting called a couple of times but it's called not from the place I'm expecting it to get called from. I'm expecting a call from remember<LazyLayoutMeasureScope.(Constraints) -> MeasureResult>() { ... } line which is in rememberLazyListMeasurePolicy() method but it never gets called from that method. Then the second breakpoint which is on checkScrollableContainerConstraints() line is getting hit and when I check the contentPadding I'm seeing the old value (24dp). So I'm assuming that the remember<LazyLayoutMeasureScope.(Constraints) -> MeasureResult>() { ... } doesn't work for some reason in this situation so the lambda is not re-remebered in other words it's the previous lambda that captured the previous contentPadding. I would really like to see what's going on inside remember<LazyLayoutMeasureScope.(Constraints) -> MeasureResult>() { ... } but I can't place a breakpoint there because it tells me to download the source code and when I click download it can't find it. I tried installing the compose-runtime dependency but it still won't show me the source code of that remember<LazyLayoutMeasureScope.(Constraints) -> MeasureResult>() { ... } function. This feels like a bug in that one remember function. However I also tried to create a new project and copy some of the code to try and reproduce the issue but it works just fine there (same compose version). So I'm kinda lost now. I'm not sure I should report this on the issue tracker because I have no MRE and it also works in a new test project but doesn't work in the main project. The app I'm developing is open-source so I can provide the whole project if needed."

STR:

Clone this branch: https://github.com/K1rakishou/KurobaExLite/tree/lazycolumn_remember_weird_behavior
Click "Select a catalog" text in the toolbar.
Scroll to the very bottom (100th item).
Click the "Search" icon.
Keyboard will show up. Scroll to bottom again you will see that it can't scroll past ~94th item.

If you do the steps described above (the one with breakpoints) you will see the bug. I'm also attaching a video where I'm doing those steps.

As I have already said above I tried to create a MRE in a new project but was surprised to see that it works there flawlessly. Also I have multiple places in this project where I'm using the same method to set the bottom padding of the LazyColumn when the keyboard shows up and it works in some of those places.

studio64_MvsnAZTM9k.mp4

11 MB

Download

Comments

ma...@google.com <ma...@google.com> Dec 22, 2022 10:56AM

Reassigned to as...@google.com.

as...@google.com <as...@google.com> #2Dec 23, 2022 03:21AM

when click backspace, we call DeleteSurroundingTextInCodePoints, then the app with textField crash.
简单来说就是按删除按键的时候，调用了DeleteSurroundingTextInCodePoints ，然后，编辑框所在应用竟然崩了，报出了上述堆栈BUG。这么低级的错误不应该发生的。

Message last modified on Dec 23, 2022 03:22AM

ke...@gmail.com <ke...@gmail.com> #3Dec 23, 2022 04:23AM

Could you please clarify if you are able to reproduce it with soft keyboard or only with the dedicated keyboard attached?

as...@google.com <as...@google.com> #4Dec 29, 2022 07:34PM

Surely it can be reproduced. This is a bug in the source code, which can be reproduced no matter what the soft keyboard, but another condition is that the soft keyboard calls DeleteSurroundingTextInCodePoints(), regardless of whether there is text in the input box. When the compose framework receives the delete command, it will check the text in the input box. If it finds that the text in the input box does not match the length of the delete command to be deleted, an exception is thrown. Then, the APP crashed. Note that it is not the input method crashed.
--From Google Translate

Message last modified on Dec 29, 2022 07:43PM

as...@google.com <as...@google.com> #5Dec 29, 2022 08:34PM

Como arreglar errores

Message last modified on Jan 5, 2023 08:51PM

as...@google.com <as...@google.com> #6Dec 29, 2022 08:36PM

We are having similar crash in our app, I am not able to reproduce it but it seems like it is coming from play stores monkey test runners. all users who crash has email something like normalnames.nnnnn@gmail.com

Fatal Exception: java.lang.IllegalArgumentException: end cannot negative. [end: -1]
at androidx.compose.ui.text.TextRangeKt.packWithCheck(TextRangeKt.java:124)
at androidx.compose.ui.text.TextRangeKt.TextRange(TextRangeKt.java:37)
at androidx.compose.ui.text.input.EditingBuffer.delete$ui_text_release(EditingBuffer.kt:190)
at androidx.compose.ui.text.input.DeleteSurroundingTextCommand.applyTo(EditCommand.kt:271)
at androidx.compose.ui.text.input.EditProcessor.apply(EditProcessor.kt:93)
at androidx.compose.foundation.text.TextFieldKeyInput.apply(TextFieldKeyInput.kt:62)
at androidx.compose.foundation.text.TextFieldKeyInput.access$apply(TextFieldKeyInput.java:49)
at androidx.compose.foundation.text.TextFieldKeyInput$process$2.invoke(TextFieldKeyInput.kt:134)
at androidx.compose.foundation.text.TextFieldKeyInput$process$2.invoke(TextFieldKeyInput.kt:102)
at androidx.compose.foundation.text.TextFieldKeyInput.commandExecutionContext(TextFieldKeyInput.java:211)
at androidx.compose.foundation.text.TextFieldKeyInput.process-ZmokQxo(TextFieldKeyInput.java:102)
at androidx.compose.foundation.text.TextFieldKeyInputKt$textFieldKeyInput$2$1.invoke-ZmokQxo(TextFieldKeyInput.kt:243)
at androidx.compose.foundation.text.TextFieldKeyInputKt$textFieldKeyInput$2$1.invoke(TextFieldKeyInput.kt:243)
at androidx.compose.ui.node.ModifiedKeyInputNode.propagateKeyEvent-ZmokQxo(ModifiedKeyInputNode.kt:47)
at androidx.compose.ui.input.key.KeyInputModifier.processKeyInput-ZmokQxo(KeyInputModifier.java:78)
at androidx.compose.ui.platform.AndroidComposeView.sendKeyEvent-ZmokQxo(AndroidComposeView.android.kt:563)
at androidx.compose.ui.platform.AndroidComposeView.dispatchKeyEvent(AndroidComposeView.android.kt:570)
at android.view.ViewGroup.dispatchKeyEvent(ViewGroup.java:1640)
at android.view.ViewGroup.dispatchKeyEvent(ViewGroup.java:1640)
at android.view.ViewGroup.dispatchKeyEvent(ViewGroup.java:1640)
at android.view.ViewGroup.dispatchKeyEvent(ViewGroup.java:1640)
at android.view.ViewGroup.dispatchKeyEvent(ViewGroup.java:1640)
at android.view.ViewGroup.dispatchKeyEvent(ViewGroup.java:1640)
at android.view.ViewGroup.dispatchKeyEvent(ViewGroup.java:1640)
at com.android.internal.policy.PhoneWindow$DecorView.superDispatchKeyEvent(PhoneWindow.java:2395)
at com.android.internal.policy.PhoneWindow.superDispatchKeyEvent(PhoneWindow.java:1727)
at androidx.core.view.KeyEventDispatcher.activitySuperDispatchKeyEventPre28(KeyEventDispatcher.java:130)
at androidx.core.view.KeyEventDispatcher.dispatchKeyEvent(KeyEventDispatcher.java:87)
at androidx.core.app.ComponentActivity.dispatchKeyEvent(ComponentActivity.java:140)
at androidx.appcompat.app.AppCompatActivity.dispatchKeyEvent(AppCompatActivity.java:599)
at androidx.appcompat.view.WindowCallbackWrapper.dispatchKeyEvent(WindowCallbackWrapper.java:59)
at androidx.appcompat.app.AppCompatDelegateImpl$AppCompatWindowCallback.dispatchKeyEvent(AppCompatDelegateImpl.java:3090)
at com.android.internal.policy.PhoneWindow$DecorView.dispatchKeyEvent(PhoneWindow.java:2310)
at android.view.ViewRootImpl$ViewPostImeInputStage.processKeyEvent(ViewRootImpl.java:4127)
at android.view.ViewRootImpl$ViewPostImeInputStage.onProcess(ViewRootImpl.java:4089)
at android.view.ViewRootImpl$InputStage.deliver(ViewRootImpl.java:3642)
at android.view.ViewRootImpl$InputStage.onDeliverToNext(ViewRootImpl.java:3695)
at android.view.ViewRootImpl$InputStage.forward(ViewRootImpl.java:3661)
at android.view.ViewRootImpl$AsyncInputStage.forward(ViewRootImpl.java:3787)
at android.view.ViewRootImpl$InputStage.apply(ViewRootImpl.java:3669)
at android.view.ViewRootImpl$AsyncInputStage.apply(ViewRootImpl.java:3844)
at android.view.ViewRootImpl$InputStage.deliver(ViewRootImpl.java:3642)
at android.view.ViewRootImpl$InputStage.onDeliverToNext(ViewRootImpl.java:3695)
at android.view.ViewRootImpl$InputStage.forward(ViewRootImpl.java:3661)
at android.view.ViewRootImpl$InputStage.apply(ViewRootImpl.java:3669)
at android.view.ViewRootImpl$InputStage.deliver(ViewRootImpl.java:3642)
at android.view.ViewRootImpl$InputStage.onDeliverToNext(ViewRootImpl.java:3695)
at android.view.ViewRootImpl$InputStage.forward(ViewRootImpl.java:3661)
at android.view.ViewRootImpl$AsyncInputStage.forward(ViewRootImpl.java:3820)
at android.view.ViewRootImpl$ImeInputStage.onFinishedInputEvent(ViewRootImpl.java:3981)
at android.view.inputmethod.InputMethodManager$PendingEvent.run(InputMethodManager.java:2253)
at android.view.inputmethod.InputMethodManager.invokeFinishedInputEventCallback(InputMethodManager.java:1874)
at android.view.inputmethod.InputMethodManager.finishedInputEvent(InputMethodManager.java:1865)
at android.view.inputmethod.InputMethodManager$ImeInputEventSender.onInputEventFinished(InputMethodManager.java:2230)
at android.view.InputEventSender.dispatchInputEventFinished(InputEventSender.java:141)
at android.os.MessageQueue.nativePollOnce(MessageQueue.java)
at android.os.MessageQueue.next(MessageQueue.java:323)
at android.os.Looper.loop(Looper.java:135)
at android.app.ActivityThread.main(ActivityThread.java:5421)
at java.lang.reflect.Method.invoke(Method.java)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:726)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:616)

ap...@google.com <ap...@google.com> #7Oct 17, 2023 01:43PM

This is happening to me also and its incredibly annoying! Del button on samsung android tablet keyboard will just crash everything.

E/AndroidRuntime: FATAL EXCEPTION: main
    Process: com.dealermade.next.photostudio, PID: 9116
    java.lang.IllegalArgumentException: end cannot negative. [end: -1]
        at androidx.compose.ui.text.TextRangeKt.packWithCheck(TextRange.kt:124)
        at androidx.compose.ui.text.TextRangeKt.TextRange(TextRange.kt:37)
        at androidx.compose.ui.text.input.EditingBuffer.delete$ui_text_release(EditingBuffer.kt:205)
        at androidx.compose.ui.text.input.DeleteSurroundingTextCommand.applyTo(EditCommand.kt:271)
        at androidx.compose.ui.text.input.EditProcessor.apply(EditProcessor.kt:110)
        at androidx.compose.foundation.text.TextFieldKeyInput.apply(TextFieldKeyInput.kt:62)
        at androidx.compose.foundation.text.TextFieldKeyInput.access$apply(TextFieldKeyInput.kt:49)
        at androidx.compose.foundation.text.TextFieldKeyInput$process$2.invoke(TextFieldKeyInput.kt:134)
        at androidx.compose.foundation.text.TextFieldKeyInput$process$2.invoke(TextFieldKeyInput.kt:102)
        at androidx.compose.foundation.text.TextFieldKeyInput.commandExecutionContext(TextFieldKeyInput.kt:211)
        at androidx.compose.foundation.text.TextFieldKeyInput.process-ZmokQxo(TextFieldKeyInput.kt:102)
        at androidx.compose.foundation.text.TextFieldKeyInputKt$textFieldKeyInput$2$1.invoke-ZmokQxo(TextFieldKeyInput.kt:243)
        at androidx.compose.foundation.text.TextFieldKeyInputKt$textFieldKeyInput$2$1.invoke(TextFieldKeyInput.kt:243)
        at androidx.compose.ui.node.ModifiedKeyInputNode.propagateKeyEvent-ZmokQxo(ModifiedKeyInputNode.kt:47)
        at androidx.compose.ui.input.key.KeyInputModifier.processKeyInput-ZmokQxo(KeyInputModifier.kt:78)
        at androidx.compose.ui.platform.AndroidComposeView.sendKeyEvent-ZmokQxo(AndroidComposeView.android.kt:562)
        at androidx.compose.ui.platform.AndroidComposeView.dispatchKeyEvent(AndroidComposeView.android.kt:569)
        at android.view.ViewGroup.dispatchKeyEvent(ViewGroup.java:1986)
        at android.view.ViewGroup.dispatchKeyEvent(ViewGroup.java:1986)
        at android.view.ViewGroup.dispatchKeyEvent(ViewGroup.java:1986)
        at android.view.ViewGroup.dispatchKeyEvent(ViewGroup.java:1986)
        at com.android.internal.policy.DecorView.superDispatchKeyEvent(DecorView.java:903)
        at com.android.internal.policy.PhoneWindow.superDispatchKeyEvent(PhoneWindow.java:1947)
        at android.app.Activity.dispatchKeyEvent(Activity.java:4142)
        at androidx.core.app.ComponentActivity.superDispatchKeyEvent(ComponentActivity.java:126)
        at androidx.core.view.KeyEventDispatcher.dispatchKeyEvent(KeyEventDispatcher.java:86)
        at androidx.core.app.ComponentActivity.dispatchKeyEvent(ComponentActivity.java:144)
        at io.sentry.android.core.internal.gestures.WindowCallbackAdapter.dispatchKeyEvent(WindowCallbackAdapter.java:29)
        at io.sentry.android.core.internal.gestures.SentryWindowCallback.dispatchKeyEvent(SentryWindowCallback.java:13)
        at com.android.internal.policy.DecorView.dispatchKeyEvent(DecorView.java:758)
        at android.view.ViewRootImpl$ViewPostImeInputStage.processKeyEvent(ViewRootImpl.java:7365)
        at android.view.ViewRootImpl$ViewPostImeInputStage.onProcess(ViewRootImpl.java:7191)
        at android.view.ViewRootImpl$InputStage.deliver(ViewRootImpl.java:6595)
        at android.view.ViewRootImpl$InputStage.onDeliverToNext(ViewRootImpl.java:6652)
        at android.view.ViewRootImpl$InputStage.forward(ViewRootImpl.java:6618)
        at android.view.ViewRootImpl$AsyncInputStage.forward(ViewRootImpl.java:6786)
        at android.view.ViewRootImpl$InputStage.apply(ViewRootImpl.java:6626)
        at android.view.ViewRootImpl$AsyncInputStage.apply(ViewRootImpl.java:6843)
        at android.view.ViewRootImpl$InputStage.deliver(ViewRootImpl.java:6599)
        at android.view.ViewRootImpl$InputStage.onDeliverToNext(ViewRootImpl.java:6652)
        at android.view.ViewRootImpl$InputStage.forward(ViewRootImpl.java:6618)
        at android.view.ViewRootImpl$InputStage.apply(ViewRootImpl.java:6626)
        at android.view.ViewRootImpl$InputStage.deliver(ViewRootImpl.java:6599)
        at android.view.ViewRootImpl.deliverInputEvent(ViewRootImpl.java:9880)
        at android.view.ViewRootImpl.doProcessInputEvents(ViewRootImpl.java:9718)
E/AndroidRuntime:     at android.view.ViewRootImpl.enqueueInputEvent(ViewRootImpl.java:9671)
        at android.view.ViewRootImpl$ViewRootHandler.handleMessage(ViewRootImpl.java:6206)
        at android.os.Handler.dispatchMessage(Handler.java:106)
        at android.os.Looper.loop(Looper.java:246)
        at android.app.ActivityThread.main(ActivityThread.java:8653)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:602)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1130)

// Top-level build file where you can add configuration options common to all sub-projects/modules.
buildscript {
    ext.kotlin_version = '1.6.10'
    ext.apollo_version = '2.5.11'
    ext.jetpack_compose_version = '1.2.0-alpha04'
    ext.coil_version = "1.3.2"
    repositories {
        google()
        mavenCentral()
        maven { url 'https://jitpack.io' }
    }
    dependencies {
        classpath 'com.android.tools.build:gradle:7.1.2'
        classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:1.6.10"
        classpath "io.realm:realm-gradle-plugin:10.8.0"
        // https://docs.mongodb.com/realm/sdk/android/quick-start-local/
        // NOTE: Do not place your application dependencies here; they belong
        // in the individual module build.gradle files, unless of course it is a build plugin, like realm or kotlin :)
    }
}

allprojects {
    repositories {
        google()
        maven { url "https://jitpack.io" }
        mavenCentral()
    }
}

task clean(type: Delete) {
    delete rootProject.buildDir
}

plugins {
    id 'com.android.application'
    id 'kotlin-android'
    id("com.apollographql.apollo").version("$apollo_version")
    id "io.sentry.android.gradle" version "3.0.0-beta.2"
}
apply plugin: 'kotlin-kapt'
apply plugin: 'realm-android'

configurations.all {
    resolutionStrategy.force "org.antlr:antlr4-runtime:4.9.2"
    resolutionStrategy.force "org.antlr:antlr4-tool:4.9.2"
}

sentry {
    // Enables or disables the automatic upload of mapping files
    // during a build. If you disable this, you'll need to manually
    // upload the mapping files with sentry-cli when you do a release.
    // Default is enabled.
    autoUpload = true

    // Disables or enables the automatic configuration of Native Symbols
    // for Sentry. This executes sentry-cli automatically so
    // you don't need to do it manually.
    // Default is disabled.
    uploadNativeSymbols = true

    // Does or doesn't include the source code of native code for Sentry.
    // This executes sentry-cli with the --include-sources param. automatically so
    // you don't need to do it manually.
    // Default is disabled.
    includeNativeSources = true

    // Enable or disable the tracing instrumentation.
    // Does auto instrumentation for 'androidx.sqlite' and 'androidx.room' libraries.
    // It starts and finishes a Span within any CRUD operation.
    // Default is enabled.
    // Only available v3.0.0 and above.
    tracingInstrumentation {
        enabled = true
    }
}

android {
    signingConfigs {

    }
    compileSdk 32
    buildToolsVersion '32.0.0'

    defaultConfig {
        minSdkVersion 29
        targetSdkVersion 31
        versionCode 1
        versionName "1.2"

        testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
        vectorDrawables {
            useSupportLibrary true
        }
    }

    buildTypes {
        release {
            signingConfig signingConfigs.release
            shrinkResources = true
            minifyEnabled = true
            proguardFiles getDefaultProguardFile('proguard-android-optimize.txt'), 'proguard-rules.pro'
        }
        debug {
            versionNameSuffix 'debug'
        }
    }
    compileOptions {
        sourceCompatibility JavaVersion.VERSION_1_8
        targetCompatibility JavaVersion.VERSION_1_8
    }
    kotlinOptions {
        jvmTarget = '1.8'
        useIR = true
    }
    buildFeatures {
        viewBinding true
        compose true
    }
    composeOptions {
        kotlinCompilerExtensionVersion jetpack_compose_version
    }
    packagingOptions {
        resources {
            excludes += '/META-INF/{AL2.0,LGPL2.1}'
        }
    }
}

apollo {
    generateKotlinModels.set(true)
}

dependencies {
    implementation("androidx.compose.compiler:compiler:$jetpack_compose_version")
    implementation 'com.google.android.gms:play-services-cronet:18.0.1'

    implementation("io.coil-kt:coil:$coil_version")
    implementation("io.coil-kt:coil-compose:$coil_version")
    // https://github.com/coil-kt/coil or using jetpack https://coil-kt.github.io/coil/compose/
    implementation 'com.google.android.play:core-ktx:1.8.1'
    implementation 'androidx.paging:paging-common-ktx:3.1.0'
    implementation 'com.google.accompanist:accompanist-insets:0.24.3-alpha'
    //debugImplementation 'com.squareup.leakcanary:leakcanary-android:2.7'
    //https://square.github.io/leakcanary/fundamentals-how-leakcanary-works/
    implementation "org.jetbrains.kotlin:kotlin-stdlib:$kotlin_version"
    implementation 'id.zelory:compressor:3.0.1'
    implementation 'org.jetbrains.kotlinx:kotlinx-coroutines-android:1.6.0-native-mt'
    implementation 'androidx.core:core-ktx:1.9.0-alpha01'
    implementation 'androidx.appcompat:appcompat:1.4.1'
    implementation 'com.google.android.material:material:1.5.0'
    implementation 'androidx.constraintlayout:constraintlayout:2.1.3'
    implementation 'androidx.lifecycle:lifecycle-livedata-ktx:2.4.1'
    implementation 'androidx.lifecycle:lifecycle-viewmodel-ktx:2.4.1'
    implementation 'androidx.navigation:navigation-fragment-ktx:2.4.1'
    implementation 'androidx.navigation:navigation-ui-ktx:2.4.1'
    implementation "androidx.compose.ui:ui-tooling-preview:$jetpack_compose_version"
    implementation 'androidx.lifecycle:lifecycle-runtime-ktx:2.4.1'
    implementation "androidx.datastore:datastore:1.0.0"
    implementation "androidx.datastore:datastore-preferences:1.0.0"

    testImplementation 'junit:junit:4.13.2'
    androidTestImplementation 'androidx.test.ext:junit:1.1.3'
    androidTestImplementation 'androidx.test.espresso:espresso-core:3.4.0'
    implementation('androidx.paging:paging-runtime-ktx:3.1.0')
    implementation('androidx.paging:paging-compose:1.0.0-alpha14')
    //jetpack added
    implementation('androidx.activity:activity-compose:1.4.0')
    implementation("androidx.compose.ui:ui:$jetpack_compose_version")
    // Tooling support (Previews, etc.)
    implementation("androidx.compose.ui:ui-tooling:$jetpack_compose_version")
    // Foundation (Border, Background, Box, Image, Scroll, shapes, animations, etc.)
    implementation("androidx.compose.foundation:foundation:$jetpack_compose_version")
    // Material Design
    implementation("androidx.compose.material:material:$jetpack_compose_version")
    // Material design icons
    implementation("androidx.compose.material:material-icons-core:$jetpack_compose_version")
    implementation("androidx.compose.material:material-icons-extended:$jetpack_compose_version")
    implementation("androidx.compose.material:material-ripple:$jetpack_compose_version")
    // Integration with observables
    implementation("androidx.compose.runtime:runtime-livedata:$jetpack_compose_version")
    implementation("androidx.compose.runtime:runtime-rxjava2:$jetpack_compose_version")
    implementation 'com.google.mlkit:barcode-scanning:17.0.2'
    def camerax_version = '1.1.0-beta02'
    def camera_view = '1.1.0-beta02'
// CameraX core library using camera2 implementation
    implementation "androidx.camera:camera-core:$camerax_version"
    implementation "androidx.camera:camera-camera2:$camerax_version"
// CameraX Lifecycle Library
    implementation "androidx.camera:camera-lifecycle:$camerax_version"
// CameraX View class
    implementation "androidx.camera:camera-view:$camera_view"
    // If you want to additionally use the CameraX Extensions library
    implementation("androidx.camera:camera-extensions:$camera_view")
    //apollo added
    implementation("com.apollographql.apollo:apollo-runtime:$apollo_version")
    // optional: for coroutines support
    implementation("com.apollographql.apollo:apollo-coroutines-support:$apollo_version")
    // optional: Most of apollo-android does not depend on Android in practice and runs on any JVM or on Kotlin native. apollo-android-support contains a few Android-only helper classes. For an example to send logs to logcat or run callbacks on the main thread.
    implementation("com.apollographql.apollo:apollo-android-support:$apollo_version")
    implementation 'com.github.javafaker:javafaker:1.0.2'
    implementation 'io.sentry:sentry-android:6.0.0-alpha.2'

    implementation 'io.intercom.android:intercom-sdk:10.6.2'
    implementation 'com.google.accompanist:accompanist-permissions:0.20.0'
// Compose tooling
    implementation 'androidx.compose.ui:ui-tooling:1.2.0-alpha04'
// Compose Constraint layout
    implementation 'androidx.constraintlayout:constraintlayout-compose:1.0.0'
//    re-orderable lazyColumn
    implementation('org.burnoutcrew.composereorderable:reorderable:0.7.4')
//    Placeholder UI
    implementation('com.google.accompanist:accompanist-placeholder:0.19.0')
//    Adding Gson
    implementation('com.google.code.gson:gson:2.9.0')
// J2mod
    implementation('com.ghgande:j2mod:3.1.1')
//    Google nearby api
    implementation('com.google.android.gms:play-services-nearby:18.0.2')
    // UI Tests
    androidTestImplementation("androidx.compose.ui:ui-test-junit4:$jetpack_compose_version")
}

ap...@google.com <ap...@google.com> #8Nov 9, 2023 02:54PM

Thank you for the details!

al...@google.com <al...@google.com> #9Mar 14, 2024 07:54PM

I get the exact same error from the play console testing. In my case, it is caused on a Nexus 5X. I even did a check before, but it still occurs regularly and I don't understand why. The original code was take from here.

I am using version 1.2.0-alpha04 for all compose dependencies.

My implementation:

    val start = newText.length
    val end = newText.length

    if (start <= 0 || end <= 0) {
        Log.debug { "START OR END NEGATIVE: start $start, end $end, value $this, newString $newString" }
        return this
    }

    val newSelection = TextRange(
        start = start,
        end = end
    )

The crash log:

java.lang.IllegalArgumentException: end cannot negative. [end: -1]
    at androidx.compose.ui.text.TextRangeKt.packWithCheck(TextRange.kt:124)
    at androidx.compose.ui.text.TextRangeKt.TextRange(TextRange.kt:37)
    at androidx.compose.ui.text.input.EditingBuffer.delete$ui_text_release(EditingBuffer.kt:205)
    at androidx.compose.ui.text.input.DeleteSurroundingTextCommand.applyTo(EditCommand.kt:271)
    at androidx.compose.ui.text.input.EditProcessor.apply(EditProcessor.kt:110)
    at androidx.compose.foundation.text.TextFieldKeyInput.apply(TextFieldKeyInput.kt:62)
    at androidx.compose.foundation.text.TextFieldKeyInput.access$apply(TextFieldKeyInput.kt:49)
    at androidx.compose.foundation.text.TextFieldKeyInput$process$2.invoke(TextFieldKeyInput.kt:134)
    at androidx.compose.foundation.text.TextFieldKeyInput$process$2.invoke(TextFieldKeyInput.kt:102)
    at androidx.compose.foundation.text.TextFieldKeyInput.commandExecutionContext(TextFieldKeyInput.kt:211)
    at androidx.compose.foundation.text.TextFieldKeyInput.process-ZmokQxo(TextFieldKeyInput.kt:102)
    at androidx.compose.foundation.text.TextFieldKeyInputKt$textFieldKeyInput$2$1.invoke-ZmokQxo(TextFieldKeyInput.kt:243)
    at androidx.compose.foundation.text.TextFieldKeyInputKt$textFieldKeyInput$2$1.invoke(TextFieldKeyInput.kt:243)
    at androidx.compose.ui.node.ModifiedKeyInputNode.propagateKeyEvent-ZmokQxo(ModifiedKeyInputNode.kt:47)
    at androidx.compose.ui.input.key.KeyInputModifier.processKeyInput-ZmokQxo(KeyInputModifier.kt:78)
    at androidx.compose.ui.platform.AndroidComposeView.sendKeyEvent-ZmokQxo(AndroidComposeView.android.kt:562)
    at androidx.compose.ui.platform.AndroidComposeView.dispatchKeyEvent(AndroidComposeView.android.kt:569)
    at android.view.ViewGroup.dispatchKeyEvent(ViewGroup.java:1640)
    at android.view.ViewGroup.dispatchKeyEvent(ViewGroup.java:1640)
    at android.view.ViewGroup.dispatchKeyEvent(ViewGroup.java:1640)
    at android.view.ViewGroup.dispatchKeyEvent(ViewGroup.java:1640)
    at android.view.ViewGroup.dispatchKeyEvent(ViewGroup.java:1640)
    at android.view.ViewGroup.dispatchKeyEvent(ViewGroup.java:1640)
    at com.android.internal.policy.PhoneWindow$DecorView.superDispatchKeyEvent(PhoneWindow.java:2395)
    at com.android.internal.policy.PhoneWindow.superDispatchKeyEvent(PhoneWindow.java:1727)
    at androidx.core.view.KeyEventDispatcher.activitySuperDispatchKeyEventPre28(KeyEventDispatcher.java:136)
    at androidx.core.view.KeyEventDispatcher.dispatchKeyEvent(KeyEventDispatcher.java:89)
    at androidx.core.app.ComponentActivity.dispatchKeyEvent(ComponentActivity.java:142)
    at androidx.appcompat.app.AppCompatActivity.dispatchKeyEvent(AppCompatActivity.java:599)
    at androidx.appcompat.view.WindowCallbackWrapper.dispatchKeyEvent(WindowCallbackWrapper.java:59)
    at androidx.appcompat.app.AppCompatDelegateImpl$AppCompatWindowCallback.dispatchKeyEvent(AppCompatDelegateImpl.java:3090)
    at com.android.internal.policy.PhoneWindow$DecorView.dispatchKeyEvent(PhoneWindow.java:2310)
    at android.view.ViewRootImpl$ViewPostImeInputStage.processKeyEvent(ViewRootImpl.java:4127)
    at android.view.ViewRootImpl$ViewPostImeInputStage.onProcess(ViewRootImpl.java:4089)
    at android.view.ViewRootImpl$InputStage.deliver(ViewRootImpl.java:3642)
    at android.view.ViewRootImpl$InputStage.onDeliverToNext(ViewRootImpl.java:3695)
    at android.view.ViewRootImpl$InputStage.forward(ViewRootImpl.java:3661)
    at android.view.ViewRootImpl$AsyncInputStage.forward(ViewRootImpl.java:3787)
    at android.view.ViewRootImpl$InputStage.apply(ViewRootImpl.java:3669)
    at android.view.ViewRootImpl$AsyncInputStage.apply(ViewRootImpl.java:3844)
    at android.view.ViewRootImpl$InputStage.deliver(ViewRootImpl.java:3642)
    at android.view.ViewRootImpl$InputStage.onDeliverToNext(ViewRootImpl.java:3695)
    at android.view.ViewRootImpl$InputStage.forward(ViewRootImpl.java:3661)
    at android.view.ViewRootImpl$InputStage.apply(ViewRootImpl.java:3669)
    at android.view.ViewRootImpl$InputStage.deliver(ViewRootImpl.java:3642)
    at android.view.ViewRootImpl$InputStage.onDeliverToNext(ViewRootImpl.java:3695)
    at android.view.ViewRootImpl$InputStage.forward(ViewRootImpl.java:3661)
    at android.view.ViewRootImpl$AsyncInputStage.forward(ViewRootImpl.java:3820)
    at android.view.ViewRootImpl$ImeInputStage.onFinishedInputEvent(ViewRootImpl.java:3981)
    at android.view.inputmethod.InputMethodManager$PendingEvent.run(InputMethodManager.java:2253)
    at android.view.inputmethod.InputMethodManager.invokeFinishedInputEventCallback(InputMethodManager.java:1874)
    at android.view.inputmethod.InputMethodManager.finishedInputEvent(InputMethodManager.java:1865)
    at android.view.inputmethod.InputMethodManager$ImeInputEventSender.onInputEventFinished(InputMethodManager.java:2230)
    at android.view.InputEventSender.dispatchInputEventFinished(InputEventSender.java:141)
    at android.os.MessageQueue.nativePollOnce(MessageQueue.java)
    at android.os.MessageQueue.next(MessageQueue.java:323)
    at android.os.Looper.loop(Looper.java:135)
    at android.app.ActivityThread.main(ActivityThread.java:5421)
    at java.lang.reflect.Method.invoke(Method.java)
    at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:726)
    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:616)

as...@google.com <as...@google.com> #10Mar 14, 2024 07:57PM

Marked as fixed.

This bug is still present in 1.2.0-alpha06.