java.util.ConcurrentModificationException at java.util.ArrayList$Itr.next(ArrayList.java:860) at androidx.fragment.app.FragmentManager.executeOpsTogether(FragmentManager.java:2184) [236538905]

Fixed

Bug

Status Update

No update yet.

Description

be...@gmail.com

created issue #1

Jun 20, 2022 06:59PM

Component used: Fragment

Version used:
dependencies {
...
implementation 'androidx.appcompat:appcompat:1.4.2'
...

Devices/Android versions reproduced on: Samsung Galaxy Note 10 / Android 12

This is a problem in androidx library:

--------- beginning of crash
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: FATAL EXCEPTION: main
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: Process: sk.alteris.app.kalendarsk, PID: 6643
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: java.util.ConcurrentModificationException
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at java.util.ArrayList$Itr.next(ArrayList.java:860)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at androidx.fragment.app.FragmentManager.executeOpsTogether(FragmentManager.java:2184)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at androidx.fragment.app.FragmentManager.removeRedundantOperationsAndExecute(FragmentManager.java:2100)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at androidx.fragment.app.FragmentManager.execSingleAction(FragmentManager.java:1971)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at androidx.fragment.app.BackStackRecord.commitNowAllowingStateLoss(BackStackRecord.java:311)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at androidx.fragment.app.FragmentStatePagerAdapter.finishUpdate(FragmentStatePagerAdapter.java:274)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at androidx.viewpager.widget.ViewPager.populate(ViewPager.java:1244)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at androidx.viewpager.widget.ViewPager.populate(ViewPager.java:1092)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at androidx.viewpager.widget.ViewPager.onMeasure(ViewPager.java:1622)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.View.measure(View.java:27129)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:7980)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.widget.LinearLayout.measureChildBeforeLayout(LinearLayout.java:1552)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.widget.LinearLayout.measureVertical(LinearLayout.java:842)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.widget.LinearLayout.onMeasure(LinearLayout.java:721)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.View.measure(View.java:27129)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.widget.LinearLayout.measureVertical(LinearLayout.java:995)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.widget.LinearLayout.onMeasure(LinearLayout.java:721)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.View.measure(View.java:27129)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:7980)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.widget.FrameLayout.onMeasure(FrameLayout.java:197)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at androidx.appcompat.widget.ContentFrameLayout.onMeasure(ContentFrameLayout.java:145)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.View.measure(View.java:27129)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:7980)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at androidx.appcompat.widget.ActionBarOverlayLayout.onMeasure(ActionBarOverlayLayout.java:496)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.View.measure(View.java:27129)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:7980)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.widget.FrameLayout.onMeasure(FrameLayout.java:197)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.View.measure(View.java:27129)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:7980)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.widget.LinearLayout.measureChildBeforeLayout(LinearLayout.java:1552)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.widget.LinearLayout.measureVertical(LinearLayout.java:842)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.widget.LinearLayout.onMeasure(LinearLayout.java:721)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.View.measure(View.java:27129)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:7980)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.widget.FrameLayout.onMeasure(FrameLayout.java:197)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at com.android.internal.policy.DecorView.onMeasure(DecorView.java:1277)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.View.measure(View.java:27129)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.ViewRootImpl.performMeasure(ViewRootImpl.java:4528)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.ViewRootImpl.measureHierarchy(ViewRootImpl.java:3220)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:3525)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.ViewRootImpl.doTraversal(ViewRootImpl.java:2911)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.ViewRootImpl$TraversalRunnable.run(ViewRootImpl.java:10458)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.Choreographer$CallbackRecord.run(Choreographer.java:1108)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.Choreographer.doCallbacks(Choreographer.java:866)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.Choreographer.doFrame(Choreographer.java:797)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:1092)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.os.Handler.handleCallback(Handler.java:938)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.os.Handler.dispatchMessage(Handler.java:99)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.os.Looper.loopOnce(Looper.java:226)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.os.Looper.loop(Looper.java:313)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at android.app.ActivityThread.main(ActivityThread.java:8663)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at java.lang.reflect.Method.invoke(Native Method)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:567)
06-20 19:51:20.465 10283 6643 6643 E AndroidRuntime: at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1135)
06-20 19:51:20.465 10227 7726 7726 I HWRResConfig: [setPreloadRootDirectoryPath] mPreloadRootDirPath is set with /product/HWRDB/
06-20 19:51:20.467 1000 966 1340 D SyncManager: userId - 0 auto - true
06-20 19:51:20.469 1000 966 1544 D Debug : low && ship && 3rdparty app crash, do not dump
06-20 19:51:20.469 1000 966 1544 W ActivityManager: crash : sk.alteris.app.kalendarsk,10283

