Closed Bug 1344543 Opened 3 years ago Closed 2 years ago

Remove DownloadUtils non-Intl API fallbacks

Categories

(Toolkit :: Downloads API, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla56
Tracking Status
firefox56 --- fixed

People

(Reporter: JanH, Assigned: JanH)

References

Details

Attachments

(1 file)

These were already removed in bug 1200494, but since then it was decided that for the time being the Intl API will be Nightly/Aurora-only on Android and these changes have been backed out again in bug 1344143.

Once the Intl API is definitively being enabled everywhere, we can remove them once more.
Do we have a tracking bug for when the Intl API is going to be fully re-enabled on Android (and the build-time option to turn it on/off removed)?
Flags: needinfo?(m_kato)
See Also: → 1344143
Depends on: 1344625
(In reply to Jan Henning [:JanH] from comment #1)
> Do we have a tracking bug for when the Intl API is going to be fully
> re-enabled on Android (and the build-time option to turn it on/off removed)?

I filed as bug 1344625
Flags: needinfo?(m_kato)
Priority: -- → P3
Once more unto the breach...
Assignee: nobody → jh+bugzilla
Comment on attachment 8888188 [details]
Bug 1344543 - Remove Intl API fallbacks from DownloadUtils.jsm now that Fennec no longer requires it.

Stealing the review. Thanks for cleaning up the code!
Attachment #8888188 - Flags: review?(mak77) → review+
Thanks, but can you mark it r+ in mozreview as well, so as to unblock autoland?
Flags: needinfo?(paolo.mozmail)
Comment on attachment 8888188 [details]
Bug 1344543 - Remove Intl API fallbacks from DownloadUtils.jsm now that Fennec no longer requires it.

https://reviewboard.mozilla.org/r/158904/#review164540
Done!
Flags: needinfo?(paolo.mozmail)
Pushed by mozilla@buttercookie.de:
https://hg.mozilla.org/integration/autoland/rev/50fd0f26c2df
Remove Intl API fallbacks from DownloadUtils.jsm now that Fennec no longer requires it. r=Paolo
https://hg.mozilla.org/mozilla-central/rev/50fd0f26c2df
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.