Remove AsyncTask initialisation workaround once we're building with minSdk >= 16

NEW
Unassigned

Status

()

P5
normal
2 years ago
4 months ago

People

(Reporter: JanH, Unassigned)

Tracking

(Blocks: 1 bug)

Trunk
All
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

2 years ago
As of API 16, this (https://dxr.mozilla.org/mozilla-central/rev/f81bcc23d37d7bec48f08b19a9327e93c54d37b5/mobile/android/base/java/org/mozilla/gecko/GeckoApp.java#1280) is no longer needed.

https://developer.android.com/reference/android/os/AsyncTask.html says:
> The AsyncTask class must be loaded on the UI thread. This is done automatically as of JELLY_BEAN.

Comment 1

2 years ago
Hello Jan!
Should I remove the try,catch block of AsyncTask and submit a patch?
Flags: needinfo?(jh+bugzilla)
(Reporter)

Comment 2

2 years ago
Sorry for not making this clearer in the bug description, but this bug becomes relevant only when we stop supporting API level 15, since the underlying Android bug this is working around has been fixed only starting from API 16.

You can look at bug 1363843 for something similar if you want.
Mentor: jh+bugzilla
Flags: needinfo?(jh+bugzilla)
Summary: Remove AsyncTask initialisation workaround → Remove AsyncTask initialisation workaround once we're building with minSdk >= 16
Re-triaging per https://bugzilla.mozilla.org/show_bug.cgi?id=1473195

Needinfo :susheel if you think this bug should be re-triaged.
Priority: -- → P5
You need to log in before you can comment on or make changes to this bug.