1382328 - Crash on "Close all tabs" on the iPad

Status: VERIFIED FIXED

(Firefox for iOS :: Menu and Toolbar, defect, P1)

Description

[ugarlapati]

Issue is related to iPad

1. Open the app
2. Open more than 2 tabs
3. Go to tabs screen
4. Click on menu button
5. Click on Close All Tabs button

Noticed App crashes.

This issue occurs only in iPad.

Tested on iPadAir 2 simulator
Reproducible: 10/10

Comment 1

[No-Jun Park [:njpark]]

Stack Trace:
    2017-07-19 11:24:54.449 Client[81360:1760671] *** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'attempt to delete item 2 from section 0 which only contains 1 items before the update'
    *** First throw call stack:
    (
        0   CoreFoundation                      0x000000010de5cb0b __exceptionPreprocess + 171
        1   libobjc.A.dylib                     0x000000010d21a141 objc_exception_throw + 48
        2   CoreFoundation                      0x000000010de60cf2 +[NSException raise:format:arguments:] + 98
        3   Foundation                          0x000000010cdb4536 -[NSAssertionHandler handleFailureInMethod:object:file:lineNumber:description:] + 193
        4   UIKit                               0x000000010fb41f80 -[UICollectionView _endItemAnimationsWithInvalidationContext:tentativelyForReordering:animator:] + 5843
        5   UIKit                               0x000000010fb406ce -[UICollectionView _updateRowsAtIndexPaths:updateAction:] + 372
        6   UIKit                               0x0000000126f2b5f1 -[UICollectionViewAccessibility reloadItemsAtIndexPaths:] + 42
        7   Client                              0x000000010965985c _TFFC6Client21TopTabsViewControllerP33_3434E9307FC2B0225EF0EAC66FE8101010reloadDataFT_T_U_FT_T_ + 220
        8   Client                              0x00000001095cca17 _TTRXFo___XFdCb___ + 39
        9   UIKit                               0x000000010f2b13da +[UIView(UIViewAnimationWithBlocks) _setupAnimationWithDuration:delay:view:options:factory:animations:start:animationStateGenerator:completion:] + 572
        10  UIKit                               0x000000010f2b194c +[UIView(UIViewAnimationWithBlocks) animateWithDuration:animations:completion:] + 86
        11  Client                              0x000000010964f526 _TFC6Client21TopTabsViewControllerP33_3434E9307FC2B0225EF0EAC66FE8101010reloadDatafT_T_ + 1158
        12  Client                              0x0000000109659098 _TFFC6Client21TopTabsViewController14updateTabsFromFTGSqGSaCS_3Tab__2toGSaS1__2onGSqFT_T___T_U2_FSbT_ + 152
        13  Client                              0x00000001095d16d3 _TTRXFo_dSb__XFdCb_dSb__ + 51
        14  Foundation                          0x000000010cd65328 __NSFireDelayedPerform + 409
        15  CoreFoundation                      0x000000010ddf0454 __CFRUNLOOP_IS_CALLING_OUT_TO_A_TIMER_CALLBACK_FUNCTION__ + 20
        16  CoreFoundation                      0x000000010ddf00e3 __CFRunLoopDoTimer + 1011
        17  CoreFoundation                      0x000000010ddefcaa __CFRunLoopDoTimers + 266
        18  CoreFoundation                      0x000000010dde79ec __CFRunLoopRun + 1916
        19  CoreFoundation                      0x000000010dde7016 CFRunLoopRunSpecific + 406
        20  GraphicsServices                    0x0000000113e43a24 GSEventRunModal + 62
        21  UIKit                               0x000000010f1f8134 UIApplicationMain + 159
        22  Client                              0x000000010996154b main + 443
        23  libdyld.dylib                       0x00000001120f265d start + 1
        24  ???                                 0x0000000000000003 0x0 + 3
    )
    libc++abi.dylib: terminating with uncaught exception of type NSException
    (lldb)

Comment 2

[Justin D'Arcangelo [:justindarc]]

Just verified in the simulator that this is a regression from the patch that landed in Bug 1346820.

Comment 3

[Farhan Patel [:farhan]]

Attachment: Pull Request

Comment 4

[Farhan Patel [:farhan]]

master https://github.com/mozilla-mobile/firefox-ios/commit/44f653078b05c6bee6a3dfe7da16fa82fbea79be

Comment 5

[Simon Basca [:SimonB]]

Verifying as fix on master 35383ea5.

Comment 6

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

Uplifted to v8.x