WebAudio code uses both kRenderQuantumFrames and ProcessingSizeInFrames to denote the rendering quantum size of 128 frames. Unify this.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b7198aeec6e82994ef5060872852997fccd74410 commit b7198aeec6e82994ef5060872852997fccd74410 Author: rtoy <rtoy@chromium.org> Date: Tue Nov 01 00:09:07 2016 Use AudioUtilities::kRenderQuantumFrames everywhere Use this instead of AudioHandler::ProcessingSizeInFrames so that there's only one definition of the rendering size. Basically ran sed -i 's;ProcessingSizeInFrames;AudioUtilities::kRenderQuantumFrames;g' for all files and then manually fixing up a few places that said AudioHandler::AudioUtilities::kRenderQuantumFrames BUG= 657620 TESTS=none; an internal change not visible. Review-Url: https://codereview.chromium.org/2440583002 Cr-Commit-Position: refs/heads/master@{#428875} [modify] https://crrev.com/b7198aeec6e82994ef5060872852997fccd74410/third_party/WebKit/Source/modules/webaudio/AudioDestinationNode.h [modify] https://crrev.com/b7198aeec6e82994ef5060872852997fccd74410/third_party/WebKit/Source/modules/webaudio/AudioNode.h [modify] https://crrev.com/b7198aeec6e82994ef5060872852997fccd74410/third_party/WebKit/Source/modules/webaudio/AudioNodeInput.cpp [modify] https://crrev.com/b7198aeec6e82994ef5060872852997fccd74410/third_party/WebKit/Source/modules/webaudio/AudioNodeOutput.cpp [modify] https://crrev.com/b7198aeec6e82994ef5060872852997fccd74410/third_party/WebKit/Source/modules/webaudio/AudioParam.cpp [modify] https://crrev.com/b7198aeec6e82994ef5060872852997fccd74410/third_party/WebKit/Source/modules/webaudio/AudioParamTimeline.cpp [modify] https://crrev.com/b7198aeec6e82994ef5060872852997fccd74410/third_party/WebKit/Source/modules/webaudio/AudioScheduledSourceNode.cpp [modify] https://crrev.com/b7198aeec6e82994ef5060872852997fccd74410/third_party/WebKit/Source/modules/webaudio/ConstantSourceNode.cpp [modify] https://crrev.com/b7198aeec6e82994ef5060872852997fccd74410/third_party/WebKit/Source/modules/webaudio/ConvolverNode.cpp [modify] https://crrev.com/b7198aeec6e82994ef5060872852997fccd74410/third_party/WebKit/Source/modules/webaudio/DelayDSPKernel.cpp [modify] https://crrev.com/b7198aeec6e82994ef5060872852997fccd74410/third_party/WebKit/Source/modules/webaudio/GainNode.cpp [modify] https://crrev.com/b7198aeec6e82994ef5060872852997fccd74410/third_party/WebKit/Source/modules/webaudio/MediaStreamAudioDestinationNode.cpp [modify] https://crrev.com/b7198aeec6e82994ef5060872852997fccd74410/third_party/WebKit/Source/modules/webaudio/OscillatorNode.cpp [modify] https://crrev.com/b7198aeec6e82994ef5060872852997fccd74410/third_party/WebKit/Source/modules/webaudio/PannerNode.cpp [modify] https://crrev.com/b7198aeec6e82994ef5060872852997fccd74410/third_party/WebKit/Source/modules/webaudio/ScriptProcessorNode.cpp [modify] https://crrev.com/b7198aeec6e82994ef5060872852997fccd74410/third_party/WebKit/Source/modules/webaudio/StereoPannerNode.cpp
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6204f7071ce4b5ee06ba23ba7014f8e09b603ba6 commit 6204f7071ce4b5ee06ba23ba7014f8e09b603ba6 Author: rtoy <rtoy@chromium.org> Date: Mon Nov 07 19:27:45 2016 Use kRenderQuantumFrames in more places https://codereview.chromium.org/2440583002/ missed a few places where the constant 128 was definitely referring to the render quantum size. Update the code to use kRenderQuantumFrames. BUG= 657620 TEST=none Review-Url: https://codereview.chromium.org/2478283003 Cr-Commit-Position: refs/heads/master@{#430344} [modify] https://crrev.com/6204f7071ce4b5ee06ba23ba7014f8e09b603ba6/third_party/WebKit/Source/modules/webaudio/OfflineAudioDestinationNode.cpp [modify] https://crrev.com/6204f7071ce4b5ee06ba23ba7014f8e09b603ba6/third_party/WebKit/Source/modules/webaudio/OfflineAudioDestinationNode.h [modify] https://crrev.com/6204f7071ce4b5ee06ba23ba7014f8e09b603ba6/third_party/WebKit/Source/modules/webaudio/WaveShaperDSPKernel.cpp [modify] https://crrev.com/6204f7071ce4b5ee06ba23ba7014f8e09b603ba6/third_party/WebKit/Source/platform/audio/AudioDestination.cpp [modify] https://crrev.com/6204f7071ce4b5ee06ba23ba7014f8e09b603ba6/third_party/WebKit/Source/platform/audio/HRTFPanner.cpp
Comment 1 by bugdroid1@chromium.org
, Nov 1 2016