`IndexOutOfBoundsException` when removing account

[rfc2822]

I created three accounts for testing. Then I deleted one of them and DAVx5 crashed.

Sync pending(Account {name=play@bitfire.at, type=bitfire.at.davdroid}, com.android.calendar) = false
Sync pending(Account {name=play@bitfire.at, type=bitfire.at.davdroid}, at.techbee.jtx.provider) = false
Sync pending(Account {name=play@bitfire.at, type=bitfire.at.davdroid}, at.techbee.jtx.provider) = false
Sync pending(Account {name=card (play@bitfire.at) #350, type=at.bitfire.davdroid.address_book}, com.android.contacts) = false
Cleared Reference was only reachable from finalizer (only reported once)
Background concurrent mark compact GC freed 10MB AllocSpace bytes, 20(680KB) LOS objects, 49% free, 12MB/24MB, paused 1.032ms,6.624ms total 70.300ms
Sync pending(Account {name=play@bitfire.at, type=bitfire.at.davdroid}, com.android.calendar) = false
Sync pending(Account {name=play@bitfire.at, type=bitfire.at.davdroid}, at.techbee.jtx.provider) = false
Sync pending(Account {name=card (play@bitfire.at) #350, type=at.bitfire.davdroid.address_book}, com.android.contacts) = false
Sync pending(Account {name=play@bitfire.at, type=bitfire.at.davdroid}, at.techbee.jtx.provider) = false
Sync pending(Account {name=card (play@bitfire.at) #350, type=at.bitfire.davdroid.address_book}, com.android.contacts) = false
Sync pending(Account {name=play@bitfire.at, type=bitfire.at.davdroid}, com.android.calendar) = false
Sync pending(Account {name=play@bitfire.at, type=bitfire.at.davdroid}, at.techbee.jtx.provider) = false
Sync pending(Account {name=card (play@bitfire.at) #350, type=at.bitfire.davdroid.address_book}, com.android.contacts) = false
Sync pending(Account {name=play@bitfire.at, type=bitfire.at.davdroid}, com.android.calendar) = false
Shutting down VM
Sync pending(Account {name=play@bitfire.at, type=bitfire.at.davdroid}, com.android.calendar) = false
FATAL EXCEPTION: main
Process: at.bitfire.davdroid, PID: 3137
java.lang.IndexOutOfBoundsException: Index 1 out of bounds for length 1
	at jdk.internal.util.Preconditions.outOfBounds(Preconditions.java:64)
	at jdk.internal.util.Preconditions.outOfBoundsCheckIndex(Preconditions.java:70)
	at jdk.internal.util.Preconditions.checkIndex(Preconditions.java:266)
	at java.util.Objects.checkIndex(Objects.java:385)
	at java.util.ArrayList.get(ArrayList.java:434)
	at androidx.compose.material3.TabIndicatorOffsetNode.measure-3p2s80s(TabRow.kt:704)
	at androidx.compose.ui.node.LayoutModifierNodeCoordinator$LookaheadDelegateForLayoutModifierNode.measure-BRTryo0(LayoutModifierNodeCoordinator.kt:93)
	at androidx.compose.ui.node.LookaheadPassDelegate$performMeasureBlock$1.invoke(LookaheadPassDelegate.kt:442)
	at androidx.compose.ui.node.LookaheadPassDelegate$performMeasureBlock$1.invoke(LookaheadPassDelegate.kt:441)
	at androidx.compose.runtime.snapshots.SnapshotStateObserver.observeReads(SnapshotStateObserver.kt:699)
	at androidx.compose.ui.node.LookaheadPassDelegate.performMeasure-BRTryo0$ui(LookaheadPassDelegate.kt:951)
	at androidx.compose.ui.node.LayoutNodeLayoutDelegate.performLookaheadMeasure-BRTryo0$ui(LayoutNodeLayoutDelegate.kt:333)
	at androidx.compose.ui.node.LookaheadPassDelegate.remeasure-BRTryo0(LookaheadPassDelegate.kt:492)
	at androidx.compose.ui.node.LookaheadPassDelegate.measure-BRTryo0(LookaheadPassDelegate.kt:402)
	at androidx.compose.material3.TabRowKt$TabRowImpl$1$2$1.measure-3p2s80s(TabRow.kt:484)
	at androidx.compose.ui.layout.MultiContentMeasurePolicyImpl.measure-3p2s80s(MultiContentMeasurePolicy.kt:206)
	at androidx.compose.ui.node.InnerNodeCoordinator$LookaheadDelegateImpl.measure-BRTryo0(InnerNodeCoordinator.kt:82)
	at androidx.compose.foundation.layout.FillNode.measure-3p2s80s(Size.kt:721)
	at androidx.compose.ui.node.LayoutModifierNodeCoordinator$LookaheadDelegateForLayoutModifierNode.measure-BRTryo0(LayoutModifierNodeCoordinator.kt:93)
	at androidx.compose.ui.node.LookaheadPassDelegate$performMeasureBlock$1.invoke(LookaheadPassDelegate.kt:442)
	at androidx.compose.ui.node.LookaheadPassDelegate$performMeasureBlock$1.invoke(LookaheadPassDelegate.kt:441)
	at androidx.compose.runtime.snapshots.SnapshotStateObserver.observeReads(SnapshotStateObserver.kt:728)
	at androidx.compose.ui.node.LookaheadPassDelegate.performMeasure-BRTryo0$ui(LookaheadPassDelegate.kt:951)
	at androidx.compose.ui.node.LayoutNodeLayoutDelegate.performLookaheadMeasure-BRTryo0$ui(LayoutNodeLayoutDelegate.kt:333)
	at androidx.compose.ui.node.LookaheadPassDelegate.remeasure-BRTryo0(LookaheadPassDelegate.kt:492)
	at androidx.compose.ui.node.LayoutNode.lookaheadRemeasure-_Sx5XlM$ui(LayoutNode.kt:1267)
	at androidx.compose.ui.node.LayoutNode.lookaheadRemeasure-_Sx5XlM$ui$default(LayoutNode.kt:1261)
	at androidx.compose.ui.node.MeasureAndLayoutDelegate.doLookaheadRemeasure-sdFAvZA(MeasureAndLayoutDelegate.kt:359)
	at androidx.compose.ui.node.MeasureAndLayoutDelegate.remeasureAndRelayoutIfNeeded(MeasureAndLayoutDelegate.kt:583)
	at androidx.compose.ui.node.MeasureAndLayoutDelegate.onlyRemeasureIfPending(MeasureAndLayoutDelegate.kt:683)
	at androidx.compose.ui.node.MeasureAndLayoutDelegate.forceMeasureTheSubtreeInternal(MeasureAndLayoutDelegate.kt:710)
	at androidx.compose.ui.node.MeasureAndLayoutDelegate.forceMeasureTheSubtreeInternal(MeasureAndLayoutDelegate.kt:717)
	at androidx.compose.ui.node.MeasureAndLayoutDelegate.forceMeasureTheSubtreeInternal(MeasureAndLayoutDelegate.kt:717)
	at androidx.compose.ui.node.MeasureAndLayoutDelegate.forceMeasureTheSubtreeInternal(MeasureAndLayoutDelegate.kt:717)
	at androidx.compose.ui.node.MeasureAndLayoutDelegate.forceMeasureTheSubtree(MeasureAndLayoutDelegate.kt:674)
	at androidx.compose.ui.platform.AndroidComposeView.forceMeasureTheSubtree(AndroidComposeView.android.kt:1767)
	at androidx.compose.ui.node.LookaheadPassDelegate.remeasure-BRTryo0(LookaheadPassDelegate.kt:503)
	at androidx.compose.ui.node.LayoutNode.lookaheadRemeasure-_Sx5XlM$ui(LayoutNode.kt:1267)
	at androidx.compose.ui.node.LayoutNode.lookaheadRemeasure-_Sx5XlM$ui$default(LayoutNode.kt:1261)
	at androidx.compose.ui.node.MeasureAndLayoutDelegate.doLookaheadRemeasure-sdFAvZA(MeasureAndLayoutDelegate.kt:359)
	at androidx.compose.ui.node.MeasureAndLayoutDelegate.remeasureOnly(MeasureAndLayoutDelegate.kt:650)
	at androidx.compose.ui.node.MeasureAndLayoutDelegate.remeasureLookaheadRootsInSubtree(MeasureAndLayoutDelegate.kt:457)
	at androidx.compose.ui.node.MeasureAndLayoutDelegate.remeasureLookaheadRootsInSubtree(MeasureAndLayoutDelegate.kt:460)
	at androidx.compose.ui.node.MeasureAndLayoutDelegate.remeasureLookaheadRootsInSubtree(MeasureAndLayoutDelegate.kt:460)
	at androidx.compose.ui.node.MeasureAndLayoutDelegate.remeasureLookaheadRootsInSubtree(MeasureAndLayoutDelegate.kt:460)
	at androidx.compose.ui.node.MeasureAndLayoutDelegate.remeasureLookaheadRootsInSubtree(MeasureAndLayoutDelegate.kt:460)
	at androidx.compose.ui.node.MeasureAndLayoutDelegate.remeasureLookaheadRootsInSubtree(MeasureAndLayoutDelegate.kt:460)
	at androidx.compose.ui.node.MeasureAndLayoutDelegate.measureOnly(MeasureAndLayoutDelegate.kt:443)
	at androidx.compose.ui.platform.AndroidComposeView.onMeasure(AndroidComposeView.android.kt:1847)
	at android.view.View.measure(View.java:28407)
	at androidx.compose.ui.platform.AbstractComposeView.internalOnMeasure$ui(ComposeView.android.kt:314)
	at androidx.compose.ui.platform.AbstractComposeView.onMeasure(ComposeView.android.kt:301)
	at android.view.View.measure(View.java:28407)
	at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:7028)
	at android.widget.FrameLayout.onMeasure(FrameLayout.java:194)
	at androidx.appcompat.widget.ContentFrameLayout.onMeasure(ContentFrameLayout.java:141)
	at android.view.View.measure(View.java:28407)
	at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:7028)
	at android.widget.LinearLayout.measureChildBeforeLayout(LinearLayout.java:1608)
	at android.widget.LinearLayout.measureVertical(LinearLayout.java:878)
	at android.widget.LinearLayout.onMeasure(LinearLayout.java:721)
	at android.view.View.measure(View.java:28407)
	at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:7028)
	at android.widget.FrameLayout.onMeasure(FrameLayout.java:194)
	at android.view.View.measure(View.java:28407)
	at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:7028)
	at android.widget.LinearLayout.measureChildBeforeLayout(LinearLayout.java:1608)
	at android.widget.LinearLayout.measureVertical(LinearLayout.java:878)
	at android.widget.LinearLayout.onMeasure(LinearLayout.java:721)
	at android.view.View.measure(View.java:28407)
	at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:7028)
	at android.widget.FrameLayout.onMeasure(FrameLayout.java:194)
	at com.android.internal.policy.DecorView.onMeasure(DecorView.java:736)
	at android.view.View.measure(View.java:28407)
	at android.view.ViewRootImpl.performMeasure(ViewRootImpl.java:4998)
	at android.view.ViewRootImpl.measureHierarchy(ViewRootImpl.java:3391)
	at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:3699)
	at android.view.ViewRootImpl.doTraversal(ViewRootImpl.java:3076)
	at android.view.ViewRootImpl$TraversalRunnable.run(ViewRootImpl.java:10643)
	at android.view.Choreographer$CallbackRecord.run(Choreographer.java:1570)
	at android.view.Choreographer$CallbackRecord.run(Choreographer.java:1579)
	at android.view.Choreographer.doCallbacks(Choreographer.java:1179)
	at android.view.Choreographer.doFrame(Choreographer.java:1108)
	at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:1553)
	at android.os.Handler.handleCallback(Handler.java:995)
	at android.os.Handler.dispatchMessage(Handler.java:103)
	at android.os.Looper.loopOnce(Looper.java:248)
	at android.os.Looper.loop(Looper.java:338)
	at android.app.ActivityThread.main(ActivityThread.java:9067)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:593)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:932)