Comments

il...@google.com <il...@google.com> #2Jun 21, 2022 03:36PM

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

commit b697d397d051f91a6f6ed95a55af421f048f3c89
Author: Sanura Njaka <sanura@google.com>
Date: Wed Aug 02 12:16:12 2023

Folding lifecycle-livedata-core-ktx back into main
module

Now that Lifecycle tests are written in kotlin, LiveDataTest
can be moved from lifecycle-livedata-core-ktx into LiveDataTest
within the main lifecycle-livedata-core module.

Test: ./gradlew bOS
Bug: 274800183
Change-Id: I98806ad751bd674a554de665e0e16bf05d67900d

D lifecycle/lifecycle-livedata-core-ktx/src/test/java/androidx/lifecycle/LiveDataTest.kt
M lifecycle/lifecycle-livedata-core/src/test/java/androidx/lifecycle/LiveDataTest.kt

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

be...@gmail.com <be...@gmail.com> #3Jun 21, 2022 05:53PM

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

commit cd3c1d7751462b5e42ae54e2d68299f955dcb6c3
Author: Sanura Njaka <sanura@google.com>
Date: Mon Aug 07 07:34:20 2023

Folding lifecycle-livedata-ktx back into main
module

Now that Lifecycle tests are written in kotlin, all files in lifecycle-livedata-ktx can now be added into the main lifecycle-livedata module.

RelNote: "All files from the lifecycle-livedata-ktx modules have been moved into the main lifecycle-livedata module"
Test: ./gradlew bOS
Bug: 274800183
Change-Id: I10c6f70c82a64e80eec704f548911f42bdd0b56a

M lifecycle/lifecycle-livedata-ktx/api/current.ignore
M lifecycle/lifecycle-livedata-ktx/api/current.txt
M lifecycle/lifecycle-livedata-ktx/api/restricted_current.ignore
M lifecycle/lifecycle-livedata-ktx/api/restricted_current.txt
M lifecycle/lifecycle-livedata/api/current.txt
M lifecycle/lifecycle-livedata/api/restricted_current.txt
M lifecycle/lifecycle-livedata/build.gradle
M lifecycle/lifecycle-livedata/src/androidTest/java/androidx.lifecycle/FlowAsLiveDataIntegrationTest.kt
M lifecycle/lifecycle-livedata/src/main/java/androidx/lifecycle/CoroutineLiveData.kt
M lifecycle/lifecycle-livedata/src/main/java/androidx/lifecycle/FlowLiveData.kt
M lifecycle/lifecycle-livedata/src/test/java/androidx/lifecycle/BuildLiveDataTest.kt
M lifecycle/lifecycle-livedata/src/test/java/androidx/lifecycle/FlowAsLiveDataTest.kt
M lifecycle/lifecycle-livedata/src/test/java/androidx/lifecycle/LiveDataAsFlowTest.kt
M lifecycle/lifecycle-livedata/src/test/java/androidx/lifecycle/LiveDataFlowJavaTest.java
M lifecycle/lifecycle-livedata/src/test/java/androidx/lifecycle/ScopesRule.kt

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

il...@google.com <il...@google.com> #4Jun 21, 2022 05:58PM

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

commit 32c1896b0506529674f03076a1141daaa0749b28
Author: Sanura Njaka <sanura@google.com>
Date: Tue Aug 22 11:20:50 2023

Moving ScopesRule into util package

ScopesRule.kt will now be included in lifecycle-livedata's util package

