960202 - crash in java.lang.IllegalStateException: Fragment already added: PinSiteDialog{$HEX_ADDRESS #$ pin_site} at android.support.v4.app.FragmentManagerImpl.addFragment(FragmentManager.java)

Status: RESOLVED WONTFIX

(Firefox for Android Graveyard :: General, defect, P5)

Description

[Kevin Brosnan [Ex-Mozilla]]

This bug was filed from the Socorro interface and is 
report bp-ee0f863a-7606-49cc-87a6-e7a0e2140114.
=============================================================

java.lang.IllegalStateException: Fragment already added: PinSiteDialog{42c73d40 #4 pin_site}
	at android.support.v4.app.FragmentManagerImpl.addFragment(FragmentManager.java:1159)
	at android.support.v4.app.BackStackRecord.run(BackStackRecord.java:616)
	at android.support.v4.app.FragmentManagerImpl.execPendingActions(FragmentManager.java:1444)
	at android.support.v4.app.FragmentManagerImpl$1.run(FragmentManager.java:429)
	at android.os.Handler.handleCallback(Handler.java:733)
	at android.os.Handler.dispatchMessage(Handler.java:95)
	at android.os.Looper.loop(Looper.java:136)
	at android.app.ActivityThread.main(ActivityThread.java:5017)
	at java.lang.reflect.Method.invokeNative(Native Method)
	at java.lang.reflect.Method.invoke(Method.java:515)
	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:779)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:595)
	at dalvik.system.NativeStart.main(Native Method)

This crash has two parameters that change (memory address and pin_site #) so it will never show up as a top crash but accounts for 400 or so crashes a week.

Comment 1

[Lucas Rocha (:lucasr)]

My patches in bug 935542 might fix this bug.

Comment 2

[Aaron Train [:aaronmt]]

Doesn't look like. We hit this in AppThwack through UI fuzzing.

Comment 3

[Lucas Rocha (:lucasr)]

(In reply to Aaron Train [:aaronmt] from comment #2)
> Doesn't look like. We hit this in AppThwack through UI fuzzing.

Aaron, any chance you can post a recent stacktrace for this bug? Just to make the it matches latest code in m-c.

Comment 4

[Aaron Train [:aaronmt]]

:( I missed this NI. I'll try out today.

Comment 5

[Aaron Train [:aaronmt]]

Attachment: logcat (Aurora 30)

Log-cat from a recent Aurora (30) AppThwack run on an Xperia tablet

Comment 6

[Kevin Brosnan [Ex-Mozilla]]

I have looked at this and it is in the #40 to #60 range for release. The following search is useful it contains a few breakdowns of the crash into crashes per Fx version, per Android version, phone model, phone brand. Click the tabs at the top of the search columns to see the data.

https://crash-stats.mozilla.com/search/?java_stack_trace=java.lang.IllegalStateException%3A+Fragment&_facets=android_brand&_facets=version&_columns=android_device&_columns=android_version

Search that checks for current instances of this signature.

https://crash-stats.mozilla.com/query/?product=FennecAndroid&version=ALL%3AALL&range_value=1&range_unit=weeks&query_search=signature&query_type=contains&query=java.lang.IllegalStateException%3A+Fragment&reason=&release_channels=&build_id=&process_type=any&hang_type=any

Comment 7

[Aaron Train [:aaronmt]]

Looks like this is still happening

Comment 8

[Brad Lassey [:blassey] (use needinfo?)]

filter on [mass-p5]

Comment 9

[Sylvestre Ledru [:Sylvestre]]

Closing because no crash reported since 12 weeks.

Comment 10

[Sylvestre Ledru [:Sylvestre]]

Closing because no crash reported since 12 weeks.