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

Side by Side Diff: third_party/WebKit/Source/modules/webaudio/AudioDestinationNode.cpp

Issue 2060833002: Implementation of 'AudioContext.getOutputTimestamp' method (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Implementation of 'AudioContext.getOutputTimestamp' method Created 4 years, 6 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2010, Google Inc. All rights reserved. 2 * Copyright (C) 2010, Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 1. Redistributions of source code must retain the above copyright 7 * 1. Redistributions of source code must retain the above copyright
8 * notice, this list of conditions and the following disclaimer. 8 * notice, this list of conditions and the following disclaimer.
9 * 2. Redistributions in binary form must reproduce the above copyright 9 * 2. Redistributions in binary form must reproduce the above copyright
10 * notice, this list of conditions and the following disclaimer in the 10 * notice, this list of conditions and the following disclaimer in the
11 * documentation and/or other materials provided with the distribution. 11 * documentation and/or other materials provided with the distribution.
12 * 12 *
13 * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS'' AND AN Y 13 * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS'' AND AN Y
14 * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED 14 * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
15 * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE 15 * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
16 * DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS BE LIABLE FOR AN Y 16 * DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS BE LIABLE FOR AN Y
17 * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES 17 * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
18 * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; 18 * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
19 * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND O N 19 * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND O N
20 * ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT 20 * ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
21 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS 21 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
22 * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 22 * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
23 */ 23 */
24 24
25 #include "modules/webaudio/AudioDestinationNode.h" 25 #include "modules/webaudio/AudioDestinationNode.h"
26
26 #include "modules/webaudio/AbstractAudioContext.h" 27 #include "modules/webaudio/AbstractAudioContext.h"
27 #include "modules/webaudio/AudioNodeInput.h" 28 #include "modules/webaudio/AudioNodeInput.h"
28 #include "modules/webaudio/AudioNodeOutput.h" 29 #include "modules/webaudio/AudioNodeOutput.h"
29 #include "platform/audio/AudioUtilities.h" 30 #include "platform/audio/AudioUtilities.h"
30 #include "platform/audio/DenormalDisabler.h" 31 #include "platform/audio/DenormalDisabler.h"
31 #include "wtf/Atomics.h" 32 #include "wtf/Atomics.h"
32 33
33 namespace blink { 34 namespace blink {
34 35
35 AudioDestinationHandler::AudioDestinationHandler(AudioNode& node, float sampleRa te) 36 AudioDestinationHandler::AudioDestinationHandler(AudioNode& node, float sampleRa te)
36 : AudioHandler(NodeTypeDestination, node, sampleRate) 37 : AudioHandler(NodeTypeDestination, node, sampleRate)
37 , m_currentSampleFrame(0) 38 , m_currentSampleFrame(0)
38 { 39 {
39 addInput(); 40 addInput();
40 } 41 }
41 42
42 AudioDestinationHandler::~AudioDestinationHandler() 43 AudioDestinationHandler::~AudioDestinationHandler()
43 { 44 {
44 ASSERT(!isInitialized()); 45 ASSERT(!isInitialized());
45 } 46 }
46 47
47 void AudioDestinationHandler::render(AudioBus* sourceBus, AudioBus* destinationB us, size_t numberOfFrames) 48 void AudioDestinationHandler::render(AudioBus* sourceBus, AudioBus* destinationB us, size_t numberOfFrames, const WebAudioTimestamp& outputTimestamp)
48 { 49 {
50 {
51 MutexLocker locker(m_mutex);
Raymond Toy 2016/06/14 16:42:40 Can this be blocked by the main thread? Generally
hongchan 2016/06/14 17:10:38 Consider to move this operation to context's pre-r
52 m_outputTimestamp = outputTimestamp;
53 }
54
49 // We don't want denormals slowing down any of the audio processing 55 // We don't want denormals slowing down any of the audio processing
50 // since they can very seriously hurt performance. 56 // since they can very seriously hurt performance.
51 // This will take care of all AudioNodes because they all process within thi s scope. 57 // This will take care of all AudioNodes because they all process within thi s scope.
52 DenormalDisabler denormalDisabler; 58 DenormalDisabler denormalDisabler;
53 59
54 // Need to check if the context actually alive. Otherwise the subsequent 60 // Need to check if the context actually alive. Otherwise the subsequent
55 // steps will fail. If the context is not alive somehow, return immediately 61 // steps will fail. If the context is not alive somehow, return immediately
56 // and do nothing. 62 // and do nothing.
57 // 63 //
58 // TODO(hongchan): because the context can go away while rendering, so this 64 // TODO(hongchan): because the context can go away while rendering, so this
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
98 context()->deferredTaskHandler().processAutomaticPullNodes(numberOfFrames); 104 context()->deferredTaskHandler().processAutomaticPullNodes(numberOfFrames);
99 105
100 // Let the context take care of any business at the end of each render quant um. 106 // Let the context take care of any business at the end of each render quant um.
101 context()->handlePostRenderTasks(); 107 context()->handlePostRenderTasks();
102 108
103 // Advance current sample-frame. 109 // Advance current sample-frame.
104 size_t newSampleFrame = m_currentSampleFrame + numberOfFrames; 110 size_t newSampleFrame = m_currentSampleFrame + numberOfFrames;
105 releaseStore(&m_currentSampleFrame, newSampleFrame); 111 releaseStore(&m_currentSampleFrame, newSampleFrame);
106 } 112 }
107 113
114 WebAudioTimestamp AudioDestinationHandler::outputTimestamp() const
115 {
116 MutexLocker locker(m_mutex);
hongchan 2016/06/14 17:10:38 This will block the audio thread when the mutex is
117 return m_outputTimestamp;
118 }
119
108 // ---------------------------------------------------------------- 120 // ----------------------------------------------------------------
109 121
110 AudioDestinationNode::AudioDestinationNode(AbstractAudioContext& context) 122 AudioDestinationNode::AudioDestinationNode(AbstractAudioContext& context)
111 : AudioNode(context) 123 : AudioNode(context)
112 { 124 {
113 } 125 }
114 126
115 AudioDestinationHandler& AudioDestinationNode::audioDestinationHandler() const 127 AudioDestinationHandler& AudioDestinationNode::audioDestinationHandler() const
116 { 128 {
117 return static_cast<AudioDestinationHandler&>(handler()); 129 return static_cast<AudioDestinationHandler&>(handler());
118 } 130 }
119 131
120 unsigned long AudioDestinationNode::maxChannelCount() const 132 unsigned long AudioDestinationNode::maxChannelCount() const
121 { 133 {
122 return audioDestinationHandler().maxChannelCount(); 134 return audioDestinationHandler().maxChannelCount();
123 } 135 }
124 136
125 } // namespace blink 137 } // namespace blink
126 138
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698