Test: ./gradlew bOS
BUg: 274800183
Change-Id: I2df34eebffcdacc716ce0235e29601436918f955

M lifecycle/lifecycle-livedata/src/test/java/androidx/lifecycle/BuildLiveDataTest.kt
M lifecycle/lifecycle-livedata/src/test/java/androidx/lifecycle/FlowAsLiveDataTest.kt
M lifecycle/lifecycle-livedata/src/test/java/androidx/lifecycle/LiveDataAsFlowTest.kt
M lifecycle/lifecycle-livedata/src/test/java/androidx/lifecycle/util/ScopesRule.kt

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

be...@gmail.com <be...@gmail.com> #5Jun 21, 2022 06:35PM

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

commit 4140c78328013f4fea4bd48f286c46a98c358270
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Fri Jan 26 14:43:43 2024

Move `lifecycle-viewmodel-ktx` source to the base module

* Transferred source code from `lifecycle-viewmodel-ktx` to the main `lifecycle-viewmodel` module.
* Kotlin STD Lib is now a dependency in the base AndroidX Lifecycle artifacts, separate `-ktx` modules for Kotlin extensions are no longer needed.
* Renamed the file from `ViewModel` to `ViewModelCoroutineScope` to reflect the content: coroutines support. Use `file:JvmName("ViewModelKt")` to maintain source compatibility.
* Add a resolutionStrategy to `:playground` and `:presentation` in `:tv:integration-tests` to resolve "minifyReleaseWithR8 FAILED: Class is defined multiple times" issue caused by moving classes from `-ktx` to the base module.

RelNote: "The `lifecycle-viewmodel-ktx` kotlin extensions have now been moved to the base lifecycle module."
Bug: 274800183
Test: ViewModelCoroutinesTest
Change-Id: Id787b6db6817e69e4ec50748a07ccdbef074e99d

A lifecycle/lifecycle-viewmodel-ktx/api/current.ignore
M lifecycle/lifecycle-viewmodel-ktx/api/current.txt
A lifecycle/lifecycle-viewmodel-ktx/api/restricted_current.ignore
M lifecycle/lifecycle-viewmodel-ktx/api/restricted_current.txt
M lifecycle/lifecycle-viewmodel-ktx/build.gradle
M lifecycle/lifecycle-viewmodel/api/current.txt
M lifecycle/lifecycle-viewmodel/api/restricted_current.txt
M lifecycle/lifecycle-viewmodel/build.gradle
M lifecycle/lifecycle-viewmodel/src/androidTest/java/androidx/lifecycle/ViewModelCoroutineScopeTest.kt
M lifecycle/lifecycle-viewmodel/src/main/java/androidx/lifecycle/ViewModelCoroutineScope.kt
M tv/integration-tests/playground/build.gradle
M tv/integration-tests/presentation/build.gradle

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

be...@gmail.com <be...@gmail.com> #6Jul 5, 2022 03:56PM

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

commit ce327843c4c6bfeb3410a1203e0e2aac0f0a59cc
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Fri Jan 26 16:21:38 2024

Move `lifecycle-livedata-core-ktx` source to the base module

* Transferred source code from `lifecycle-livedata-core-ktx` to the base `lifecycle-livedata-core` module.
* Kotlin STD Lib is now a dependency in the base AndroidX Lifecycle artifacts, separate `-ktx` modules for Kotlin extensions are no longer needed.

RelNote: "The `lifecycle-livedata-core-ktx` kotlin extensions have now been moved to the base lifecycle module."
Bug: 274800183
Test: manual
Change-Id: I54a3d4346b4a9c5861f5f3bdb0079dbb27f29668

A lifecycle/lifecycle-livedata-core-ktx/api/current.ignore
M lifecycle/lifecycle-livedata-core-ktx/api/current.txt
A lifecycle/lifecycle-livedata-core-ktx/api/restricted_current.ignore
M lifecycle/lifecycle-livedata-core-ktx/api/restricted_current.txt
M lifecycle/lifecycle-livedata-core-ktx/build.gradle
M lifecycle/lifecycle-livedata-core/api/current.txt
M lifecycle/lifecycle-livedata-core/api/restricted_current.txt
M lifecycle/lifecycle-livedata-core/src/main/java/androidx/lifecycle/LiveData.kt

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

