902060 - GeckoProfile shouldn't be cached on the Activity

Status: RESOLVED INCOMPLETE

(Firefox for Android Graveyard :: General, defect)

Description

[Wesley Johnston (:wesj)]

We currently store a cached GeckoProfile for the current profile on GeckoApp. GeckoApp's lifecycle and Gecko's lifecycle aren't the same though, meaning this profile can be destroyed while Gecko lives on.

We should cache GeckoProfile on the Application instead.

Comment 2

[Brian Nicholson (:bnicholson)]

Are you sure this is a dupe? I thought this bug was about the fact that we hold onto the profile in GeckoApp and then reference it elsewhere (http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/GeckoProfile.java#72). Bug 937945 won't fix this.

Comment 3

[Wesley Johnston (:wesj)]

Hmm... you're right. I know a filed a bug for 937945, and thought from the title this was it :)

Comment 4

[BMO Automation]

We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.