When the activity is destroyed, make sure any background thread is cancelled.
There was a race condition when the activity was destroyed when the background thread was still running. Before we release the handler, cancel any running thread, wait for it to actually exit before purging the handler. This code should be rewritten to use AsyncTask instead of raw threads in my opinion. Tested: I tried to import +100 vcard and make the app exit and it did not crash however the current bug will show under high load. Need to wait for a monkey run to confirm it is fixed. Bug:2517391 Change-Id: I37e7619f6b93a8faa7da4506caffb9ed40e4f844
Loading
Please register or sign in to comment