Sync pending(Account {name=play@bitfire.at, type=bitfire.at.davdroid}, at.techbee.jtx.provider) = false
Sync pending(Account {name=card (play@bitfire.at) #350, type=at.bitfire.davdroid.address_book}, com.android.contacts) = false
Sending signal. PID: 3137 SIG: 9
---------------------------- PROCESS ENDED (3137) for package at.bitfire.davdroid ----------------------------
---------------------------- PROCESS STARTED (3843) for package at.bitfire.davdroid ----------------------------
Load libframework-connectivity-tiramisu-jni.so using APEX ns com_android_tethering for caller /apex/com.android.tethering/javalib/framework-connectivity-t.jar: ok

[rfc2822]

The same crash appears in Google Play (20 times in the last 28 days, possibly SDK-related):

Exception java.lang.IndexOutOfBoundsException: Index 1 out of bounds for length 1
  at jdk.internal.util.Preconditions.outOfBounds (Preconditions.java:64)
  at jdk.internal.util.Preconditions.outOfBoundsCheckIndex (Preconditions.java:70)
  at jdk.internal.util.Preconditions.checkIndex (Preconditions.java:266)
  at java.util.Objects.checkIndex (Objects.java:385)
  at java.util.ArrayList.get (ArrayList.java:434)
  at androidx.compose.material3.TabIndicatorOffsetNode.measure-3p2s80s (TabRow.kt:704)
  at androidx.compose.ui.node.LayoutModifierNodeCoordinator$LookaheadDelegateForLayoutModifierNode.measure-BRTryo0 (LayoutModifierNodeCoordinator.kt:93)
  at androidx.compose.ui.node.LookaheadPassDelegate$performMeasure$1.invoke (LookaheadPassDelegate.kt:438)
  at androidx.compose.ui.node.LookaheadPassDelegate$performMeasure$1.invoke (LookaheadPassDelegate.kt:437)
  at androidx.compose.runtime.snapshots.Snapshot$Companion.observe (Snapshot.java:502)
  at androidx.compose.runtime.snapshots.SnapshotStateObserver$ObservedScopeMap.observe (SnapshotStateObserver.kt:464)
  at androidx.compose.runtime.snapshots.SnapshotStateObserver.observeReads (SnapshotStateObserver.kt:248)
  at androidx.compose.ui.node.OwnerSnapshotObserver.observeReads$ui_release (OwnerSnapshotObserver.kt:124)
  at androidx.compose.ui.node.OwnerSnapshotObserver.observeMeasureSnapshotReads$ui_release (OwnerSnapshotObserver.kt:105)
  at androidx.compose.ui.node.OwnerSnapshotObserver.observeMeasureSnapshotReads$ui_release$default (OwnerSnapshotObserver.java:99)
  at androidx.compose.ui.node.LookaheadPassDelegate.performMeasure-BRTryo0$ui_release (LookaheadPassDelegate.java:437)
  at androidx.compose.ui.node.LayoutNodeLayoutDelegate.performLookaheadMeasure-BRTryo0$ui_release (LayoutNodeLayoutDelegate.kt:333)
  at androidx.compose.ui.node.LookaheadPassDelegate.remeasure-BRTryo0 (LookaheadPassDelegate.kt:481)
  at androidx.compose.ui.node.LookaheadPassDelegate.measure-BRTryo0 (LookaheadPassDelegate.kt:398)
  at androidx.compose.material3.TabRowKt$TabRowImpl$1$2$1.measure-3p2s80s (TabRow.kt:484)
  at androidx.compose.ui.layout.MultiContentMeasurePolicyImpl.measure-3p2s80s (MultiContentMeasurePolicy.kt:206)
  at androidx.compose.ui.node.InnerNodeCoordinator$LookaheadDelegateImpl.measure-BRTryo0 (InnerNodeCoordinator.kt:82)
  at androidx.compose.foundation.layout.FillNode.measure-3p2s80s (Size.kt:721)
  at androidx.compose.ui.node.LayoutModifierNodeCoordinator$LookaheadDelegateForLayoutModifierNode.measure-BRTryo0 (LayoutModifierNodeCoordinator.kt:93)
  at androidx.compose.ui.node.LookaheadPassDelegate$performMeasure$1.invoke (LookaheadPassDelegate.kt:438)
  at androidx.compose.ui.node.LookaheadPassDelegate$performMeasure$1.invoke (LookaheadPassDelegate.kt:437)
  at androidx.compose.runtime.snapshots.Snapshot$Companion.observe (Snapshot.java:502)
  at androidx.compose.runtime.snapshots.SnapshotStateObserver$ObservedScopeMap.observe (SnapshotStateObserver.kt:464)
  at androidx.compose.runtime.snapshots.SnapshotStateObserver.observeReads (SnapshotStateObserver.kt:248)
  at androidx.compose.ui.node.OwnerSnapshotObserver.observeReads$ui_release (OwnerSnapshotObserver.kt:124)
  at androidx.compose.ui.node.OwnerSnapshotObserver.observeMeasureSnapshotReads$ui_release (OwnerSnapshotObserver.kt:105)
  at androidx.compose.ui.node.OwnerSnapshotObserver.observeMeasureSnapshotReads$ui_release$default (OwnerSnapshotObserver.java:99)
  at androidx.compose.ui.node.LookaheadPassDelegate.performMeasure-BRTryo0$ui_release (LookaheadPassDelegate.java:437)
  at androidx.compose.ui.node.LayoutNodeLayoutDelegate.performLookaheadMeasure-BRTryo0$ui_release (LayoutNodeLayoutDelegate.kt:333)
  at androidx.compose.ui.node.LookaheadPassDelegate.remeasure-BRTryo0 (LookaheadPassDelegate.kt:481)
  at androidx.compose.ui.node.LookaheadPassDelegate.measure-BRTryo0 (LookaheadPassDelegate.kt:398)
  at androidx.compose.foundation.layout.BoxMeasurePolicy.measure-3p2s80s (Box.kt:145)
  at androidx.compose.ui.node.InnerNodeCoordinator$LookaheadDelegateImpl.measure-BRTryo0 (InnerNodeCoordinator.kt:82)
  at androidx.compose.ui.graphics.SimpleGraphicsLayerModifier.measure-3p2s80s (GraphicsLayerModifier.kt:794)
  at androidx.compose.ui.node.LayoutModifierNodeCoordinator$LookaheadDelegateForLayoutModifierNode.measure-BRTryo0 (LayoutModifierNodeCoordinator.kt:93)
  at androidx.compose.ui.node.LookaheadPassDelegate$performMeasure$1.invoke (LookaheadPassDelegate.kt:438)
  at androidx.compose.ui.node.LookaheadPassDelegate$performMeasure$1.invoke (LookaheadPassDelegate.kt:437)
  at androidx.compose.runtime.snapshots.Snapshot$Companion.observe (Snapshot.java:502)
  at androidx.compose.runtime.snapshots.SnapshotStateObserver$ObservedScopeMap.observe (SnapshotStateObserver.kt:464)
  at androidx.compose.runtime.snapshots.SnapshotStateObserver.observeReads (SnapshotStateObserver.kt:248)
  at androidx.compose.ui.node.OwnerSnapshotObserver.observeReads$ui_release (OwnerSnapshotObserver.kt:124)
  at androidx.compose.ui.node.OwnerSnapshotObserver.observeMeasureSnapshotReads$ui_release (OwnerSnapshotObserver.kt:105)
  at androidx.compose.ui.node.OwnerSnapshotObserver.observeMeasureSnapshotReads$ui_release$default (OwnerSnapshotObserver.java:99)
  at androidx.compose.ui.node.LookaheadPassDelegate.performMeasure-BRTryo0$ui_release (LookaheadPassDelegate.java:437)
  at androidx.compose.ui.node.LayoutNodeLayoutDelegate.performLookaheadMeasure-BRTryo0$ui_release (LayoutNodeLayoutDelegate.kt:333)
  at androidx.compose.ui.node.LookaheadPassDelegate.remeasure-BRTryo0 (LookaheadPassDelegate.kt:481)
  at androidx.compose.ui.node.LookaheadPassDelegate.measure-BRTryo0 (LookaheadPassDelegate.kt:398)
  at androidx.compose.animation.AnimatedEnterExitMeasurePolicy.measure-3p2s80s (AnimatedVisibility.kt:791)
  at androidx.compose.ui.node.InnerNodeCoordinator$LookaheadDelegateImpl.measure-BRTryo0 (InnerNodeCoordinator.kt:82)
  at androidx.compose.animation.EnterExitTransitionModifierNode.measure-3p2s80s (EnterExitTransition.kt:1160)
  at androidx.compose.ui.node.LayoutModifierNodeCoordinator$LookaheadDelegateForLayoutModifierNode.measure-BRTryo0 (LayoutModifierNodeCoordinator.kt:93)
  at androidx.compose.ui.graphics.BlockGraphicsLayerModifier.measure-3p2s80s (GraphicsLayerModifier.kt:718)
  at androidx.compose.ui.node.LayoutModifierNodeCoordinator$LookaheadDelegateForLayoutModifierNode.measure-BRTryo0 (LayoutModifierNodeCoordinator.kt:93)
  at androidx.compose.animation.AnimatedContentKt$AnimatedContent$6$1$1$1.invoke-3p2s80s (AnimatedContent.kt:842)
  at androidx.compose.animation.AnimatedContentKt$AnimatedContent$6$1$1$1.invoke (AnimatedContent.kt:841)
  at androidx.compose.ui.layout.LayoutModifierImpl.measure-3p2s80s (LayoutModifier.kt:278)
  at androidx.compose.ui.node.LayoutModifierNodeCoordinator$LookaheadDelegateForLayoutModifierNode.measure-BRTryo0 (LayoutModifierNodeCoordinator.kt:93)
  at androidx.compose.ui.node.LookaheadPassDelegate$performMeasure$1.invoke (LookaheadPassDelegate.kt:438)
  at androidx.compose.ui.node.LookaheadPassDelegate$performMeasure$1.invoke (LookaheadPassDelegate.kt:437)
  at androidx.compose.runtime.snapshots.Snapshot$Companion.observe (Snapshot.java:502)
  at androidx.compose.runtime.snapshots.SnapshotStateObserver$ObservedScopeMap.observe (SnapshotStateObserver.kt:464)
  at androidx.compose.runtime.snapshots.SnapshotStateObserver.observeReads (SnapshotStateObserver.kt:248)
  at androidx.compose.ui.node.OwnerSnapshotObserver.observeReads$ui_release (OwnerSnapshotObserver.kt:124)
  at androidx.compose.ui.node.OwnerSnapshotObserver.observeMeasureSnapshotReads$ui_release (OwnerSnapshotObserver.kt:105)
  at androidx.compose.ui.node.OwnerSnapshotObserver.observeMeasureSnapshotReads$ui_release$default (OwnerSnapshotObserver.java:99)
  at androidx.compose.ui.node.LookaheadPassDelegate.performMeasure-BRTryo0$ui_release (LookaheadPassDelegate.java:437)
  at androidx.compose.ui.node.LayoutNodeLayoutDelegate.performLookaheadMeasure-BRTryo0$ui_release (LayoutNodeLayoutDelegate.kt:333)
  at androidx.compose.ui.node.LookaheadPassDelegate.remeasure-BRTryo0 (LookaheadPassDelegate.kt:481)
  at androidx.compose.ui.node.LookaheadPassDelegate.measure-BRTryo0 (LookaheadPassDelegate.kt:398)
  at androidx.compose.animation.AnimatedContentMeasurePolicy.measure-3p2s80s (AnimatedContent.kt:894)
  at androidx.compose.ui.node.InnerNodeCoordinator$LookaheadDelegateImpl.measure-BRTryo0 (InnerNodeCoordinator.kt:82)
  at androidx.compose.animation.AnimatedContentTransitionScopeImpl$SizeModifierNode.measure-3p2s80s (AnimatedContent.kt:655)
  at androidx.compose.ui.node.LayoutModifierNodeCoordinator$LookaheadDelegateForLayoutModifierNode.measure-BRTryo0 (LayoutModifierNodeCoordinator.kt:93)
  at androidx.compose.ui.node.LookaheadPassDelegate$performMeasure$1.invoke (LookaheadPassDelegate.kt:438)
  at androidx.compose.ui.node.LookaheadPassDelegate$performMeasure$1.invoke (LookaheadPassDelegate.kt:437)
  at androidx.compose.runtime.snapshots.Snapshot$Companion.observe (Snapshot.java:502)
  at androidx.compose.runtime.snapshots.SnapshotStateObserver$ObservedScopeMap.observe (SnapshotStateObserver.kt:464)
  at androidx.compose.runtime.snapshots.SnapshotStateObserver.observeReads (SnapshotStateObserver.kt:248)
  at androidx.compose.ui.node.OwnerSnapshotObserver.observeReads$ui_release (OwnerSnapshotObserver.kt:124)
  at androidx.compose.ui.node.OwnerSnapshotObserver.observeMeasureSnapshotReads$ui_release (OwnerSnapshotObserver.kt:105)
  at androidx.compose.ui.node.OwnerSnapshotObserver.observeMeasureSnapshotReads$ui_release$default (OwnerSnapshotObserver.java:99)
  at androidx.compose.ui.node.LookaheadPassDelegate.performMeasure-BRTryo0$ui_release (LookaheadPassDelegate.java:437)
  at androidx.compose.ui.node.LayoutNodeLayoutDelegate.performLookaheadMeasure-BRTryo0$ui_release (LayoutNodeLayoutDelegate.kt:333)
  at androidx.compose.ui.node.LookaheadPassDelegate.remeasure-BRTryo0 (LookaheadPassDelegate.kt:481)
  at androidx.compose.ui.node.LookaheadPassDelegate.measure-BRTryo0 (LookaheadPassDelegate.kt:398)
  at androidx.compose.foundation.layout.BoxMeasurePolicy.measure-3p2s80s (Box.kt:145)
  at androidx.compose.ui.node.InnerNodeCoordinator$LookaheadDelegateImpl.measure-BRTryo0 (InnerNodeCoordinator.kt:82)
  at androidx.compose.animation.SharedTransitionScopeKt$SharedTransitionScope$1$1$1.invoke-3p2s80s (SharedTransitionScope.kt:128)
  at androidx.compose.animation.SharedTransitionScopeKt$SharedTransitionScope$1$1$1.invoke (SharedTransitionScope.kt:127)
  at androidx.compose.ui.layout.LayoutModifierImpl.measure-3p2s80s (LayoutModifier.kt:278)
  at androidx.compose.ui.node.LayoutModifierNodeCoordinator$LookaheadDelegateForLayoutModifierNode.measure-BRTryo0 (LayoutModifierNodeCoordinator.kt:93)
  at androidx.compose.ui.node.LookaheadPassDelegate$performMeasure$1.invoke (LookaheadPassDelegate.kt:438)
  at androidx.compose.ui.node.LookaheadPassDelegate$performMeasure$1.invoke (LookaheadPassDelegate.kt:437)
  at androidx.compose.runtime.snapshots.Snapshot$Companion.observe (Snapshot.java:502)
  at androidx.compose.runtime.snapshots.SnapshotStateObserver$ObservedScopeMap.observe (SnapshotStateObserver.kt:464)
  at androidx.compose.runtime.snapshots.SnapshotStateObserver.observeReads (SnapshotStateObserver.kt:248)
  at androidx.compose.ui.node.OwnerSnapshotObserver.observeReads$ui_release (OwnerSnapshotObserver.kt:124)
  at androidx.compose.ui.node.OwnerSnapshotObserver.observeMeasureSnapshotReads$ui_release (OwnerSnapshotObserver.kt:105)
  at androidx.compose.ui.node.OwnerSnapshotObserver.observeMeasureSnapshotReads$ui_release$default (OwnerSnapshotObserver.java:99)
  at androidx.compose.ui.node.LookaheadPassDelegate.performMeasure-BRTryo0$ui_release (LookaheadPassDelegate.java:437)
  at androidx.compose.ui.node.LayoutNodeLayoutDelegate.performLookaheadMeasure-BRTryo0$ui_release (LayoutNodeLayoutDelegate.kt:333)
  at androidx.compose.ui.node.LookaheadPassDelegate.remeasure-BRTryo0 (LookaheadPassDelegate.kt:481)
  at androidx.compose.ui.node.LookaheadPassDelegate.measure-BRTryo0 (LookaheadPassDelegate.kt:398)
  at androidx.compose.ui.node.MeasurePassDelegate.measure-BRTryo0 (MeasurePassDelegate.kt:450)
  at androidx.compose.foundation.layout.RowColumnMeasurePolicyKt.measure (RowColumnMeasurePolicy.kt:126)
  at androidx.compose.foundation.layout.RowColumnMeasurePolicyKt.measure$default (RowColumnMeasurePolicy.kt:77)
  at androidx.compose.foundation.layout.ColumnMeasurePolicy.measure-3p2s80s (Column.kt:205)
  at androidx.compose.ui.node.InnerNodeCoordinator.measure-BRTryo0 (InnerNodeCoordinator.kt:128)
  at androidx.compose.foundation.layout.PaddingValuesModifier.measure-3p2s80s (Padding.kt:470)
  at androidx.compose.ui.node.LayoutModifierNodeCoordinator.measure-BRTryo0 (LayoutModifierNodeCoordinator.kt:190)
  at androidx.compose.ui.node.MeasurePassDelegate$performMeasureBlock$1.invoke (MeasurePassDelegate.java:173)
  at androidx.compose.ui.node.MeasurePassDelegate$performMeasureBlock$1.invoke (MeasurePassDelegate.java:172)
  at androidx.compose.runtime.snapshots.Snapshot$Companion.observe (Snapshot.java:2495)
  at androidx.compose.runtime.snapshots.SnapshotStateObserver$ObservedScopeMap.observe (SnapshotStateObserver.kt:464)
  at androidx.compose.runtime.snapshots.SnapshotStateObserver.observeReads (SnapshotStateObserver.kt:248)
  at androidx.compose.ui.node.OwnerSnapshotObserver.observeReads$ui_release (OwnerSnapshotObserver.kt:124)
  at androidx.compose.ui.node.OwnerSnapshotObserver.observeMeasureSnapshotReads$ui_release (OwnerSnapshotObserver.kt:107)
  at androidx.compose.ui.node.MeasurePassDelegate.performMeasure-BRTryo0$ui_release (MeasurePassDelegate.kt:426)
  at androidx.compose.ui.node.MeasurePassDelegate.remeasure-BRTryo0 (MeasurePassDelegate.kt:477)
  at androidx.compose.ui.node.LayoutNode.remeasure-_Sx5XlM$ui_release (LayoutNode.kt:1283)
  at androidx.compose.ui.node.LayoutNode.remeasure-_Sx5XlM$ui_release$default (LayoutNode.kt:1276)
  at androidx.compose.ui.node.MeasureAndLayoutDelegate.doRemeasure-sdFAvZA (MeasureAndLayoutDelegate.kt:380)
  at androidx.compose.ui.node.MeasureAndLayoutDelegate.remeasureAndRelayoutIfNeeded (MeasureAndLayoutDelegate.kt:595)
  at androidx.compose.ui.node.MeasureAndLayoutDelegate.onlyRemeasureIfPending (MeasureAndLayoutDelegate.kt:689)
  at androidx.compose.ui.node.MeasureAndLayoutDelegate.forceMeasureTheSubtreeInternal (MeasureAndLayoutDelegate.kt:716)
  at androidx.compose.ui.node.MeasureAndLayoutDelegate.forceMeasureTheSubtreeInternal (MeasureAndLayoutDelegate.kt:723)
  at androidx.compose.ui.node.MeasureAndLayoutDelegate.forceMeasureTheSubtreeInternal (MeasureAndLayoutDelegate.kt:723)
  at androidx.compose.ui.node.MeasureAndLayoutDelegate.forceMeasureTheSubtreeInternal (MeasureAndLayoutDelegate.kt:723)
  at androidx.compose.ui.node.MeasureAndLayoutDelegate.forceMeasureTheSubtreeInternal (MeasureAndLayoutDelegate.kt:723)
  at androidx.compose.ui.node.MeasureAndLayoutDelegate.forceMeasureTheSubtree (MeasureAndLayoutDelegate.kt:680)
  at androidx.compose.ui.platform.AndroidComposeView.forceMeasureTheSubtree (AndroidComposeView.android.kt:1593)
  at androidx.compose.ui.node.MeasurePassDelegate.checkChildrenPlaceOrderForUpdates (MeasurePassDelegate.kt:256)
  at androidx.compose.ui.node.Owner.forceMeasureTheSubtree$default (Owner.java:256)
  at androidx.compose.ui.node.MeasurePassDelegate.remeasure-BRTryo0 (MeasurePassDelegate.kt:492)
  at androidx.compose.ui.node.LayoutNode.remeasure-_Sx5XlM$ui_release (LayoutNode.kt:1283)
  at androidx.compose.ui.node.MeasureAndLayoutDelegate.doRemeasure-sdFAvZA (MeasureAndLayoutDelegate.kt:378)
  at androidx.compose.ui.node.MeasureAndLayoutDelegate.remeasureOnly (MeasureAndLayoutDelegate.kt:658)
  at androidx.compose.ui.node.MeasureAndLayoutDelegate.measureOnly (MeasureAndLayoutDelegate.kt:446)
  at androidx.compose.ui.platform.AndroidComposeView.onMeasure (AndroidComposeView.android.kt:1671)
  at android.view.View.measure (View.java:29816)
  at androidx.compose.ui.platform.AbstractComposeView.internalOnMeasure$ui_release (ComposeView.android.kt:299)
  at androidx.compose.ui.platform.AbstractComposeView.onMeasure (ComposeView.android.kt:286)
  at android.view.View.measure (View.java:29816)
  at android.view.ViewGroup.measureChildWithMargins (ViewGroup.java:7415)
  at android.widget.FrameLayout.onMeasure (FrameLayout.java:207)
  at androidx.appcompat.widget.ContentFrameLayout.onMeasure (ContentFrameLayout.java:141)
  at android.view.View.measure (View.java:29816)
  at android.view.ViewGroup.measureChildWithMargins (ViewGroup.java:7415)
  at android.widget.LinearLayout.measureChildBeforeLayout (LinearLayout.java:1608)
  at android.widget.LinearLayout.measureVertical (LinearLayout.java:878)
  at android.widget.LinearLayout.onMeasure (LinearLayout.java:721)
  at android.view.View.measure (View.java:29816)
  at android.view.ViewGroup.measureChildWithMargins (ViewGroup.java:7415)
  at android.widget.FrameLayout.onMeasure (FrameLayout.java:207)
  at android.view.View.measure (View.java:29816)
  at android.view.ViewGroup.measureChildWithMargins (ViewGroup.java:7415)
  at android.widget.LinearLayout.measureChildBeforeLayout (LinearLayout.java:1608)
  at android.widget.LinearLayout.measureVertical (LinearLayout.java:878)
  at android.widget.LinearLayout.onMeasure (LinearLayout.java:721)
  at android.view.View.measure (View.java:29816)
  at android.view.ViewGroup.measureChildWithMargins (ViewGroup.java:7415)
  at android.widget.FrameLayout.onMeasure (FrameLayout.java:207)
  at com.android.internal.policy.DecorView.onMeasure (DecorView.java:1031)
  at android.view.View.measure (View.java:29816)
  at android.view.ViewRootImpl.performMeasure (ViewRootImpl.java:6084)
  at android.view.ViewRootImpl.measureHierarchy (ViewRootImpl.java:4260)
  at android.view.ViewRootImpl.performTraversals (ViewRootImpl.java:4662)
  at android.view.ViewRootImpl.doTraversal (ViewRootImpl.java:3924)
  at android.view.ViewRootImpl$TraversalRunnable.run (ViewRootImpl.java:12903)
  at android.view.Choreographer$CallbackRecord.run (Choreographer.java:1901)
  at android.view.Choreographer$CallbackRecord.run (Choreographer.java:1910)
  at android.view.Choreographer.doCallbacks (Choreographer.java:1367)
  at android.view.Choreographer.doFrame (Choreographer.java:1292)
  at android.view.Choreographer$FrameDisplayEventReceiver.run (Choreographer.java:1870)
  at android.os.Handler.handleCallback (Handler.java:995)
  at android.os.Handler.dispatchMessage (Handler.java:103)
  at android.os.Looper.loopOnce (Looper.java:273)
  at android.os.Looper.loop (Looper.java:363)
  at android.app.ActivityThread.main (ActivityThread.java:10060)
  at java.lang.reflect.Method.invoke
  at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run (RuntimeInit.java:632)
  at com.android.internal.os.ZygoteInit.main (ZygoteInit.java:975)

[rfc2822]

@ArnyminerZ Do you have an idea what the reason could be?

[ArnyminerZ]

It looks like it crashes because at some point, the current selected tab (pagerState.currentPage) is greater than the amount of tabs (in this case, it looks like index 1 was requested, but length was 1). So basically, I think idxCurrentPage of PrimaryTabRow in AccountScreen.kt is being set to a value higher than the amount of tabs available.

I think this can happen during account deletion, in some cases if all the has* variables (hasCalDav, hasCardDav, hasWebcal) get recomposed before recomposing the pager state. Which I guess may be happening because somehow the lazy initializer of the page count of rememberPagerState is delayed more than the other recompositions.

So, basically, I think this delay can be caused by the fact that we remove collections before removing the account (obviously), because of cross-references. If for some reason the account deletion takes a bit too long, the UI gets updated with the new non-existing collections, and as such, the tabs get hidden, and the index breaks.

---

I'm not sure about what can the solution be. Maybe we have to block re-compositions somehow by remembering a shared key or something like that.

Either that, or simply navigating back before removing the account, first go back to the accounts screen, then request deletion. In that case, the tabs won't be visible anymore, and the problem will be gone. I don't know how we handle errors in this case, but I don't see why not we could notify about errors in the accounts screen if any.

We would have to think then about how we take care of invalidAccount. I don't know if it would be necessary now. @rfc2822 do we have any kind of listener for accounts on system level? I mean, it's a bit strange, but in this situation, the only way of having an actual missing account unless the user removes it, would be that it's deleted by the system for some reason? Are we now reacting to this? Or just our updates? Because in that case, we can just get rid of invalidAccount, and just redirect back whenever the user requests to remove the account.

Edit: thinking about it, I don't know if we can do that now. Since we are using activity-scoped-models, I think that if we exit the activity, the view model scope from AccountScreenModel would be cancelled, so the coroutine that is deleting the account cannot be launched. We should launch it from some kind of worker, or from the accounts list model, which sounds a bit complicated/overkill. On the other hand, maybe it would be useful for being able to delete accounts directly from the accounts screen, instead of having to access the individual account, if we ever want to allow that.

[ArnyminerZ]

Actually, another issue regarding this invalidAccount field, is that when the account is renamed, you are moved back automatically to the accounts screen, because technically, the account that you were looking at no longer exists.

[rfc2822]

I think this can happen during account deletion, in some cases if all the has* variables (hasCalDav, hasCardDav, hasWebcal) get recomposed before recomposing the pager state. Which I guess may be happening because somehow the lazy initializer of the page count of rememberPagerState is delayed more than the other recompositions.

Also we have

davx5-ose/app/src/main/kotlin/at/bitfire/davdroid/ui/account/AccountScreen.kt

Lines 243 to 250 in cc43998

	val idxCalDav: Int? = if (hasCalDav) ++nextIdx else null
	val idxCardDav: Int? = if (hasCardDav) ++nextIdx else null
	val idxWebcal: Int? = if (hasWebcal) ++nextIdx else null
	val nrPages =
	(if (idxCalDav != null) 1 else 0) +
	(if (idxCardDav != null) 1 else 0) +
	(if (idxWebcal != null) 1 else 0)
	val pagerState = rememberPagerState(pageCount = { nrPages })

and the idx... and nrPages can change, but the pagerState is initialized with the initial pageCount and then remembered. So as I understand it, it's possible that the nrPages changes for instance from 3 to 1, but pagerState will still be remembered with pageCount=3.

A somehow radical idea: What if we always show all three tabs, and if there's for instance no CalDAV service on this server, then just show a text like "No CalDAV detected for this server"? @devvv4ever

Or otherwise we should make sure that dynamically changing the number of services keeps in sync with the tabs. (If that's possible, I'd prefer that.)

---

do we have any kind of listener for accounts on system level? I mean, it's a bit strange, but in this situation, the only way of having an actual missing account unless the user removes it, would be that it's deleted by the system for some reason?

Yes, the system, or another app, or maybe the user with adb... many possibilities. Unfortunately we use the AccountManager as the source for accounts, instead of our database (#843).

Are we now reacting to this? Or just our updates? Because in that case, we can just get rid of invalidAccount, and just redirect back whenever the user requests to remove the account.

Yes, that's the reason for invalidAccount. Also it's possible that the AccountActivity is called with an invalid account, for instance from a tasks app.

---

Actually, another issue regarding this invalidAccount field, is that when the account is renamed, you are moved back automatically to the accounts screen, because technically, the account that you were looking at no longer exists.

Yes, this was by intention. Maybe it would be better to stay in the same screen (with updated account name), but I'd see that as a feature.

[rfc2822]

BTW yes it's reproducible:

1. Create the account for the test server (with 3 tabs). Let's say account name is "test".
2. Navigate to Webcal tab.
3. Open the database inspector in Android Studio, open the services.db / service table.
4. Change the accountName of the CardDAV entry from "test" to "test123".

→

    Process: at.bitfire.davdroid, PID: 13381
    java.lang.IndexOutOfBoundsException: Index 2 out of bounds for length 2
    	at jdk.internal.util.Preconditions.outOfBounds(Preconditions.java:64)
    	at jdk.internal.util.Preconditions.outOfBoundsCheckIndex(Preconditions.java:70)
    	at jdk.internal.util.Preconditions.checkIndex(Preconditions.java:266)
    	at java.util.Objects.checkIndex(Objects.java:385)
    	at java.util.ArrayList.get(ArrayList.java:434)
    	at androidx.compose.material3.TabIndicatorOffsetNode.measure-3p2s80s(TabRow.kt:704)

because the CardDAV tab is dynamically removed, but the current index is still 2.

I think we would either have to update the pagerState according to nrPages, or use a fixed number of tabs.

[devvv4ever]

A somehow radical idea: What if we always show all three tabs, and if there's for instance no CalDAV service on this server, then just show a text like "No CalDAV detected for this server"? @devvv4ever

yes. possible, and to the worst to tell people what's going in. sometimes they even wonder why there is for example only caldav detected although they enter a carddav URL (👀 @ synology users). would be good if they would at least see the other tab then with some message.