Unexpected behavior on quick forward-back navigation in Fragments about viewbindingpropertydelegate

[gmk57]

I couldn't press Back quickly enough, but it is easily reproducible programmatically, by calling FragmentManager.popBackStack() immediately after commit(), or from second fragment's onCreate().

[RuslanDemyanov]

Also reproduced this error when switching fragments programmatically, is there any way to destroy binding without using a handler?

[kirich1409]

Also reproduced this error when switching fragments programmatically, is there any way to destroy binding without using a handler?

Yes, you can keep reference to the delegate instance and call clear()

[kirich1409]

Need to check the bug with AndroidX Fragment 1.3.0. @midery , please do this and write about results

[midery]

Need to check the bug with AndroidX Fragment 1.3.0. @midery , please do this and write about results

Checked this issue in AndroidX Fragment 1.3.0. Surprisingly, it isn't reproducing anymore.

It happens because the Fragment.onDestroyView call is now "postponed" and cancellable in new FragmentManager implementation.

Fragment.destroyFragmentView now is bound to a FragmentTransition, which is used to show fragment's removal. If the transition is not yet completed - Fragment's view won't be destroyed.

Thus, if we will try to press back after the new fragment is added and current fragment is being replaced, it will happen before transition's completion, and the viewBindings would not be cleared.

So, the logs now are showing something like this:

1. Fragment B: onCreateView
2. Fragment B: BackPressed
3. Fragment A: onDestroyView
4. Fragment A: onCreateView <-- valid viewBinding here
5. Fragment A: onStart
6. Fragment A: onResume //on destroy is never called

TL;DR: Update the AndroidX Fragment artifact to version 1.3.0+ and this issue will be fixed.

[gmk57]

viewBindings would not be cleared

Sorry, but this is exactly the issue. I see no difference between Fragment 1.2.5 & 1.3.0 in this respect.
Here is a sample project to reproduce. After forward-back navigation the text on screen is wrong, because FirstFragment's binding still points to old view during onViewCreated.

[kirich1409]

viewBindings would not be cleared

Sorry, but this is exactly the issue. I see no difference between Fragment 1.2.5 & 1.3.0 in this respect.
Here is a sample project to reproduce. After forward-back navigation the text on screen is wrong, because FirstFragment's binding still points to old view during onViewCreated.

Please open separate issue with your bug