Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Crash in [@ java.lang.IllegalArgumentException: at androidx.dynamicanimation.animation.DynamicAnimation.start(DynamicAnimation.java)] by overscrolling the tab switch gesture #13856

Closed
TrianguloY opened this issue Aug 15, 2020 · 1 comment
Labels
b:crash Crashes Fenix: should link to Sentry, Crash-Stats or GPlay info 🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified Feature:Gesture Feature:Tabs

Comments

@TrianguloY
Copy link
Contributor

TrianguloY commented Aug 15, 2020

This is about the scrolling between tabs feature (when you scroll the navigation bar to switch tabs)

Under normal use, it's impossible to scroll more than the screen width for physical reasons, however there are several ways to achieve this, and when doing so and releasing the finger, the browser crashes. It is repeatable.

Steps to reproduce

Make sure you have at least two tabs opened, and you are on one of them so you can initiate the tab-swapping scrolling. Now:

  • Way 1) Multiple fingers (unusual but easy to perform)

    • Press with one finger on the navigation bar and start the scroll, don't release the finger yet.
    • Press with another finger (anywhere on the screen), and release the first one. Now you can still move the tabs with this second finger.
    • Repeat the process pressing and releasing fingers so that you can scroll more than the screen width.
      photo_2020-08-15_11-14-16
  • Way 2) Windowed app (maybe it requires the 'allow any app in windowed mode' setting enabled)

    • Open the browser in windowed mode (this vary between devices and android version)
    • Simply scroll more than the window width (possible because it is smaller than the screen width)
      photo_2020-08-15_11-14-20
      Note: this also makes the tab go outside of the app window, but not sure if that is the browser issue of android.
  • Way 3) Horizontal split screen. Probably the easiest for a user

    • Activate split screen between the browser and another app.
    • Turn to landscape, make sure the browser is at the left if you are going to swipe to the right (or vice versa)
    • Simply swipe, now you have twice the browser width
      photo_2020-08-15_11-14-30
      Note: when making a screenshot it shows the tab inside the other app. While performing it it doesn't. Maybe an android issue?

Now, in any of those three ways, release the finger.

Expected behavior

The browser doesn't crash and:

  • Ideally (but probably hard to accomplish and not a big requirement) you can scroll to the second, third or even more next tab
  • Better: the scrolling stops when the other tab is fully visible (that is, when the scroll reaches the browser width)

Actual behavior

The browser crashes. Crash report here: https://crash-stats.mozilla.org/report/index/d5dca2f7-371f-4180-aa0d-4d4b70200815

Device information

  • Android device: Nexus 5X with Oreo 8.1.0
  • Fenix version: Nightly 200814 06:02 (Build #2015757875)

┆Issue is synchronized with this Jira Task

@TrianguloY TrianguloY added the 🐞 bug Crashes, Something isn't working, .. label Aug 15, 2020
@github-actions github-actions bot added the needs:triage Issue needs triage label Aug 15, 2020
@mcarare mcarare added the b:crash Crashes Fenix: should link to Sentry, Crash-Stats or GPlay info label Aug 17, 2020
@kbrosnan kbrosnan changed the title [Bug] Tab-Scrolling more than the width of the screen crashes the browser Crash in [@ java.lang.IllegalArgumentException: at androidx.dynamicanimation.animation.DynamicAnimation.start(DynamicAnimation.java)] by overscrolling the tab switch gesture Aug 18, 2020
@kbrosnan kbrosnan added Feature:Gesture Feature:Tabs and removed needs:triage Issue needs triage labels Aug 18, 2020
@kbrosnan kbrosnan added this to Needs triage in Android: Crash Tracking via automation Aug 18, 2020
@kbrosnan kbrosnan moved this from Needs triage to Crashes with STR in Android: Crash Tracking Aug 18, 2020
@jakefleming156 jakefleming156 added this to Sour Patch in Bug Fest Aug 26, 2020
@jakefleming156 jakefleming156 removed this from Sour Patch in Bug Fest Aug 26, 2020
@jakefleming156 jakefleming156 added this to Sour Patch in Bug Fest Aug 26, 2020
@ekager ekager added the eng:qa:needed QA Needed label Sep 29, 2020
@ebalazs-sv
Copy link

Verified as fixed on Nightly 200929 05:08 (Build #2015766697) GV 83.0a1 from 9/29 with Motorola Moto G6 (Android 8) and Nexus 5 (Android 6.0.1).
I will close this issue and remove the qa:needed label.

Android: Crash Tracking automation moved this from Crashes with STR to Closed Sep 29, 2020
@ebalazs-sv ebalazs-sv added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels Sep 29, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
b:crash Crashes Fenix: should link to Sentry, Crash-Stats or GPlay info 🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified Feature:Gesture Feature:Tabs
Projects
No open projects
Development

No branches or pull requests

5 participants