SwipeDismissableNavHost throws IllegalArgumentException: The WearNavigator backstack is empty [277700155]

Fixed

Bug

Status Update

No update yet.

Description

ys...@google.com

created issue #1

Apr 11, 2023 08:09AM

Component used: Wear Compose
Version used: 1.2.0-alpha07
Devices/Android versions reproduced on: Pixel Watch

Not 100% reproducible, and I haven't nailed down exactly what is going on. Feel free to close, but raising here as we saw it once previously in compose-wear slack.

Happens with

https://github.com/joreilly/confetti, this app syncs the mobile theme and so can end up recomposing the main app composable here

https://github.com/joreilly/Confetti/blob/69191c4279958ee734546a0c83030d97ac6a99a9/wearApp/src/main/java/dev/johnoreilly/confetti/wear/ui/ConfettiApp.kt#L53

```
Fatal Exception: java.lang.IllegalArgumentException: The WearNavigator backstack is empty, there is no navigation destination to display.
at androidx.wear.compose.navigation.SwipeDismissableNavHostKt.SwipeDismissableNavHost(SwipeDismissableNavHost.kt:562)
at androidx.wear.compose.navigation.SwipeDismissableNavHostKt.SwipeDismissableNavHost(SwipeDismissableNavHost.kt:154)
at com.google.android.horologist.compose.navscaffold.WearNavScaffoldKt$WearNavScaffold$4.invoke(WearNavScaffold.kt:210)
at com.google.android.horologist.compose.navscaffold.WearNavScaffoldKt$WearNavScaffold$4.invoke(WearNavScaffold.kt:8)
at androidx.compose.runtime.internal.ComposableLambdaImpl.invoke(ComposableLambdaImpl.java:49)
at androidx.compose.runtime.internal.ComposableLambdaImpl.invoke(ComposableLambdaImpl.java:8)
at androidx.wear.compose.material.ScaffoldKt.Scaffold(Scaffold.kt:381)
at com.google.android.horologist.compose.navscaffold.WearNavScaffoldKt.WearNavScaffold(WearNavScaffold.kt:245)
at dev.johnoreilly.confetti.wear.ui.ConfettiAppKt$ConfettiApp$1.invoke(ConfettiApp.kt:57)
at dev.johnoreilly.confetti.wear.ui.ConfettiAppKt$ConfettiApp$1.invoke(ConfettiApp.kt:8)
at androidx.compose.runtime.internal.ComposableLambdaImpl.invoke(ComposableLambdaImpl.java:49)
at androidx.compose.runtime.internal.ComposableLambdaImpl.invoke(ComposableLambdaImpl.java:8)
at dev.johnoreilly.confetti.wear.ui.ThemeKt$ConfettiTheme$2.invoke(Theme.kt:43)
at dev.johnoreilly.confetti.wear.ui.ThemeKt$ConfettiTheme$2.invoke(Theme.kt:8)
at androidx.compose.runtime.internal.ComposableLambdaImpl.invoke(ComposableLambdaImpl.java:49)
at androidx.compose.runtime.internal.ComposableLambdaImpl.invoke(ComposableLambdaImpl.java:8)
at androidx.compose.runtime.CompositionLocalKt.CompositionLocalProvider(CompositionLocal.kt:40)
at androidx.wear.compose.material.TextKt.ProvideTextStyle(Text.kt:104)
at androidx.wear.compose.material.MaterialThemeKt$MaterialTheme$1.invoke(MaterialTheme.kt:45)
at androidx.wear.compose.material.MaterialThemeKt$MaterialTheme$1.invoke(MaterialTheme.kt:8)
at androidx.compose.runtime.internal.ComposableLambdaImpl.invoke(ComposableLambdaImpl.java:49)
at androidx.compose.runtime.internal.ComposableLambdaImpl.invoke(ComposableLambdaImpl.java:8)
at androidx.compose.runtime.CompositionLocalKt.CompositionLocalProvider(CompositionLocal.kt:40)
at androidx.wear.compose.material.MaterialThemeKt.MaterialTheme(MaterialTheme.kt:424)
at dev.johnoreilly.confetti.wear.ui.ThemeKt.ConfettiTheme(Theme.kt:112)
at dev.johnoreilly.confetti.wear.ui.ThemeKt.ConfettiTheme(Theme.kt:173)
at dev.johnoreilly.confetti.wear.ui.ConfettiAppKt.ConfettiApp(ConfettiApp.kt:130)
at dev.johnoreilly.confetti.wear.MainActivity$onCreate$1$1.invoke(MainActivity.java:45)
at dev.johnoreilly.confetti.wear.MainActivity$onCreate$1$1.invoke(MainActivity.java:8)
at androidx.compose.runtime.internal.ComposableLambdaImpl.invoke(ComposableLambdaImpl.java:49)
at androidx.compose.runtime.internal.ComposableLambdaImpl.invoke(ComposableLambdaImpl.java:8)
at androidx.compose.runtime.CompositionLocalKt.CompositionLocalProvider(CompositionLocal.kt:40)
at dev.johnoreilly.confetti.wear.MainActivity$onCreate$1.invoke(MainActivity.java:106)
at dev.johnoreilly.confetti.wear.MainActivity$onCreate$1.invoke(MainActivity.java:8)
at androidx.compose.runtime.internal.ComposableLambdaImpl.invoke(ComposableLambdaImpl.java:49)
at androidx.compose.runtime.internal.ComposableLambdaImpl.invoke(ComposableLambdaImpl.java:8)
at androidx.compose.ui.platform.ComposeView.Content(ComposeView.java:35)
at androidx.compose.ui.platform.AbstractComposeView$ensureCompositionCreated$1.invoke(AbstractComposeView.java:35)
at androidx.compose.ui.platform.AbstractComposeView$ensureCompositionCreated$1.invoke(AbstractComposeView.java:8)
at androidx.compose.runtime.internal.ComposableLambdaImpl.invoke(ComposableLambdaImpl.java:49)
at androidx.compose.runtime.internal.ComposableLambdaImpl.invoke(ComposableLambdaImpl.java:8)
at androidx.compose.runtime.CompositionLocalKt.CompositionLocalProvider(CompositionLocal.kt:40)
at androidx.compose.ui.platform.CompositionLocalsKt.ProvideCommonCompositionLocals(CompositionLocals.kt:347)
at androidx.compose.ui.platform.AndroidCompositionLocals_androidKt$ProvideAndroidCompositionLocals$3.invoke(AndroidCompositionLocals_android.kt:45)
at androidx.compose.ui.platform.AndroidCompositionLocals_androidKt$ProvideAndroidCompositionLocals$3.invoke(AndroidCompositionLocals_android.kt:8)
at androidx.compose.runtime.internal.ComposableLambdaImpl.invoke(ComposableLambdaImpl.java:49)
at androidx.compose.runtime.internal.ComposableLambdaImpl.invoke(ComposableLambdaImpl.java:8)
at androidx.compose.runtime.CompositionLocalKt.CompositionLocalProvider(CompositionLocal.kt:40)
at androidx.compose.ui.platform.AndroidCompositionLocals_androidKt.ProvideAndroidCompositionLocals(AndroidCompositionLocals_android.kt:302)
at androidx.compose.ui.platform.WrappedComposition$setContent$1$1$2.invoke(WrappedComposition.java:41)
at androidx.compose.ui.platform.WrappedComposition$setContent$1$1$2.invoke(WrappedComposition.java:8)
at androidx.compose.runtime.internal.ComposableLambdaImpl.invoke(ComposableLambdaImpl.java:49)
at androidx.compose.runtime.internal.ComposableLambdaImpl.invoke(ComposableLambdaImpl.java:8)
at androidx.compose.runtime.CompositionLocalKt.CompositionLocalProvider(CompositionLocal.kt:40)
at androidx.compose.ui.platform.WrappedComposition$setContent$1$1.invoke(WrappedComposition.java:155)
at androidx.compose.ui.platform.WrappedComposition$setContent$1$1.invoke(WrappedComposition.java:8)
at androidx.compose.runtime.internal.ComposableLambdaImpl.invoke(ComposableLambdaImpl.java:49)
at androidx.compose.runtime.internal.ComposableLambdaImpl.invoke(ComposableLambdaImpl.java:8)
at androidx.compose.runtime.ActualJvm_jvmKt.invokeComposable(ActualJvm_jvm.kt:22)
at androidx.compose.runtime.ComposerImpl$doCompose$2$5.invoke(ComposerImpl.java:19)
at androidx.compose.runtime.ComposerImpl$doCompose$2$5.invoke(ComposerImpl.java)
at androidx.compose.runtime.SnapshotStateKt__DerivedStateKt.observeDerivedStateRecalculations(SnapshotStateKt__DerivedState.kt:45)
at androidx.compose.runtime.SnapshotStateKt.observeDerivedStateRecalculations(SnapshotState.kt)
at androidx.compose.runtime.ComposerImpl.doCompose(ComposerImpl.java:138)
at androidx.compose.runtime.ComposerImpl.composeContent$runtime_release(ComposerImpl.java:18)
at androidx.compose.runtime.CompositionImpl.composeContent(CompositionImpl.java:17)
at androidx.compose.runtime.Recomposer.composeInitial$runtime_release(Recomposer.java:34)
at androidx.compose.runtime.CompositionImpl.setContent(CompositionImpl.java:15)
at androidx.compose.ui.platform.WrappedComposition$setContent$1.invoke(WrappedComposition.java:82)
at androidx.compose.ui.platform.WrappedComposition$setContent$1.invoke(WrappedComposition.java:2)
at androidx.compose.ui.platform.AndroidComposeView.setOnViewTreeOwnersAvailable(AndroidComposeView.java:11)
at androidx.compose.ui.platform.WrappedComposition.setContent(WrappedComposition.java:12)
at androidx.compose.ui.platform.WrappedComposition.onStateChanged(WrappedComposition.java:28)
at androidx.lifecycle.LifecycleRegistry$ObserverWithState.dispatchEvent(LifecycleRegistry.java:24)
at androidx.lifecycle.LifecycleRegistry.addObserver(LifecycleRegistry.java:105)
at androidx.compose.ui.platform.WrappedComposition$setContent$1.invoke(WrappedComposition.java:43)
at androidx.compose.ui.platform.WrappedComposition$setContent$1.invoke(WrappedComposition.java:2)
at androidx.compose.ui.platform.AndroidComposeView.onAttachedToWindow(AndroidComposeView.java:115)
at android.view.View.dispatchAttachedToWindow(View.java:20516)
at android.view.ViewGroup.dispatchAttachedToWindow(ViewGroup.java:3493)
at android.view.ViewGroup.dispatchAttachedToWindow(ViewGroup.java:3500)
at android.view.ViewGroup.dispatchAttachedToWindow(ViewGroup.java:3500)
at android.view.ViewGroup.dispatchAttachedToWindow(ViewGroup.java:3500)
at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:2462)
at android.view.ViewRootImpl.doTraversal(ViewRootImpl.java:1997)
at android.view.ViewRootImpl$TraversalRunnable.run(ViewRootImpl.java:8250)
at android.view.Choreographer$CallbackRecord.run(Choreographer.java:972)
at android.view.Choreographer.doCallbacks(Choreographer.java:796)
at android.view.Choreographer.doFrame(Choreographer.java:731)
at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:957)
at android.os.Handler.handleCallback(Handler.java:938)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:223)
at android.app.ActivityThread.main(ActivityThread.java:7660)
at java.lang.reflect.Method.invoke(Method.java)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:592)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:947)
```

