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

Unified Diff: Source/modules/webaudio/AudioDestinationNode.h

Issue 135653002: Update modules classes to use OVERRIDE / FINAL when needed (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Remove change to web/ Created 6 years, 11 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/modules/webaudio/AudioContext.h ('k') | Source/modules/webaudio/AudioNode.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/modules/webaudio/AudioDestinationNode.h
diff --git a/Source/modules/webaudio/AudioDestinationNode.h b/Source/modules/webaudio/AudioDestinationNode.h
index 801d7fd656dbbca7936e67ffbb5344b7e135f1ac..616005e1ea70589dde5fe9067e237b69c90d17e8 100644
--- a/Source/modules/webaudio/AudioDestinationNode.h
+++ b/Source/modules/webaudio/AudioDestinationNode.h
@@ -42,12 +42,12 @@ public:
virtual ~AudioDestinationNode();
// AudioNode
- virtual void process(size_t) { }; // we're pulled by hardware so this is never called
- virtual void reset() { m_currentSampleFrame = 0; };
+ virtual void process(size_t) OVERRIDE FINAL { } // we're pulled by hardware so this is never called
+ virtual void reset() OVERRIDE FINAL { m_currentSampleFrame = 0; }
// The audio hardware calls render() to get the next render quantum of audio into destinationBus.
// It will optionally give us local/live audio input in sourceBus (if it's not 0).
- virtual void render(AudioBus* sourceBus, AudioBus* destinationBus, size_t numberOfFrames);
+ virtual void render(AudioBus* sourceBus, AudioBus* destinationBus, size_t numberOfFrames) OVERRIDE FINAL;
size_t currentSampleFrame() const { return m_currentSampleFrame; }
double currentTime() const { return currentSampleFrame() / static_cast<double>(sampleRate()); }
@@ -64,7 +64,7 @@ public:
protected:
// LocalAudioInputProvider allows us to expose an AudioSourceProvider for local/live audio input.
// If there is local/live audio input, we call set() with the audio input data every render quantum.
- class LocalAudioInputProvider : public AudioSourceProvider {
+ class LocalAudioInputProvider FINAL : public AudioSourceProvider {
public:
LocalAudioInputProvider()
: m_sourceBus(AudioBus::create(2, AudioNode::ProcessingSizeInFrames)) // FIXME: handle non-stereo local input.
@@ -78,7 +78,7 @@ protected:
}
// AudioSourceProvider.
- virtual void provideInput(AudioBus* destinationBus, size_t numberOfFrames)
+ virtual void provideInput(AudioBus* destinationBus, size_t numberOfFrames) OVERRIDE
{
bool isGood = destinationBus && destinationBus->length() == numberOfFrames && m_sourceBus->length() == numberOfFrames;
ASSERT(isGood);
@@ -90,8 +90,8 @@ protected:
RefPtr<AudioBus> m_sourceBus;
};
- virtual double tailTime() const OVERRIDE { return 0; }
- virtual double latencyTime() const OVERRIDE { return 0; }
+ virtual double tailTime() const OVERRIDE FINAL { return 0; }
+ virtual double latencyTime() const OVERRIDE FINAL { return 0; }
// Counts the number of sample-frames processed by the destination.
size_t m_currentSampleFrame;
« no previous file with comments | « Source/modules/webaudio/AudioContext.h ('k') | Source/modules/webaudio/AudioNode.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698