Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(457)

Unified Diff: media/filters/audio_renderer_algorithm.cc

Issue 10389138: Remove media::AudioRendererImpl::SignalEndOfStream() and some other minor cleanup. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src
Patch Set: canfillbuffer Created 8 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/filters/audio_renderer_algorithm.cc
diff --git a/media/filters/audio_renderer_algorithm.cc b/media/filters/audio_renderer_algorithm.cc
index 8004339c8269fdc0906fcf23cd57497ac0edcf98..030c1aceaf95c3da6a7f4432fec9b68b6eb2135f 100644
--- a/media/filters/audio_renderer_algorithm.cc
+++ b/media/filters/audio_renderer_algorithm.cc
@@ -420,12 +420,8 @@ void AudioRendererAlgorithm::EnqueueBuffer(Buffer* buffer_in) {
request_read_cb_.Run();
}
-bool AudioRendererAlgorithm::NeedsMoreData() {
- return needs_more_data_ || IsQueueEmpty();
-}
-
-bool AudioRendererAlgorithm::IsQueueEmpty() {
- return audio_buffer_.forward_bytes() == 0;
+bool AudioRendererAlgorithm::CanFillBuffer() {
+ return audio_buffer_.forward_bytes() > 0 && !needs_more_data_;
}
bool AudioRendererAlgorithm::IsQueueFull() {

Powered by Google App Engine
This is Rietveld 408576698