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

[Bug] Crash in BookmarkFragment #2474

Closed
AaronMT opened this issue May 14, 2019 · 6 comments
Closed

[Bug] Crash in BookmarkFragment #2474

AaronMT opened this issue May 14, 2019 · 6 comments
Assignees
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 P1 Current sprint

Comments

@AaronMT
Copy link
Contributor

AaronMT commented May 14, 2019

  • Build: 917399d
  • Device: Pixel 2 (Android 9)
  1. From the Library, open a bookmark in a new private tab
  2. Tap the 'All Tabs' button on the address bar

(CC @colintheshots @sblatz)

2019-05-14 11:28:30.256 11585-11585/org.mozilla.fenix.debug E/AndroidRuntime: FATAL EXCEPTION: main
    Process: org.mozilla.fenix.debug, PID: 11585
    java.lang.IllegalStateException: Fragment BookmarkFragment{b10d5fb (72dbc11f-f8ca-479e-85b1-063963453973) id=0x7f0800e1} did not return a View from onCreateView() or this was called before onCreateView().
        at androidx.fragment.app.Fragment.requireView(Fragment.java:1708)
        at org.mozilla.fenix.library.bookmarks.BookmarkFragment$navigation$2.invoke(BookmarkFragment.kt:62)
        at org.mozilla.fenix.library.bookmarks.BookmarkFragment$navigation$2.invoke(BookmarkFragment.kt:56)
        at kotlin.SynchronizedLazyImpl.getValue(LazyJVM.kt:74)
        at org.mozilla.fenix.library.bookmarks.BookmarkFragment.getNavigation(Unknown Source:25)
        at org.mozilla.fenix.library.bookmarks.BookmarkFragment.onDestroy(BookmarkFragment.kt:123)
        at androidx.fragment.app.Fragment.performDestroy(Fragment.java:2873)
        at androidx.fragment.app.FragmentManagerImpl.moveToState(FragmentManagerImpl.java:1018)
        at androidx.fragment.app.FragmentManagerImpl.moveFragmentToExpectedState(FragmentManagerImpl.java:1228)
        at androidx.fragment.app.FragmentManagerImpl.moveToState(FragmentManagerImpl.java:1300)
        at androidx.fragment.app.FragmentManagerImpl.executeOpsTogether(FragmentManagerImpl.java:1871)
        at androidx.fragment.app.FragmentManagerImpl.removeRedundantOperationsAndExecute(FragmentManagerImpl.java:1817)
        at androidx.fragment.app.FragmentManagerImpl.execPendingActions(FragmentManagerImpl.java:1717)
        at androidx.fragment.app.FragmentManagerImpl$2.run(FragmentManagerImpl.java:150)
        at android.os.Handler.handleCallback(Handler.java:873)
        at android.os.Handler.dispatchMessage(Handler.java:99)
        at android.os.Looper.loop(Looper.java:193)
        at android.app.ActivityThread.main(ActivityThread.java:6718)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:493)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:858)

┆Issue is synchronized with this [Jira Task](https://mozilla-hub.atlassian.net/browse/FNXV2-11448)
@AaronMT AaronMT added 🐞 bug Crashes, Something isn't working, .. b:crash Crashes Fenix: should link to Sentry, Crash-Stats or GPlay info labels May 14, 2019
@sblatz
Copy link
Contributor

sblatz commented May 15, 2019

Should be fixed by #2499

@sblatz sblatz added the eng:qa:needed QA Needed label May 15, 2019
@lobontiumira
Copy link

lobontiumira commented May 17, 2019

@sblatz Hi,
The crash still reproduces. Tested with Samsung Galaxy S8 (Android 9), HTC Desire 820 (Android 6.0.1), HTC 10 (Android 8), and Sony Xperia Z5 Premium (Android 7.1.1).

@sv-ohorvath sv-ohorvath removed the eng:qa:needed QA Needed label May 17, 2019
@vesta0 vesta0 added the P1 Current sprint label May 18, 2019
@vesta0 vesta0 added this to the Sprint 5-2 (Milestone 5) milestone May 18, 2019
@vesta0 vesta0 added this to High priority backlog (not in the current sprint) in Fenix Sprint Kanban May 18, 2019
@ekager ekager self-assigned this May 20, 2019
@ekager ekager moved this from High priority backlog (not in the current sprint) to In Progress in Fenix Sprint Kanban May 20, 2019
ekager added a commit to ekager/fenix that referenced this issue May 20, 2019
@boek boek moved this from In Progress to Ready for QA in Fenix Sprint Kanban May 21, 2019
@boek boek added the eng:qa:needed QA Needed label May 21, 2019
@ttxhhh
Copy link

ttxhhh commented May 21, 2019

在HUAWEI HONOR 8上也有这个问题(系统为EMUI8.0.0基于Android 8.0.0)

@lobontiumira
Copy link

Testing is blocked by #2682.

@sv-ohorvath
Copy link
Contributor

sv-ohorvath commented May 22, 2019

@boek, @ekager This is still reproducing on the 5/22 build, with the same steps.

@sv-ohorvath sv-ohorvath removed the eng:qa:needed QA Needed label May 22, 2019
@lobontiumira
Copy link

Verified as fixed with the latest build 5/23, with:

  • Nexus 6P (Android 8.1.0),
  • Sony Xperia Z2 tablet (Android 6.0.1),
  • Huawei P9 Lite (Android 6),
  • OnePlus 5T (Android 9),
  • HTC Desire 820 (Android 6.0.1).

@lobontiumira lobontiumira added the eng:qa:verified QA Verified label May 23, 2019
@lobontiumira lobontiumira moved this from Ready for QA to Done in Fenix Sprint Kanban May 23, 2019
@data-sync-user data-sync-user changed the title [Bug] Crash in BookmarkFragment FNX2-18022 ⁃ [Bug] Crash in BookmarkFragment Aug 5, 2020
@data-sync-user data-sync-user changed the title FNX2-18022 ⁃ [Bug] Crash in BookmarkFragment FNX3-16349 ⁃ [Bug] Crash in BookmarkFragment Aug 11, 2020
@data-sync-user data-sync-user changed the title FNX3-16349 ⁃ [Bug] Crash in BookmarkFragment [Bug] Crash in BookmarkFragment May 20, 2022
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 P1 Current sprint
Projects
None yet
Development

No branches or pull requests

10 participants