From 8f5fc718853a128dc2f70ec4e55db4044af64bcc Mon Sep 17 00:00:00 2001 From: "tommi@chromium.org" Date: Thu, 19 Jan 2012 09:16:58 +0000 Subject: Add thread safety to AudioManagerBase to protect access to the audio thread member variable. This is is a tentative fix for an issue where the AudioManager can crash while tearing down the audio thread. I suspect that this happens because more than one cleanup attempts were made since the class wasn't thread safe. I'm also changing direct access to the thread's MessageLoop* to use MessageLoopProxy based on the same theory. BUG=110051 TEST=Run media tests. Review URL: https://chromiumcodereview.appspot.com/9255017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@118272 0039d316-1c4b-4281-b951-d872f2087c98 --- media/audio/audio_output_controller.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'media/audio/audio_output_controller.h') diff --git a/media/audio/audio_output_controller.h b/media/audio/audio_output_controller.h index 24de175..f6e5722 100644 --- a/media/audio/audio_output_controller.h +++ b/media/audio/audio_output_controller.h @@ -254,7 +254,7 @@ class MEDIA_EXPORT AudioOutputController SyncReader* sync_reader_; // The message loop of audio thread that this object runs on. - MessageLoop* message_loop_; + scoped_refptr message_loop_; // When starting stream we wait for data to become available. // Number of times left. -- cgit v1.1