1306592 - Crash in java.lang.IllegalArgumentException: Sanitize:Finished was not registered at org.mozilla.gecko.EventDispatcher.unregisterListener(EventDispatcher.java)

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Alex Caziuc]

This bug was filed from the Socorro interface and is 
report bp-47f15ccf-2c6f-484f-825e-876ac2160930.
=============================================================

I was able to reproduce this on Samsung Galaxy Note 5(6.0.1) and Samsung Galaxy S6 edge(6.0.1) on Nightly(09.29.2016).

STR:

1.Open FF -> Settings -> Mozilla Nightly -> About Nightly

Expected result:
About Nightly page appears

Actual result:
FF crashes

Note: On Nexus 6P(7.0), Sony Xperia Z2(5.1.1), Asus ZenPad 8(Android 5.0.2) works fine.

Comment 1

[Alex Caziuc]

regression window

good build: 2016-09-28
bad build: 2016-09-29

http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=66a77b9bfe5dcacd50eccf85de7c0e7e15ce0ffd&tochange=f7d5008ee2ab9200052e45ad6ecc3f3a348f7f86

Comment 2

[(inactive) Jim Chen [:jchen] [:darchons]]

Probably bug 1294821.

Comment 3

[Dylan Roeh (:droeh) (he/him)]

Attachment: Use singleton EventDispatcher

I think GeckoPreferences and RecentTabsAdapter should both be using the singleton EventDispatcher. This should fix this crash as well as bug 1306742.

Comment 4

[Dylan Roeh (:droeh) (he/him)]

Comment on attachment 8797776 [details] [diff] [review]
Use singleton EventDispatcher

(Forgot to flag this for review)

Comment 5

[Dylan Roeh (:droeh) (he/him)]

Landed under bug 1306742 by mistake, but it doesn't matter as the patch fixes both that bug and this one.

Comment 6

[Julien Cristau [:jcristau]]

marking fixed in 52 per comment 5