991864 - Use SimpleArrayMap/ArrayMap in place of HashMap where appropriate

Status: RESOLVED INCOMPLETE

(Firefox for Android Graveyard :: General, defect)

Description

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

See jchen's mobile-firefox-dev post [3], copied below for convenience:

With the latest v4 support library, we can also start using
SimpleArrayMap/ArrayMap for some of the places where we've been using
HashMap. SimpleArrayMap [1] and ArrayMap [2] have better memory
characteristics than HashMap for a small perf tradeoff.

[1]
http://developer.android.com/reference/android/support/v4/util/SimpleArrayMap.html
[2]
http://developer.android.com/reference/android/support/v4/util/ArrayMap.html

---
Awaiting bug 990161, the support library update, to land, of course.

[3]: https://mail.mozilla.org/pipermail/mobile-firefox-dev/2014-March/000584.html

Comment 1

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

;)

Comment 2

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

via bug 990161 comment 12:

> Because we are lifting the source for SwipeRefreshLayout,
> we do not need to upgrade the support library anymore.

Guess this is infeasible - closing.

Comment 3

[(inactive) Jim Chen [:jchen] [:darchons]]

We'll want to work on this when the support library gets upgraded eventually.

Comment 4

[BMO Automation]

We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.