Skip to content

[Android] Inverted FlatList + Text Input causing ANR on android 13 #35350

Description

@hannojg

Description

When having a FlatList and a TextInput in one view and the FlatList has the inverted={true} prop set and the android device is on API 33 (Android 13) the app will freeze/die in an ANR.

Screen.Recording.2022-11-15.at.12.52.22.mov

Version

0.70.5

Output of npx react-native info

info Fetching system and libraries information...
System:
    OS: macOS 12.6
    CPU: (8) arm64 Apple M1 Pro
    Memory: 96.63 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.15.1 - ~/.nvm/versions/node/v16.15.1/bin/node
    Yarn: 1.22.19 - ~/.yarn/bin/yarn
    npm: 8.11.0 - ~/.nvm/versions/node/v16.15.1/bin/npm
    Watchman: Not Found
  Managers:
    CocoaPods: Not Found
  SDKs:
    iOS SDK:
      Platforms: DriverKit 21.4, iOS 16.0, macOS 12.3, tvOS 16.0, watchOS 9.0
    Android SDK:
      API Levels: 28, 29, 30, 31, 32, 33
      Build Tools: 29.0.2, 30.0.2, 30.0.3, 31.0.0, 32.0.0, 32.1.0, 33.0.0
      System Images: android-32 | Google APIs ARM 64 v8a, android-32 | Google APIs Intel x86 Atom_64, android-32 | Google Play ARM 64 v8a, android-33 | Google APIs ARM 64 v8a, android-33 | Google Play ARM 64 v8a
      Android NDK: 21.4.7075529
  IDEs:
    Android Studio: Chipmunk 2021.2.1 Patch 1 Chipmunk 2021.2.1 Patch 1
    Xcode: 14.0.1/14A400 - /usr/bin/xcodebuild
  Languages:
    Java: 17.0.5 - /Users/hannogodecke/.jenv/shims/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: 18.1.0 => 18.1.0 
    react-native: 0.70.5 => 0.70.5 
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Steps to reproduce

  • Clone this repository
  • Run yarn
  • Run yarn start
  • Run yarn android, make sure the app launches on a device with android API 33 (Android 13)
  • Open the app
  • Open the developer tools and enable the performance monitor
  • Focus the text input
  • Run the following in your terminal:
adb shell input text "Lorem\ ipsum\ dolor\ sit\ amet,\ consetetur\ sadipscing\ elitr,\ sed\ diam\ nonumy\ eirmod\ tempor\ invidunt\ ut\ labore\ et\ dolore\ magna\ aliquyam\ erat,\ sed\ diam\ voluptua.\ At\ vero\ eos\ et\ accusam\ et\ justo\ duo\ dolores\ et\ ea\ rebum.\ Stet\ clita\ kasd\ gubergren,\ no\ sea\ takimata\ sanctus\ est\ Lorem\ ipsum\ dolor\ sit\ amet.\ Lorem\ ipsum\ dolor\ sit\ amet,\ consetetur\ sadipscing\ elitr,\ sed\ diam\ nonumy\ eirmod\ tempor\ invidunt\ ut\ labore\ et\ dolore\ magna\ aliquyam\ erat,\ sed\ diam\ voluptua.\ At\ vero\ eos\ et\ accusam\ et\ justo\ duo\ dolores\ et\ ea\ rebum.\ Stet\ clita\ kasd\ gubergren,\ no\ sea\ takimata\ sanctus\ est\ Lorem\ ipsum\ dolor\ sit\ amet."
  • Run it repeatedly until the app freezes (for me it takes ~ 3 repetitions until the ANR happens. On a real Pixel 4 of a colleague the app almost freezes immediately).

Snack, code example, screenshot, or link to a repository

https://github.com/hannojg/RN-AndroidAPI33-TextInput-ANR