App Widgets Demo1.zip

150 KB

Download

be...@gmail.com <be...@gmail.com> #7Jul 5, 2022 04:10PM

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

commit b08ac008ee00cedd68d5f21a54c4035a7778c875
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Feb 07 11:19:04 2024

Move `lifecycle-runtime-ktx` source to the base module

* Transferred source code from `lifecycle-runtime-ktx` to the main `lifecycle-runtime` module.
* Kotlin STD Lib is now a dependency in the base AndroidX Lifecycle artifacts, separate `-ktx` modules for Kotlin extensions are no longer needed.
* Changed the method signature of `androidx.lifecycle.View.findViewTreeLifecycleOwner` from `View.findViewTreeLifecycleOwner()` to `findViewTreeLifecycleOwner(view)` to avoid a conflicting overload.
* Add a resolutionStrategy to `:playground` and `:presentation` in `:tv:integration-tests` to resolve "minifyReleaseWithR8 FAILED: Class is defined multiple times" issue caused by moving classes from `-ktx` to the base module.

RelNote: "The `lifecycle-runtime-ktx` kotlin extensions have now been moved to the base lifecycle module."
Bug: 274800183
Test: FlowWithLifecycleTest
Test: LaunchWhenTest
Test: PausingDispatcherTest
Test: RepeatOnLifecycleTest
Test: ViewTreeLifecycleOwnerTest
Test: WithLifecycleStateTest
Change-Id: Ic368640a7e66dd0b748601d61b7aa23d99e5a1d6

D lifecycle/lifecycle-runtime-compose/api/current.ignore
D lifecycle/lifecycle-runtime-compose/api/restricted_current.ignore
A lifecycle/lifecycle-runtime-ktx/api/current.ignore
M lifecycle/lifecycle-runtime-ktx/api/current.txt
A lifecycle/lifecycle-runtime-ktx/api/restricted_current.ignore
M lifecycle/lifecycle-runtime-ktx/api/restricted_current.txt
M lifecycle/lifecycle-runtime-ktx/build.gradle
D lifecycle/lifecycle-runtime-ktx/src/androidInstrumentedTest/kotlin/androidx/lifecycle/ViewTreeLifecycleOwnerTest.kt
M lifecycle/lifecycle-runtime/api/current.txt
M lifecycle/lifecycle-runtime/api/restricted_current.txt
M lifecycle/lifecycle-runtime/build.gradle
M lifecycle/lifecycle-runtime/src/androidInstrumentedTest/kotlin/androidx/lifecycle/Expectations.kt
M lifecycle/lifecycle-runtime/src/androidInstrumentedTest/kotlin/androidx/lifecycle/FakeLifecycleOwner.kt
M lifecycle/lifecycle-runtime/src/androidInstrumentedTest/kotlin/androidx/lifecycle/FlowWithLifecycleTest.kt
M lifecycle/lifecycle-runtime/src/androidInstrumentedTest/kotlin/androidx/lifecycle/LaunchWhenTest.kt
M lifecycle/lifecycle-runtime/src/androidInstrumentedTest/kotlin/androidx/lifecycle/PausingDispatcherTest.kt
M lifecycle/lifecycle-runtime/src/androidInstrumentedTest/kotlin/androidx/lifecycle/RepeatOnLifecycleTest.kt
M lifecycle/lifecycle-runtime/src/androidInstrumentedTest/kotlin/androidx/lifecycle/TaskTracker.kt
M lifecycle/lifecycle-runtime/src/androidInstrumentedTest/kotlin/androidx/lifecycle/TrackedExecutor.kt
M lifecycle/lifecycle-runtime/src/androidInstrumentedTest/kotlin/androidx/lifecycle/WithLifecycleStateTest.kt
M lifecycle/lifecycle-runtime/src/androidMain/AndroidManifest.xml
M lifecycle/lifecycle-runtime/src/androidMain/kotlin/androidx/lifecycle/View.android.kt
M lifecycle/lifecycle-runtime/src/commonMain/kotlin/androidx/lifecycle/FlowExt.kt
M lifecycle/lifecycle-runtime/src/commonMain/kotlin/androidx/lifecycle/RepeatOnLifecycle.kt
M lifecycle/lifecycle-runtime/src/commonMain/kotlin/androidx/lifecycle/WithLifecycleState.kt
M tv/integration-tests/playground/build.gradle
M tv/integration-tests/presentation/build.gradle

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

