diff options
| author | dalecurtis@google.com <dalecurtis@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-11-27 01:39:21 +0000 |
|---|---|---|
| committer | dalecurtis@google.com <dalecurtis@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-11-27 01:39:21 +0000 |
| commit | f1196a4f47e6824787e84716169f5cca35fd7a83 (patch) | |
| tree | 85dc500057e72a51bb9cddb4304712168ada541e /media/audio/audio_output_controller.h | |
| parent | 228090e2a76d5cbb5f5c69fe9ee19c936188d746 (diff) | |
| download | chromium_src-f1196a4f47e6824787e84716169f5cca35fd7a83.zip chromium_src-f1196a4f47e6824787e84716169f5cca35fd7a83.tar.gz chromium_src-f1196a4f47e6824787e84716169f5cca35fd7a83.tar.bz2 | |
Always wait for DataReady() on Windows WaveOut.
Only WaveOut still needs this hack and since all renderer clients
now use the shared memory marker we can always wait for it.
I suspect on low end machines this timeout is leading to pops and
clicks for some users on XP.
BUG=161307
TEST=audio still works.
Review URL: https://codereview.chromium.org/11348166
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@169553 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'media/audio/audio_output_controller.h')
| -rw-r--r-- | media/audio/audio_output_controller.h | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/media/audio/audio_output_controller.h b/media/audio/audio_output_controller.h index 3dae719..762a948 100644 --- a/media/audio/audio_output_controller.h +++ b/media/audio/audio_output_controller.h @@ -102,11 +102,7 @@ class MEDIA_EXPORT AudioOutputController // Close this synchronous reader. virtual void Close() = 0; - // Poll if data is ready. - // Not reliable, as there is no guarantee that renderer is "new-style" - // renderer that writes metadata into buffer. After several unsuccessful - // attempts caller should assume the data is ready even if that function - // returns false. + // Check if data is ready. virtual bool DataReady() = 0; }; |