New `TooltipBox` throws `RuntimeException: Parcel: unable to marshal value` [299500338]

Fixed

Bug

Status Update

No update yet.

Description

sp...@gmail.com

created issue #1

Sep 7, 2023 07:22PM

Jetpack Compose version: 1.6.0-alpha05
Jetpack Compose component used: TooltipBox
Android Studio Build: Android Studio Iguana | 2023.2.1 Canary 2 (AGP 8.1.1)
Kotlin version: 1.9.10

After updating to compose 1.6.0-alpha05, my app was crashing after pressing the home button to leave the app. After some testing, I think the app crashes when TooltipBox is used.

FATAL EXCEPTION: main
    Process: com.example.bugapplication, PID: 2242
    java.lang.IllegalArgumentException: Parcel: unknown type for value androidx.compose.foundation.MutatorMutex@8cf1dbd
        at android.os.Parcel.getValueType(Parcel.java:2409)
        at android.os.Parcel.writeValue(Parcel.java:2308)
        at android.os.Parcel.writeList(Parcel.java:1382)
        at android.os.Parcel.writeValue(Parcel.java:2460)
        at android.os.Parcel.writeValue(Parcel.java:2316)
        at android.os.Parcel.writeList(Parcel.java:1382)
        at android.os.Parcel.writeValue(Parcel.java:2460)
        at android.os.Parcel.writeValue(Parcel.java:2316)
        at android.os.Parcel.writeArrayMapInternal(Parcel.java:1265)
        at android.os.BaseBundle.writeToParcelInner(BaseBundle.java:1802)
        at android.os.Bundle.writeToParcel(Bundle.java:1389)
        at android.os.Parcel.writeBundle(Parcel.java:1334)
        at android.os.Parcel.writeValue(Parcel.java:2433)
        at android.os.Parcel.writeValue(Parcel.java:2323)
        at android.os.Parcel.writeArrayMapInternal(Parcel.java:1265)
        at android.os.BaseBundle.writeToParcelInner(BaseBundle.java:1802)
        at android.os.Bundle.writeToParcel(Bundle.java:1389)
        at android.os.Parcel.writeBundle(Parcel.java:1334)
        at android.os.Parcel.writeValue(Parcel.java:2433)
        at android.os.Parcel.writeValue(Parcel.java:2323)
        at android.os.BaseBundle.dumpStats(BaseBundle.java:1874)
        at android.os.BaseBundle.dumpStats(BaseBundle.java:1911)
        at android.app.servertransaction.PendingTransactionActions$StopInfo.run(PendingTransactionActions.java:150)
        at android.os.Handler.handleCallback(Handler.java:942)
        at android.os.Handler.dispatchMessage(Handler.java:99)
        at android.os.Looper.loopOnce(Looper.java:201)
        at android.os.Looper.loop(Looper.java:288)
        at android.app.ActivityThread.main(ActivityThread.java:7872)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:548)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:936)

I've attached a small app that I've been using to test the bug on different devices, but any app that uses TooltipBox should have the issue.

Steps to reproduce:

Open demo app
Use the Switch to display the Icon + Tooltip composable
Click the Home button

The app should crash with a RuntimeException. If we hide the Icon + Tooltip before clicking Home, the app won't crash.

I've tested the app on a device running SDK 30 and on three emulator images running 26, 29 and 33.

I hope this is not a problem with my TooltipBox implementation. I've used the same composable from TooltipSamples to avoid mistakes.

BugApplication.zip

108 KB

Download

Comments

ma...@google.com <ma...@google.com> Sep 8, 2023 01:24PM

Reassigned to ke...@google.com.

ad...@gmail.com <ad...@gmail.com> #2Sep 9, 2023 02:18PM

There is a report about it in kotlinlang Slack: https://kotlinlang.slack.com/archives/C01D6HTPATV/p1706005056105739

zi...@gmail.com <zi...@gmail.com> #3Sep 10, 2023 04:02PM

Thanks for the report!

ki...@protonmail.com <ki...@protonmail.com> #4Sep 11, 2023 10:16AM

The release notes documentation has been edited to clarify this change in behavior for line height.

To support non-standard text sizes, we encourage users to follow the Material design system and use a different type scale rather than changing the font size directly. Alternatively, users can overwrite the line height like so: style = LocalTextStyle.current.copy(lineHeight = TextUnit.Unspecified), or create a custom Typography entirely.

Message last modified on Sep 11, 2023 11:34AM

ap...@google.com <ap...@google.com> #5Sep 11, 2023 08:22PM

deleted

ke...@google.com <ke...@google.com> Sep 11, 2023 08:53PM

Marked as fixed.

na...@google.com <na...@google.com> #6Sep 20, 2023 07:16PM

In my case, I have multiple font sizes in the same Text (using SpanStyle in AnnotatedString). There are legitimate reasons for this. For example, when combining Chinese and English (phonetic) together (for language-learning purposes).