IssueTracker

Build: AI-201.8743.12.41.6953283, 202011050241,

AI-201.8743.12.41.6953283, JRE 1.8.0_242-release-1644-b3-6222593x64 JetBrains s.r.o, OS Linux(amd64) v5.4.0-53-generic, screens 3440x1440

AS: 4.1.1; Kotlin plugin: 1.4.10-release-Studio4.1-1; Android Gradle Plugin: 4.1.1; Gradle: 6.5; NDK: from local.properties: (not specified), latest from SDK: 21.3.6528147; LLDB: pinned revision 3.1 not found, latest from SDK: (package not found); CMake: from local.properties: (not specified), latest from SDK: (not found), from PATH: 3.16.3

Affected version: androidx.fragment:fragment-ktx:1.2.5

Note: It looks like all versions starting from 1.2+ are affected

Description

When you have view pager with nested fragment that shows menu it will crash if you try to access menu items from onPrepareOptionsMenu.

Steps to reproduce

1. Add new ViewPager
2. Add first page - Fragment1
3. Add second page - Fragment2
4. In Fragment2 add nested Fragment21 and enable setHasOptionsMenu(true), also add new menu specific to this fragment
5. Try to access menu items of the Fragment21 from onPrepareOptionsMenu

Expected behavior

1. you should be able to access inflated menu, or
2. onPrepareOptionsMenu should never be called if menu is not inflated

Actual behavior

1. Menu was never inflated
2. It will crash when trying to access menu items from onPrepareOptionsMenu

Additional information

Looking through the changes I was able to pinpoint it to this behavior change: https://android.googlesource.com/platform/frameworks/support/+/4809e01d4153a5ba68cb68f36544baa1b90738c0

We are now also checking for parent menu visibility but only in dispatchCreateOptionsMenu, which means we would call dispatchPrepareOptionsMenu with no menu inflated since we skipped it.

Possible fix

1. Add matching check to all dispatch*Menu methods, or
2. Remove check from dispatchCreateOptionsMenu

Demo repository:

You can find repository with repro steps here: https://github.com/vovkab/fragment-menu-crash