Closed Bug 1196276 Opened 5 years ago Closed 5 years ago

Crash when switching between home panels rapidly

Categories

(Firefox for iOS :: Home screen, defect)

All
iOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
fxios + ---

People

(Reporter: sleroux, Assigned: sleroux)

Details

(Keywords: crash)

Attachments

(1 file)

STR:

1. Tap URL bar to bring up Home panels
2. Start tapping between the different home panels

Expected:

App should display each panel when tapped

Actual:

App will crash after a short time

-- 

Here is a partially symbolicated crash from crash-stats:

https://crash-stats.mozilla.com/report/index/798d3053-872b-4d47-b266-222d62150819

I'm currently in the process of trying to get the UIKit framework symbols loaded as well.
Note: This occurred in build #36 Aurora
How often is this reproducible?
100% of the time when switching between panels quick. I might have narrowed it down. The issue as introduced between builds 34->35. I'm going to take this to see if there is a quick fix for 1.0 because crashes suck.
Assignee: nobody → sleroux
Couple of things:

1. Safer target/action. I was able to get crashes from the selector being called on an invalid/deallocated target. I've made the add/removal safer by binding them to the willAppear/didDisappear events.

2. Added checks to make sure we only call endRefreshing if we are refreshing.

3. Renamed the refresh method and target selector. Looking at https://crash-stats.mozilla.com/report/index/a04d4cb7-038e-435a-a203-ee0752150819#allthreads it seems that the crash is happening inside UIKit and might be a naming collision with 'refresh' considering it's a popular name for a method. Added just in case.

I haven't been able to reproduce the crash with these fixes
Attachment #8650087 - Flags: review?(rnewman)
Attachment #8650087 - Flags: review?(rnewman) → review+
Status: NEW → ASSIGNED
Keywords: crash
Hardware: Other → All
Merged
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.