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

NEW
Unassigned

Status

()

Firefox for Android
General
9 months ago
9 months ago

People

(Reporter: JanH, Unassigned)

Tracking

(Blocks: 1 bug)

Trunk
All
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

9 months 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

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

Comment 2

9 months 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@buttercookie.de
Flags: needinfo?(jh+bugzilla)
Summary: Remove AsyncTask initialisation workaround → Remove AsyncTask initialisation workaround once we're building with minSdk >= 16
You need to log in before you can comment on or make changes to this bug.