Activity

  1. rudenick0309 commented on Nov 15, 2022

    @rudenick0309

    #35155
    The similar problem seems to occur even with only 'TextInput'

  2. hannojg commented on Nov 16, 2022

    @hannojg
    ContributorAuthor

    Interestingly the TextInput works for me on the device but stops working once it's next to an inverted FlatList. So I am not entirely sure if it's related, but I will test the reproduction of the issue you mentioned as well!

  3. hannojg commented on Nov 16, 2022

    @hannojg
    ContributorAuthor

    Observations I made:

    • I just put a plain android EditText view instead of RN's TextInput and the issue still happens (so I assume it isn't the TextInput implementation)
    • I tried it with VirtualizedList and the following styles (to re-create the inverted effect), but the ANR still happens:
    style={{
        transform: [{scaleY: -1}],
    }}
    contentContainerStyle={{
        transform: [{scaleY: -1}],
    }}
    • I tried it with ScrollView (used by VirtualizedList) using the styles above, and the issue keeps happening as well
    • If I don't do the inverted styling (its the same way VirtualizedList does it) the ANR doesn't happen

    Still looking for the place that's causing the issue 🔍

  4. JuanAlejandro commented on Nov 16, 2022

    @JuanAlejandro

    @hannojg we had the same problem with our app. We are using the inverted FlatList in our chat screen. One time I wasn't typing, I was just receiving messages and the app froze. So I'm not 100% sure the TextInput is needed to repro. The workaround we implemented was to invert the list with styles instead of the prop. You can check the solution in this Github issue.

    Ah! And also this issue was happening only for Pixel devices with Android 13. Since the first family of devices to get Android 13 was the Pixel, maybe that's why we only saw this issue in Pixel devices.

  5. hannojg commented on Nov 16, 2022

    @hannojg
    ContributorAuthor

    Wow thx for sharing this information, will give it a try!

    What I found is, that you don't even need a list. You just need two views that have scaleY: 1 set:

         <View
            style={{
              flex: 1,
              backgroundColor: 'red',
              transform: [{scaleY: -1}],
            }}>
            <View
              style={{
                transform: [{scaleY: -1}],
              }}>
               {/* Some more children inside here that got inverted */}

    However, I still need to input text constantly into a TextInput to cause the ANR 😅

  6. hannojg commented on Nov 16, 2022

    @hannojg
    ContributorAuthor

    Okay, the issue doesn't happen anymore if I also add scaleX: -1 👀 :

         <View
            style={{
              flex: 1,
              backgroundColor: 'red',
              transform: [{scaleX: -1}, {scaleY: -1}],
            }}>
            <View
              style={{
                transform: [{scaleX: -1}, {scaleY: -1}],
              }}>

    Going to dig into the native code to see where the issue happens, maybe also a bug with yoga?

  7. rudenick0309 commented on Nov 16, 2022

    @rudenick0309

    @hannojg @JuanAlejandro sorry for telling this, this problem was occured at real device of galaxy22 with google play system version - 22.07.01, but, after updating google play system version to '22.10.01'. the problem was disappeared. ( i could not test my local, that device is our customer. )
    so, i am looking for other galaxy22 for verifying. after results are out, i will write down.

  8. hannojg commented on Nov 17, 2022

    @hannojg
    ContributorAuthor

    I just found the offending line that's causing the issue. If I remove the view.setRotationX line the issue stops happening.
    At this point I am not sure whether this is a react native or an android bug. Will dig deeper:

    https://github.com/facebook/react-native/blob/e504141583fc2b695fb42a1037edf92c89ae552a/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java#L441-L483

  9. hannojg commented on Nov 22, 2022

    @hannojg
    ContributorAuthor

    So at this point, I was able to reproduce the issue with just a native android project (no react native involved). It's a bug in the android framework.

    I tried to create an issue ticket on the AOSP, however, after creation, I get weird permission errors and the bug ticket just disappears.

    Sorry for the ping, but maybe @cortinico you can help with opening a bug ticket for the android team (I recall you were an android engineer? 😅 )?

    The bug report for the native android looks like this:

    Setting scaleY: -1 and rotationX: 180 for two nested views causes an ANR on android API 33

    On Android 13 we are noticing an ANR, when we have two views that have each:

    scaleY: -1

    rotationX: 180

    set.

    In the video, you can see a green view, the parent of a blue view (and the blue view has a lot of children (list)). Both views have the transformation properties set (creating an "inverted" effect).

    We then have an EditText rendered underneath these views. When entering text the ANR happens. This is much easier to reproduce on a real device. We are reproducing with a Pixel 4 Android API 33 emulator. Here you need to enter text rapidly with the help of add shell input text to cause the ANR.

    In the bottom right you can see a small FPS counter (using Choreographer), and you can see that the app starts to freeze at some point.

    The full reproduction: https://github.com/hannojg/AndroidScaleYReproductionANR-

    I also attached an adb bugreport (it's from an emulator, so no sensitive data)

    The important code part to reproduce:

    class MainActivity : AppCompatActivity() {
        private lateinit var binding: ActivityMainBinding
    
        override fun onCreate(savedInstanceState: Bundle?) {
            super.onCreate(savedInstanceState)
    
            binding = ActivityMainBinding.inflate(layoutInflater)
            setContentView(binding.root)
    
            val parentView = RelativeLayout(this)
            parentView.layoutParams = ViewGroup.LayoutParams(1000, 2000)
            parentView.setBackgroundColor(0xFF00FF00.toInt())
            parentView.scaleY = -1f
            parentView.rotationX = 180f
    
            // add a child view to the parentView which we invert as well:
            val childView = LinearLayout(this)
            childView.orientation = LinearLayout.VERTICAL
            childView.layoutParams = ViewGroup.LayoutParams(800, 1800)
            childView.setBackgroundColor(0xFF0000FF.toInt())
            childView.scaleY = -1f
            childView.rotationX = 180f
    
            parentView.addView(childView)
    
            // add 20 text views to the child view
            for (i in 0..19) {
                val textView = TextView(this)
                val layoutPrams = ViewGroup.MarginLayoutParams(200, 100)
                layoutPrams.topMargin = 10
                textView.layoutParams = layoutPrams
                textView.setBackgroundColor(0xFFD3D3D3.toInt())
                textView.gravity = Gravity.CENTER
                textView.text = "Text $i"
    
                childView.addView(textView)
            }
    
            binding.rootLayout.addView(parentView)
        }
    }
    
    

    Thanks in advance for checking!

    android-reproduction.mov
  10. cortinico commented on Nov 23, 2022

    @cortinico
    Contributor

    Sorry for the ping, but maybe @cortinico you can help with opening a bug ticket for the android team (I recall you were an android engineer? 😅 )?

    You can do so by yourself here:
    https://issuetracker.google.com/issues

  11. 32 remaining items

  12. stelmakhivan commented on Oct 3, 2023

    @stelmakhivan

    @hannojg is this issue already fixed in react-native v0.72.4?
    https://github.com/facebook/react-native/releases/tag/v0.72.4

  13. winterdouglas commented on Oct 6, 2023

    @winterdouglas

    The android bug tracker seems to be working again: https://issuetracker.google.com/issues/287304310

    Meanwhile I am preparing a workaround fix for within react native.

    Any chances that your fix is also applied to 0.71? :) @hannojg

  14. roryabraham commented on Nov 8, 2023

    @roryabraham

    It looks like as of now it's only applied in 0.73: 3dd816c

  15. github-actions commented on May 21, 2024

    @github-actions

    This issue is stale because it has been open 180 days with no activity. Remove stale label or comment or this will be closed in 7 days.

  16. added
    StaleThere has been a lack of activity on this issue and it may be closed soon.
    on May 21, 2024
  17. phatmann commented on May 21, 2024

    @phatmann

    Is this actually fixed?

  18. removed
    StaleThere has been a lack of activity on this issue and it may be closed soon.
    on May 22, 2024
  19. hannojg commented on Jun 17, 2024

    @hannojg
    ContributorAuthor

    ah yeah it is, we applied a workaround in react native:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions