Quickly switching between browsing sessions and opening a new tab will cause Fennec to crash

VERIFIED FIXED

Status

()

Firefox for iOS
General
P1
critical
VERIFIED FIXED
a year ago
9 months ago

People

(Reporter: SimonB, Assigned: farhan)

Tracking

(Blocks: 1 bug, {crash, reproducible})

unspecified
All
iOS
crash, reproducible

Firefox Tracking Flags

(fxios7.0+, fxios-v7.0 affected)

Details

(Whiteboard: [MobileCore])

Attachments

(3 attachments)

(Reporter)

Description

a year ago
Created attachment 8808094 [details]
Client  07-11-16 11-10.crash

Build: 1bc8bb71f
Device: 
iPad Pro (9.3.2)
iPad Air 2 (10.0.3)

Steps to reproduce"
1. Launch Fennec
2. Quickly tap on 'Private Browsing' button
3. While switching between browsing session tap the 'New tab' button
4. Repeat steps 2 and 3

Actual results:
- Fennec will crash. I noticed that when opening a new tab(Tapping on the + button) the privacy toggle button is disabled, but when tapping quickly on this button (switching between browsing sessions) the 'Open new tab' button is still enabled which will cause the crash to occur. 

Expected:
- A solution might be to disable the 'New tab' button when the 'privacy toggle button' is animated and re-enable the button after the animation occurs. 

See video of the issue: https://www.youtube.com/watch?v=clubVyTfQMQ
tracking-fxios: ? → 7.0+
Priority: -- → P2
Whiteboard: [MobileAS]
Whiteboard: [MobileAS] → [MobileCore]
(Assignee)

Updated

10 months ago
Blocks: 1305800
(Assignee)

Comment 1

10 months ago
master https://github.com/mozilla-mobile/firefox-ios/commit/51c7e1dbd2ee4ee0463ebf5e9867454a69840412
Assignee: nobody → fpatel
Status: NEW → RESOLVED
Iteration: --- → 1.13
Last Resolved: 10 months ago
Priority: P2 → P1
Resolution: --- → FIXED
(Reporter)

Comment 2

10 months ago
Created attachment 8830364 [details]
I have tested this on latest Nightly 51abcc7ca and I can still reproduce this issue following the STR from the description. This crash log looks similar with Bug 1303322. Is this a duplicate issue?
Flags: needinfo?(fpatel)
(Assignee)

Comment 3

9 months ago
Created attachment 8838807 [details] [review]
Pull Request

I've got a new PR to fix this. :crossesfingers:
Flags: needinfo?(fpatel)
Attachment #8838807 - Flags: review?(etoop)
(Assignee)

Updated

9 months ago
Attachment #8838807 - Flags: review?(etoop)
(Reporter)

Comment 4

9 months ago
Verifying as fix on master d0149e93.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.