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

Unified Diff: media/audio/audio_low_latency_input_output_unittest.cc

Issue 661163004: Standardize usage of virtual/override/final specifiers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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 | « media/audio/audio_input_unittest.cc ('k') | media/audio/audio_output_controller_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/audio/audio_low_latency_input_output_unittest.cc
diff --git a/media/audio/audio_low_latency_input_output_unittest.cc b/media/audio/audio_low_latency_input_output_unittest.cc
index 49a0e1c5f6a7c6d1a60d3f6a3df387b21d392c09..7d8fc0a68ae1377a5b1a9fe5368f0c46710efc8a 100644
--- a/media/audio/audio_low_latency_input_output_unittest.cc
+++ b/media/audio/audio_low_latency_input_output_unittest.cc
@@ -93,9 +93,9 @@ struct AudioDelayState {
class MockAudioManager : public AudioManagerAnyPlatform {
public:
MockAudioManager() : AudioManagerAnyPlatform(&fake_audio_log_factory_) {}
- virtual ~MockAudioManager() {}
+ ~MockAudioManager() override {}
- virtual scoped_refptr<base::SingleThreadTaskRunner> GetTaskRunner() override {
+ scoped_refptr<base::SingleThreadTaskRunner> GetTaskRunner() override {
return base::MessageLoop::current()->message_loop_proxy();
}
@@ -109,7 +109,7 @@ class AudioLowLatencyInputOutputTest : public testing::Test {
protected:
AudioLowLatencyInputOutputTest() {}
- virtual ~AudioLowLatencyInputOutputTest() {}
+ ~AudioLowLatencyInputOutputTest() override {}
AudioManager* audio_manager() { return &mock_audio_manager_; }
base::MessageLoopForUI* message_loop() { return &message_loop_; }
@@ -162,7 +162,7 @@ class FullDuplexAudioSinkSource
delay_states_.reset(new AudioDelayState[kMaxDelayMeasurements]);
}
- virtual ~FullDuplexAudioSinkSource() {
+ ~FullDuplexAudioSinkSource() override {
// Get complete file path to output file in the directory containing
// media_unittests.exe. Example: src/build/Debug/audio_delay_values_ms.txt.
base::FilePath file_name;
@@ -191,10 +191,10 @@ class FullDuplexAudioSinkSource
}
// AudioInputStream::AudioInputCallback.
- virtual void OnData(AudioInputStream* stream,
- const AudioBus* src,
- uint32 hardware_delay_bytes,
- double volume) override {
+ void OnData(AudioInputStream* stream,
+ const AudioBus* src,
+ uint32 hardware_delay_bytes,
+ double volume) override {
base::AutoLock lock(lock_);
// Update three components in the AudioDelayState for this recorded
@@ -222,11 +222,10 @@ class FullDuplexAudioSinkSource
// }
}
- virtual void OnError(AudioInputStream* stream) override {}
+ void OnError(AudioInputStream* stream) override {}
// AudioOutputStream::AudioSourceCallback.
- virtual int OnMoreData(AudioBus* audio_bus,
- uint32 total_bytes_delay) override {
+ int OnMoreData(AudioBus* audio_bus, uint32 total_bytes_delay) override {
base::AutoLock lock(lock_);
// Update one component in the AudioDelayState for the packet
@@ -254,7 +253,7 @@ class FullDuplexAudioSinkSource
return 0;
}
- virtual void OnError(AudioOutputStream* stream) override {}
+ void OnError(AudioOutputStream* stream) override {}
protected:
// Converts from bytes to milliseconds taking the sample rate and size
« no previous file with comments | « media/audio/audio_input_unittest.cc ('k') | media/audio/audio_output_controller_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698