il...@google.com <il...@google.com> Jul 13, 2022 11:19PM

Assigned to jb...@google.com.

to...@gmail.com <to...@gmail.com> #8Aug 23, 2022 05:11PM

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

commit ce705f9552422d5cccd145244b3402fbade2768f
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Mon Feb 12 15:13:37 2024

Move `lifecycle-runtime-ktx-lint` to base module

* Rename `lifecycle-runtime-ktx-lint` to `lifecycle-runtime-lint`.
* Move `lintPublish` from obsolete `lifecycle-runtime-ktx` to `lifecycle-runtime`.

Bug: 274800183
Test: manual
Change-Id: I83bc2787fb3636b32e772f100bb8733793829708

M lifecycle/lifecycle-runtime-ktx/build.gradle
M lifecycle/lifecycle-runtime-lint/build.gradle
M lifecycle/lifecycle-runtime-lint/src/main/java/androidx/lifecycle/lint/LifecycleRuntimeIssueRegistry.kt
M lifecycle/lifecycle-runtime-lint/src/main/java/androidx/lifecycle/lint/LifecycleWhenChecks.kt
M lifecycle/lifecycle-runtime-lint/src/main/java/androidx/lifecycle/lint/RepeatOnLifecycleDetector.kt
M lifecycle/lifecycle-runtime-lint/src/main/resources/META-INF/services/com.android.tools.lint.client.api.IssueRegistry
M lifecycle/lifecycle-runtime-lint/src/test/java/androidx/lifecycle/runtime/lint/ApiLintVersionsTest.kt
M lifecycle/lifecycle-runtime-lint/src/test/java/androidx/lifecycle/runtime/lint/LifecycleWhenChecksTest.kt
M lifecycle/lifecycle-runtime-lint/src/test/java/androidx/lifecycle/runtime/lint/RepeatOnLifecycleDetectorTest.kt
M lifecycle/lifecycle-runtime-lint/src/test/java/androidx/lifecycle/runtime/lint/WhenMethodsTest.kt
M lifecycle/lifecycle-runtime-lint/src/test/java/androidx/lifecycle/runtime/lint/stubs/Stubs.kt
M lifecycle/lifecycle-runtime/build.gradle
M settings.gradle

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

be...@gmail.com <be...@gmail.com> #9Aug 23, 2022 07:08PM

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

commit 7497f5ec6cd6deda184e32b612f468d739916be8
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Mon Feb 12 14:31:43 2024

Move `lifecycle-livedata-core-ktx-lint` to base module

* Rename `lifecycle-livedata-core-ktx-lint` to `lifecycle-livedata-core-lint`.
* Move `lintPublish` from obsolete `lifecycle-livedata-core-ktx` to `lifecycle-livedata-core`.

Bug: 274800183
Test: manual
Change-Id: Ib306e4c199d8a26d3465ad479249497909c46ff4

M lifecycle/lifecycle-livedata-core-ktx/build.gradle
M lifecycle/lifecycle-livedata-core-lint/build.gradle
M lifecycle/lifecycle-livedata-core-lint/src/main/java/androidx/lifecycle/lint/LiveDataCoreIssueRegistry.kt
M lifecycle/lifecycle-livedata-core-lint/src/main/java/androidx/lifecycle/lint/NonNullableMutableLiveDataDetector.kt
M lifecycle/lifecycle-livedata-core-lint/src/main/resources/META-INF/services/com.android.tools.lint.client.api.IssueRegistry
M lifecycle/lifecycle-livedata-core-lint/src/test/java/androidx/lifecycle/livedata/core/lint/ApiLintVersionsTest.kt
M lifecycle/lifecycle-livedata-core-lint/src/test/java/androidx/lifecycle/livedata/core/lint/NonNullableMutableLiveDataDetectorTest.kt
M lifecycle/lifecycle-livedata-core-lint/src/test/java/androidx/lifecycle/livedata/core/lint/stubs/Stubs.kt
M lifecycle/lifecycle-livedata-core/build.gradle
M settings.gradle

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

