search

mozilla / MozStumbler

Android Stumbler for Mozilla
http://location.services.mozilla.com
Mozilla Public License 2.0
621 stars 214 forks source link

Consider replacing java.util.timer* with android Handler class #1832

Open mcomella opened 6 years ago

mcomella commented 6 years ago

From bugzilla https://bugzilla.mozilla.org/show_bug.cgi?id=1268612

The Timer class creates a separate thread, which can affect the performance of resource-constrained mobile devices. The Android Handler class can be used to synchronize such events with separate threads, such as the main thread. See 1 for an example implementation.

This bug is specifically for the mozstumbler (I'll file another for the main code base). For a list of uses of the timer class:

https://mxr.mozilla.org/mozilla-central/search?string=java.util.timer&find=mobile%2Fandroid&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central

Victor, can you take a look at this?

abhishekd08 commented 5 years ago

Can I work on this ?

mcomella commented 5 years ago

@abhishekd08 I don't think anyone is working on this code base. If you're looking to work on an Android project at Mozilla, may I recommend Focus for Android?

abhishekd08 commented 5 years ago

I am beginner in open source development, I am more than happy to work on any project. I will look into "Focus for Android" suggested by you. Thank you.