Open Bug 865060 Opened 7 years ago Updated 4 years ago

Clean up threads in AboutHome views

Categories

(Firefox for Android :: General, defect)

ARM
Android
defect
Not set

Tracking

()

REOPENED
Firefox 23

People

(Reporter: bnicholson, Unassigned)

References

Details

Attachments

(1 file)

Our AboutHome views all call getBackgroundHandler().post(), then nest UI posts inside these runnables. We should use AsyncTasks to clean this up.
Attachment #741096 - Flags: review?(lucasr.at.mozilla)
Attachment #741096 - Flags: review?(lucasr.at.mozilla) → review+
https://hg.mozilla.org/mozilla-central/rev/7190f15f3f68
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 23
Depends on: 866150
Backed out to see if it fixes bug 866150: https://bugzilla.mozilla.org/show_bug.cgi?id=866150#c8
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(In reply to Brian Nicholson (:bnicholson) from comment #4)
> Backed out to see if it fixes bug 866150:
> https://bugzilla.mozilla.org/show_bug.cgi?id=866150#c8

FYI, only backed out the changes to TopSitesView, so the rest of this patch is still landed. It may be easier to just file a separate bug for trying to re-land that last piece.
Assignee: bnicholson → nobody
You need to log in before you can comment on or make changes to this bug.