jb...@google.com <jb...@google.com> #10Sep 22, 2022 05:21PM

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

androidx.lifecycle:lifecycle-livedata-core:2.8.0-alpha02
androidx.lifecycle:lifecycle-livedata-core-ktx:2.8.0-alpha02
androidx.lifecycle:lifecycle-runtime:2.8.0-alpha02
androidx.lifecycle:lifecycle-runtime-compose:2.8.0-alpha02
androidx.lifecycle:lifecycle-runtime-ktx:2.8.0-alpha02
androidx.lifecycle:lifecycle-viewmodel:2.8.0-alpha02
androidx.lifecycle:lifecycle-viewmodel-ktx:2.8.0-alpha02

ap...@google.com <ap...@google.com> #11Mar 21, 2023 09:50PM

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

commit 95e15d8df12b8ce7cc32abdfd2e80dc667e893d6
Author: Jeremy Woods <jbwoods@google.com>
Date: Tue Mar 21 18:37:17 2023

Add @MainThread annotations to instant commit APIs

Adding @MainThread annotations to the commitNow,
executePendingTransactions, and popBackStackImmediate APIs to ensure we
get build time checks along with the runtime check already in place.

RelNote: "The fragment `commitNow()`, `executePendingTransactions()`,
and `popBackStackImmediate()` APIs will now all throw build errors when
they are not called from the main thread."
Test: all tests pass
Bug: 236538905

Change-Id: Ic966543935dd77a4c149cfc4b796bd912c558fb2

M fragment/fragment-ktx/api/current.txt
M fragment/fragment-ktx/api/public_plus_experimental_current.txt
M fragment/fragment-ktx/api/restricted_current.txt
M fragment/fragment-ktx/src/main/java/androidx/fragment/app/FragmentManager.kt
M fragment/fragment/api/current.txt
M fragment/fragment/api/public_plus_experimental_current.txt
M fragment/fragment/api/restricted_current.txt
M fragment/fragment/src/main/java/androidx/fragment/app/FragmentManager.java
M fragment/fragment/src/main/java/androidx/fragment/app/FragmentTransaction.java

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

jb...@google.com <jb...@google.com> #12Mar 21, 2023 10:04PM

Marked as fixed.

This has been fixed internally and will be available in the the Fragment 1.6.0-alpha09 release.

na...@google.com <na...@google.com> #13Apr 5, 2023 06:27PM

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

androidx.fragment:fragment:1.6.0-alpha09
androidx.fragment:fragment-ktx:1.6.0-alpha09

ma...@gmail.com <ma...@gmail.com> #14Aug 25, 2023 09:56PM

I'm using fragment-ktx version 1.6.1, and after an upgrade to androidx.navigation 2.7.1 (from 2.3.x) we started experiencing very similar issue:

