tree ca6455cc818fcf91c9da3bddacd2c710f0b2f0dc
parent c4df102e74e16dbe8f83a039fa6324b9d612c96f
author Bo Liu <boliu@google.com> 1393280135 -0800
committer Bo Liu <boliu@google.com> 1393280135 -0800

Fork: Fix deadlock in AudioManagerAndroid.init

SettingsObserverThread is started by UI thread, and UI thread waits for
it to start by waiting on a lock. However the thread is started when the
lock is not held, meaning it could start and signal the lock, all before
UI thread even enters the lock synchronized code, thus causing the wait
to simply deadlock.

Fix by moving start to lock synchronized code.

Note that Chrome m33 is going to ship with this bug unfixed. Chromium
trunk already has this fixed. But it was only inadvertently fixed as
part of a larger refactor, so not appropriate to cherry-pick.

BUG: 13131862
Change-Id: I7b758d39e6a23ac762c937271e67a3718dafcbf2