summaryrefslogtreecommitdiffstats
path: root/media/audio/audio_output_dispatcher_impl.cc
Commit message (Collapse)AuthorAgeFilesLines
* Use microseconds instead of milliseconds for integer audio math.dalecurtis@google.com2013-02-221-3/+3
| | | | | | | | | | | | When dealing with values where +/- 1ms is an enormous amount of error, we should be using microseconds. BUG=none TEST=none Review URL: https://codereview.chromium.org/12330038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@183978 0039d316-1c4b-4281-b951-d872f2087c98
* Don't fallback if we've successfully opened a stream previously.dalecurtis@google.com2012-09-211-6/+4
| | | | | | | | | | | | | | | | | | | | | | | | | Fixes an issue where we've opened streams through the AudioOutputDispatcher successfully, but eventually fail to open a stream. At this point the old code would try to fallback to the high latency audio path and in the process destroy the old AudioOutputDispatcher during Initialize()... leaving a bunch of physical audio streams in the ether calling OnMoreDataResampler callbacks which no longer have a way to stop the associated stream before being deleted. This also explains the "double-Stop()" errors from issue 149815, since they were not double-Stop() but rather Stop() on an AudioOutputProxy that the new dispatcher didn't know about. Rolls in the following fixes as well: - Replaces lock with message_loop_ check. - Fixes an issue where physical streams were incorrectly closed on Open failure. - Rollback of previous double-Stop() and CHECK() changes. BUG=150619 TEST=new media unittest. Review URL: https://codereview.chromium.org/10958020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157940 0039d316-1c4b-4281-b951-d872f2087c98
* Force audio stream Stop() before Close().dalecurtis@google.com2012-09-191-5/+3
| | | | | | | | | | | | Bandaid over situations where we are seemingly calling Close() without calling Stop() beforehand. BUG=150619 TEST=media_unittests Review URL: https://codereview.chromium.org/10950033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157652 0039d316-1c4b-4281-b951-d872f2087c98
* Allow StopStream() to be called multiple times.dalecurtis@google.com2012-09-171-1/+7
| | | | | | | | | BUG=149815 TEST=Stop() twice no longer crashes. Review URL: https://codereview.chromium.org/10916342 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157161 0039d316-1c4b-4281-b951-d872f2087c98
* Re-land software audio mixer.enal@chromium.org2012-04-191-0/+200
| | | | | | | | | | | Code goes through old or new paths depending on the AudioParameters or command-line flag. Added unit tests. Changed suppression for http://code.google.com/p/chromium/issues/detail?id=123112 because call stack changed (class was split into 2). Review URL: http://codereview.chromium.org/9691001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@133010 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 132891 - Software audio mixer.thestig@chromium.org2012-04-191-200/+0
| | | | | | | | | | | | Code goes through old or new paths depending on the AudioParameters or command-line flag. Added unit tests. Review URL: http://codereview.chromium.org/9691001 TBR=enal@chromium.org Review URL: https://chromiumcodereview.appspot.com/10127002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@132929 0039d316-1c4b-4281-b951-d872f2087c98
* Software audio mixer.enal@chromium.org2012-04-181-0/+200
Code goes through old or new paths depending on the AudioParameters or command-line flag. Added unit tests. Review URL: http://codereview.chromium.org/9691001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@132891 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-02-13 18:37:49 +0000