Fix BrowserSearch loader lifecycle

NEW
Unassigned

Status

()

Firefox for Android
Awesomescreen
3 years ago
3 years ago

People

(Reporter: ahunt, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(firefox46 affected)

Details

(Reporter)

Description

3 years ago
Bug 1100403 (hopefully) provided a quick band-aid fix for a crasher in BrowserSearch, however we should fix the underlying lifecycle issues:

(In reply to Richard Newman [:rnewman] from comment #16)
> I'm in favor of a band-aid to avoid a NPE, but this is the symptom of a more
> significant lifecycle problem. The loader should capture its adapter, or
> otherwise untie this knot.
> 
> Please file a follow-up to do this right -- more 'final', please!

( https://bugzilla.mozilla.org/show_bug.cgi?id=1100403#c16 )
You need to log in before you can comment on or make changes to this bug.