E  java.util.ConcurrentModificationException
    at java.util.ArrayList$Itr.next(ArrayList.java:860)
    at androidx.fragment.app.FragmentManager.executeOpsTogether(FragmentManager.java:1915)
    at androidx.fragment.app.FragmentManager.removeRedundantOperationsAndExecute(FragmentManager.java:1845)
    at androidx.fragment.app.FragmentManager.popBackStackImmediate(FragmentManager.java:891)
    at androidx.fragment.app.FragmentManager.popBackStackImmediate(FragmentManager.java:797)
    at androidx.fragment.app.FragmentManager.handleOnBackPressed(FragmentManager.java:717)
    at androidx.fragment.app.FragmentManager$1.handleOnBackPressed(FragmentManager.java:447)
    at androidx.activity.OnBackPressedDispatcher.onBackPressed(OnBackPressedDispatcher.kt:213)
    at androidx.activity.OnBackPressedDispatcher$2.invoke(OnBackPressedDispatcher.kt:102)
    at androidx.activity.OnBackPressedDispatcher$2.invoke(OnBackPressedDispatcher.kt:102)
    at androidx.activity.OnBackPressedDispatcher$Api33Impl.createOnBackInvokedCallback$lambda$0(OnBackPressedDispatcher.kt:286)
    at androidx.activity.OnBackPressedDispatcher$Api33Impl.$r8$lambda$tzBBo0FLedRCWSI6_icF8qhF74s(Unknown Source:0)
    at androidx.activity.OnBackPressedDispatcher$Api33Impl$$ExternalSyntheticLambda0.onBackInvoked(Unknown Source:2)
    at android.view.ViewRootImpl$NativePreImeInputStage.onProcess(ViewRootImpl.java:7558)
    at android.view.ViewRootImpl$InputStage.deliver(ViewRootImpl.java:7213)
    at android.view.ViewRootImpl.deliverInputEvent(ViewRootImpl.java:10788)
    at android.view.ViewRootImpl.doProcessInputEvents(ViewRootImpl.java:10676)
    at android.view.ViewRootImpl.enqueueInputEvent(ViewRootImpl.java:10632)
    at android.view.ViewRootImpl$WindowInputEventReceiver.onInputEvent(ViewRootImpl.java:10926)
    at android.view.InputEventReceiver.dispatchInputEvent(InputEventReceiver.java:285)
    at android.os.MessageQueue.nativePollOnce(Native Method)
    at android.os.MessageQueue.next(MessageQueue.java:335)
    at android.os.Looper.loopOnce(Looper.java:186)
    at android.os.Looper.loop(Looper.java:313)
    at android.app.ActivityThread.main(ActivityThread.java:8757)
    at java.lang.reflect.Method.invoke(Native Method)
    at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:571)
    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1067)

I wasn't able to come up with a repro yet, since this only happens occasionally. It seems it only happens when navigating back from a backstack with multiple entries. The app opts in the predictive back feature, it uses single navigation component graph + I haven't found single FragmentManager interaction that would be called outside of the scope of the main thread.

It looks like adding @MainThread annotations wasn't enough, and people are still able to call FragmentManager methods the wrong way (I'm referring to code from latest stable navigation component library). Have you considered fixing the root cause of ConcurrentModificationException? Or maybe it would be possible to help developers identify offending code (the ones that suposedly calls FragmentManager methods on a wrong thread)? Looking at the stacktrace it looks like the issue is fully within the FragmentManager implementation 🤷‍♂️

Issue 236538905

Description

Issue summary

Comments

il...@google.com <il...@google.com> #2Jun 21, 2022 03:36PM

be...@gmail.com <be...@gmail.com> #3Jun 21, 2022 05:53PM

il...@google.com <il...@google.com> #4Jun 21, 2022 05:58PM

be...@gmail.com <be...@gmail.com> #5Jun 21, 2022 06:35PM

be...@gmail.com <be...@gmail.com> #6Jul 5, 2022 03:56PM

be...@gmail.com <be...@gmail.com> #7Jul 5, 2022 04:10PM

il...@google.com <il...@google.com> Jul 13, 2022 11:19PM

to...@gmail.com <to...@gmail.com> #8Aug 23, 2022 05:11PM

be...@gmail.com <be...@gmail.com> #9Aug 23, 2022 07:08PM

jb...@google.com <jb...@google.com> #10Sep 22, 2022 05:21PM

ap...@google.com <ap...@google.com> #11Mar 21, 2023 09:50PM

jb...@google.com <jb...@google.com> #12Mar 21, 2023 10:04PM

na...@google.com <na...@google.com> #13Apr 5, 2023 06:27PM

ma...@gmail.com <ma...@gmail.com> #14Aug 25, 2023 09:56PM

Add comment

Issue metadata