DescriptionSimplify shutdown of AudioDevice's audio thread.
This is a followup change for bug 107933 (see code review 9112029) that does the following:
* Makes Stop() asynchronous.
* Simplifies shutdown of AudioDevice's audio thread by always doing it from the IO thread.
* Simplifies access to of most member variables (added documentation).
* We no longer need synchronisation with the IO thread when shutting down the audio thread since we'll always shut it down from the IO thread.
* We don't need a lock to guard the audio_thread_ variable (and others related ones), since they're only modifed on the IO thread when the audio thread is not running.
I checked the implementations of AudioDevice::RenderCallback if this is OK and it should be fine.
BUG=107933
TEST=Run media and content tests
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=118810
Patch Set 1 #
Total comments: 2
Patch Set 2 : Removed incorrect call to ShutDownAudioThread from OnLowLatencyCreated #
Messages
Total messages: 4 (0 generated)
|