diff options
author | Steve Block <steveblock@google.com> | 2011-05-25 19:08:45 +0100 |
---|---|---|
committer | Steve Block <steveblock@google.com> | 2011-06-08 13:51:31 +0100 |
commit | 2bde8e466a4451c7319e3a072d118917957d6554 (patch) | |
tree | 28f4a1b869a513e565c7760d0e6a06e7cf1fe95a /Source/WebCore/platform/audio/chromium/AudioBusChromium.cpp | |
parent | 6939c99b71d9372d14a0c74a772108052e8c48c8 (diff) | |
download | external_webkit-2bde8e466a4451c7319e3a072d118917957d6554.zip external_webkit-2bde8e466a4451c7319e3a072d118917957d6554.tar.gz external_webkit-2bde8e466a4451c7319e3a072d118917957d6554.tar.bz2 |
Merge WebKit at r82507: Initial merge by git
Change-Id: I60ce9d780725b58b45e54165733a8ffee23b683e
Diffstat (limited to 'Source/WebCore/platform/audio/chromium/AudioBusChromium.cpp')
-rw-r--r-- | Source/WebCore/platform/audio/chromium/AudioBusChromium.cpp | 28 |
1 files changed, 18 insertions, 10 deletions
diff --git a/Source/WebCore/platform/audio/chromium/AudioBusChromium.cpp b/Source/WebCore/platform/audio/chromium/AudioBusChromium.cpp index de44b1c..f34dc77 100644 --- a/Source/WebCore/platform/audio/chromium/AudioBusChromium.cpp +++ b/Source/WebCore/platform/audio/chromium/AudioBusChromium.cpp @@ -36,22 +36,30 @@ namespace WebCore { PassOwnPtr<AudioBus> AudioBus::loadPlatformResource(const char* name, double sampleRate) { - return PlatformBridge::loadPlatformAudioResource(name, sampleRate); + // FIXME: the sampleRate parameter is ignored. It should be removed from the API. + OwnPtr<AudioBus> audioBus = PlatformBridge::loadPlatformAudioResource(name, sampleRate); + if (!audioBus.get()) + return 0; + + // If the bus is already at the requested sample-rate then return as is. + if (audioBus->sampleRate() == sampleRate) + return audioBus.release(); + + return AudioBus::createBySampleRateConverting(audioBus.get(), false, sampleRate); } PassOwnPtr<AudioBus> createBusFromInMemoryAudioFile(const void* data, size_t dataSize, bool mixToMono, double sampleRate) { + // FIXME: the sampleRate parameter is ignored. It should be removed from the API. OwnPtr<AudioBus> audioBus = PlatformBridge::decodeAudioFileData(static_cast<const char*>(data), dataSize, sampleRate); - if (audioBus.get() && audioBus->numberOfChannels() == 2 && mixToMono) { - OwnPtr<AudioBus> monoAudioBus = adoptPtr(new AudioBus(1, audioBus->length())); - - // FIXME: AudioBus::copyFrom() should be able to do a downmix to mono. - // for now simply copy the left channel. - monoAudioBus->channel(0)->copyFrom(audioBus->channel(0)); - return monoAudioBus.release(); - } + if (!audioBus.get()) + return 0; + + // If the bus needs no conversion then return as is. + if ((!mixToMono || audioBus->numberOfChannels() == 1) && audioBus->sampleRate() == sampleRate) + return audioBus.release(); - return audioBus.release(); + return AudioBus::createBySampleRateConverting(audioBus.get(), mixToMono, sampleRate); } } // namespace WebCore |