mozilla-mobile / prox

[INACTIVE] A search and discovery app for the "here & now". We're experimenting with ideas on mobile that can better surface content from the open web.
https://wiki.mozilla.org/New_Mobile_Experience
Mozilla Public License 2.0
22 stars 12 forks source link

No issue: assert Toast.show is called on main thread. #638

Closed mcomella closed 7 years ago

mcomella commented 7 years ago

I experienced Exceptions otherwise.

mcomella commented 7 years ago

Test code! I added a toast when we re-sorted places (so I could walk around and see when they're updated) which is called from the background thread.

Side note: I'm going to wait to land this until after we cut the final Chicago build so we don't crash on something that may not otherwise cause crashes.

thebnich commented 7 years ago

Note that asserts are only enabled in debug builds, so that shouldn't be a problem.