Comments

ys...@google.com <ys...@google.com> #2Apr 11, 2023 08:10AM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit b90079595f33f58fece04026a97faa0d243acdb1
Author: Yuichi Araki <yaraki@google.com>
Date: Wed Sep 18 16:55:49 2019

Change the way to detect mismatch between POJO and query

This fixes cursor mismatch warnings with expandProjection.

Bug: 140759491
Test: QueryMethodProcessorTest
Change-Id: I7659002e5e0d1ef60fc1af2a625c0c36da0664d8

M room/compiler/src/main/kotlin/androidx/room/processor/QueryMethodProcessor.kt
M room/compiler/src/main/kotlin/androidx/room/solver/TypeAdapterStore.kt
M room/compiler/src/main/kotlin/androidx/room/solver/query/result/PojoRowAdapter.kt
M room/compiler/src/test/kotlin/androidx/room/processor/QueryMethodProcessorTest.kt
M room/compiler/src/test/kotlin/androidx/room/testing/TestProcessor.kt

https://android-review.googlesource.com/1123258

https://goto.google.com/android-sha1/b90079595f33f58fece04026a97faa0d243acdb1

ys...@google.com <ys...@google.com> #3Apr 11, 2023 08:11AM

The following changes were cherrypicked through Coastguard (

https://android-build.googleplex.com/coastguard/)
Release Track: (

https://android-build.googleplex.com/coastguard/dashboard/6093116828680192/#/cherrypickHistory/all)
Created: 2019-10-14 17:23:42.610596+00:00
Changes: aosp/1123258

ys...@google.com <ys...@google.com> #4Apr 11, 2023 08:13AM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit bdde5a1a970ddc9007b28de4aa29d60ffa588f08
Author: Yigit Boyar <yboyar@google.com>
Date: Thu Apr 16 16:47:05 2020

Re-factor how errors are dismissed when query is re-written

This CL changes how we handle errors/warnings if query is
re-written.
There was a bug in expandProjection where we would report warnings
for things that Room already fixes automatically (

b/140759491

).
The solution to that problem (I7659002e5e0d1ef60fc1af2a625c0c36da0664d8)
solved it by deferring validating of columns until after re-write
decision is made. Unfortunately, this required changing PojoRowAdapter
to have a dummy mapping until it is validating, make it hard to use
as it does have a non-null mapping which is not useful.

This CL partially reverts that change and instead rely on the log
deferring logic we have in Context. This way, we don't need to break
the stability of PojoRowAdapter while still having the ability to
drop warnings that room fixes. This will also play nicer when we
have different query re-writing options that can use more information
about the query results.

Bug: 153387066
Bug: 140759491
Test: existing tests pass
Change-Id: I2ec967c763d33d7a3ff02c1a13c6953b460d1e5f

M room/compiler/src/main/kotlin/androidx/room/log/RLog.kt
M room/compiler/src/main/kotlin/androidx/room/processor/QueryMethodProcessor.kt
M room/compiler/src/main/kotlin/androidx/room/solver/TypeAdapterStore.kt
M room/compiler/src/main/kotlin/androidx/room/solver/query/result/PojoRowAdapter.kt

https://android-review.googlesource.com/1288456

st...@google.com <st...@google.com> #5Apr 17, 2023 09:24AM

Assigned to ha...@google.com.

Hardik - please take a look to see if you can reproduce this reliably.

ys...@google.com <ys...@google.com> #6Apr 17, 2023 09:55AM

Possibly lower priority. I think I've fixed by avoiding navigating shortly after startup. Instead I found a way to use TaskStackBuilder instead of intent extras.

So I think this code inside my activity setContent was triggering it.

            LaunchedEffect(Unit) {
                navigateFromTileLaunch()
            }

calling this

fun navigateFromTileLaunch() {
    val tileButton = intent.getAndRemoveKey("tile")
    if (tileButton == "session") {
        val conference = intent.getAndRemoveKey("conference")
        val sessionId = intent.getAndRemoveKey("session")

        if (conference != null && sessionId != null) {
            navController.navigate(
                SessionDetailsDestination.createNavigationRoute(
                    SessionDetailsKey(conference, sessionId)
                )
            )
        }
    } else if (tileButton == "login") {
        navController.navigate(SignInDestination.route)
    } else if (tileButton == "conferences") {
        navController.navigate(ConferencesDestination.route)
    }
}

ha...@google.com <ha...@google.com> Apr 18, 2023 07:01AM

Accepted by ha...@google.com.

ys...@google.com <ys...@google.com> #7Apr 20, 2023 07:51AM

I'm still getting this with deeplinks from Complications. appIntent works, but sessionIntent crashes about 50% of the time.

    private fun sessionIntent(conference: String, sessionDetails: SessionDetails): PendingIntent? {
        val sessionDetailIntent = Intent(
            Intent.ACTION_VIEW,
            "confetti://confetti/session/${conference}/${sessionDetails.id}".toUri()
        )
        return getActivity(
            this,
            0,
            sessionDetailIntent,
            FLAG_IMMUTABLE or FLAG_UPDATE_CURRENT
        )
    }
    private fun appIntent(): PendingIntent? {
        val appIntent = Intent(
            this,
            MainActivity::class.java
        )
        return getActivity(
            this,
            0,
            appIntent,
            FLAG_IMMUTABLE or FLAG_UPDATE_CURRENT
        )
    }

st...@google.com <st...@google.com> #8Apr 24, 2023 08:49AM

ys...@google.com <ys...@google.com> #9Apr 24, 2023 05:18PM

The current code has that deeplink disabled because it crashes.

Effective repro steps

Full Steps:

install app on Mobile, select a conference, bookmark some sessions
Install app on Wear, login, confirm bookmarks are synced automatically.
Add a complication (if you undo my edit) then it should show the session details in the complication.

Short Steps.

Change the https://github.com/joreilly/Confetti/blob/main/wearApp/src/main/java/dev/johnoreilly/confetti/wear/complication/NextSessionComplicationService.kt to use a deeplink for the result returned in all cases (remove the if conditions)
Add a complication

If you want I can make a test PR tomorrow

ys...@google.com <ys...@google.com> #10Apr 25, 2023 11:06AM

PR with a simple repro

https://github.com/joreilly/Confetti/pull/656/files

ha...@google.com <ha...@google.com> #11Apr 25, 2023 12:56PM

Thanks Yuri, Able to reproduce the issue for this application. Looking into the root cause now.

ha...@google.com <ha...@google.com> #12Apr 28, 2023 07:06AM

The issue seems to be occuring while using deep links for navigation. The navigation graph doesn't seem to be built completely hence throwing this issue.

Though the issue is not occuring 100% of the times, pasting easier steps to reproduce this issue in androidx:

Code:

@Composable
fun Demo()
{
    val navController = rememberSwipeDismissableNavController()
    SwipeDismissableNavHost(navController =  navController, startDestination = "hello"){
        composable(route  = "hello"){
            Test()
        }
        composable(route  = "hello2", deepLinks = listOf(
            navDeepLink {uriPattern = "hk1://hk1"}
        )){
            Test2()
        }
    }
}

@Composable
fun Test2(){
    Column(modifier = Modifier.fillMaxSize(), verticalArrangement =  Arrangement.Center, horizontalAlignment = Alignment.CenterHorizontally) {
        Text("Hello Test 2")
    }
}

@Composable
fun Test(){
    Column(modifier = Modifier.fillMaxSize(), verticalArrangement =  Arrangement.Center, horizontalAlignment = Alignment.CenterHorizontally) {
        Text("Hello world")
    }
}

Add the following intent filter to AndroidManifest:

 <intent-filter>
                <action android:name="android.intent.action.VIEW" />
                <category android:name="android.intent.category.DEFAULT" />
                <category android:name="android.intent.category.BROWSABLE" />

                <data
                    android:host="hk1"
                    android:scheme="hk1" />
            </intent-filter>

Next steps:

Investigate on initialization flow of navigation graph.
Link Intent deep link handling with navigation flow.

ha...@google.com <ha...@google.com> #13Apr 28, 2023 08:38AM

Created internal tracking bug: b/280026691.

ha...@google.com <ha...@google.com> #14May 2, 2023 05:38PM

The root cause of this issue is inconsistent backstack state when handling the pending intent.

The activity was being called twice once with backstack size 0 and next time with the expected size.

We need to add the following flags FLAG_ACTIVITY_NEW_TASK, FLAG_ACTIVITY_CLEAR_TASK to the session intent to ensure a consistent activity back stack.

See related bug: b/148567892

@yschimke Can you please confirm if this works for you?

ys...@google.com <ys...@google.com> #15May 2, 2023 06:33PM

So far, working perfectly.

ha...@google.com <ha...@google.com> May 3, 2023 07:10AM

Status: Won't Fix (Intended Behavior)

ys...@google.com <ys...@google.com> #16May 3, 2023 07:25AM

I'd suggest fixing this anyway, does it fail like this on mobile? Or just silently ignore the empty stack?

The launch may not be getting the desired result, but it shouldn't cause the app to crash like this.

ha...@google.com <ha...@google.com> #17May 3, 2023 07:32AM

On mobile, it silently ignores the empty backstack. code link

ys...@google.com <ys...@google.com> #18May 3, 2023 07:38AM

Can we reopen then?

ha...@google.com <ha...@google.com> May 3, 2023 07:50AM

Status: Assigned (reopened)

st...@google.com <st...@google.com> #19May 3, 2023 11:05AM

The reason for throwing the exception in the past was that the backstack could be empty due to using the wrong function to build the navigation graph - so there was a specific request to throw an error explaining why nothing was displayed on the screen.

This is something that the developer needs to address, so I think better to throw an error that describes the problem (the backstack is empty, which should not happen) and add more background in the comments as per b/148567892: "If the activity is using FLAG_ACTIVITY_NEW_TASK then it also needs to set FLAG_ACTIVITY_CLEAR_TASK, otherwise the activity will be created twice, the first of these with an empty backstack"

ys...@google.com <ys...@google.com> #20May 3, 2023 12:28PM

It may not be optimal, but it's the default and not an error on mobile.

I don't think anything means this only applies to launching your own app, but I could be missing something. It seems like a DOS if I can launch an intent with deeplinks and nuke my competitors apps. To we have other examples of where you can cause crashes like this?

st...@google.com <st...@google.com> #21May 3, 2023 07:26PM

Thanks for the extra insight Yuri - I agree that this can be seen as a vulnerability, so let's remove the exception and log a message instead, then continue (which will in some cases leave a blank screen, but the log message and extra comments in the source code should help developers in that case).

ha...@google.com <ha...@google.com> May 4, 2023 06:11AM

Accepted by ha...@google.com.

ha...@google.com <ha...@google.com> #22May 4, 2023 12:19PM

Marked as fixed.

Raised cl: aosp/2570171 for the fix.

Sample warning log: https://screenshot.googleplex.com/antPyWcV8yfBMvc

ap...@google.com <ap...@google.com> #23May 4, 2023 01:20PM

Project: platform/frameworks/support
Branch: androidx-main

commit 6d370cb96df84e695e3aa88db7cf4ad35a494bf4
Author: hardik <hardikkwl@google.com>
Date: Thu May 04 17:23:30 2023

Log warning inplace of throwing exception for empty backstack in SwipeDismissableNavHost

Relnote: SwipeDismissableNavHost now logs a warning with potential causes of empty backstack. This is done to prevent unexpected crashes caused because of IllegalArgumentException which was thrown when the backstack was empty.

Bug: 277700155

Test: Tested manually by triggering deep link intents

Change-Id: I04a812325b4a8b5d59d4ce1ba2f770ee75b0ba71

M wear/compose/compose-navigation/src/main/java/androidx/wear/compose/navigation/SwipeDismissableNavHost.kt

https://android-review.googlesource.com/2570171

ju...@google.com <ju...@google.com> #24May 24, 2023 07:37PM

The following release(s) address this bug.It is possible this bug has only been partially addressed:

androidx.wear.compose:compose-navigation:1.2.0-beta01

Issue 277700155

Description

Issue summary

Comments

ys...@google.com <ys...@google.com> #2Apr 11, 2023 08:10AM

ys...@google.com <ys...@google.com> #3Apr 11, 2023 08:11AM

ys...@google.com <ys...@google.com> #4Apr 11, 2023 08:13AM

st...@google.com <st...@google.com> #5Apr 17, 2023 09:24AM

ys...@google.com <ys...@google.com> #6Apr 17, 2023 09:55AM

ha...@google.com <ha...@google.com> Apr 18, 2023 07:01AM

ys...@google.com <ys...@google.com> #7Apr 20, 2023 07:51AM

st...@google.com <st...@google.com> #8Apr 24, 2023 08:49AM

ys...@google.com <ys...@google.com> #9Apr 24, 2023 05:18PM

ys...@google.com <ys...@google.com> #10Apr 25, 2023 11:06AM

ha...@google.com <ha...@google.com> #11Apr 25, 2023 12:56PM

ha...@google.com <ha...@google.com> #12Apr 28, 2023 07:06AM

Next steps:

ha...@google.com <ha...@google.com> #13Apr 28, 2023 08:38AM

ha...@google.com <ha...@google.com> #14May 2, 2023 05:38PM

ys...@google.com <ys...@google.com> #15May 2, 2023 06:33PM

ha...@google.com <ha...@google.com> May 3, 2023 07:10AM

ys...@google.com <ys...@google.com> #16May 3, 2023 07:25AM

ha...@google.com <ha...@google.com> #17May 3, 2023 07:32AM

ys...@google.com <ys...@google.com> #18May 3, 2023 07:38AM

ha...@google.com <ha...@google.com> May 3, 2023 07:50AM

st...@google.com <st...@google.com> #19May 3, 2023 11:05AM

ys...@google.com <ys...@google.com> #20May 3, 2023 12:28PM

st...@google.com <st...@google.com> #21May 3, 2023 07:26PM

ha...@google.com <ha...@google.com> May 4, 2023 06:11AM

ha...@google.com <ha...@google.com> #22May 4, 2023 12:19PM

ap...@google.com <ap...@google.com> #23May 4, 2023 01:20PM

ju...@google.com <ju...@google.com> #24May 24, 2023 07:37PM

Add comment

Issue metadata