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

Unified Diff: Source/core/platform/audio/AudioPullFIFO.cpp

Issue 14628008: Require use of AudioBus::create() to avoid ref-counting issues (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Require use of Created 7 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
« no previous file with comments | « Source/core/platform/audio/AudioPullFIFO.h ('k') | Source/core/platform/audio/AudioResampler.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/platform/audio/AudioPullFIFO.cpp
diff --git a/Source/core/platform/audio/AudioPullFIFO.cpp b/Source/core/platform/audio/AudioPullFIFO.cpp
index a31730ed0a491442f6cc752d3fa8308d4a4711c9..ce8c0d99050cf67a2bbe671a15477f7f21109f38 100644
--- a/Source/core/platform/audio/AudioPullFIFO.cpp
+++ b/Source/core/platform/audio/AudioPullFIFO.cpp
@@ -38,7 +38,7 @@ AudioPullFIFO::AudioPullFIFO(AudioSourceProvider& audioProvider, unsigned number
: m_provider(audioProvider)
, m_fifo(numberOfChannels, fifoLength)
, m_providerSize(providerSize)
- , m_tempBus(numberOfChannels, providerSize)
+ , m_tempBus(AudioBus::create(numberOfChannels, providerSize))
{
}
@@ -62,9 +62,9 @@ void AudioPullFIFO::fillBuffer(size_t numberOfFrames)
size_t framesProvided = 0;
while (framesProvided < numberOfFrames) {
- m_provider.provideInput(&m_tempBus, m_providerSize);
+ m_provider.provideInput(m_tempBus.get(), m_providerSize);
- m_fifo.push(&m_tempBus);
+ m_fifo.push(m_tempBus.get());
framesProvided += m_providerSize;
}
« no previous file with comments | « Source/core/platform/audio/AudioPullFIFO.h ('k') | Source/core/platform/audio/AudioResampler.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698