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

Unified Diff: content/renderer/pepper/pepper_media_stream_audio_track_host.h

Issue 670683003: Standardize usage of virtual/override/final in content/renderer/ (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
Index: content/renderer/pepper/pepper_media_stream_audio_track_host.h
diff --git a/content/renderer/pepper/pepper_media_stream_audio_track_host.h b/content/renderer/pepper/pepper_media_stream_audio_track_host.h
index 2940c7bf235c75e1b3ddb2c63087e388299e8b74..1580441fc11c5135e1d5a905f594a5fa187a7ce7 100644
--- a/content/renderer/pepper/pepper_media_stream_audio_track_host.h
+++ b/content/renderer/pepper/pepper_media_stream_audio_track_host.h
@@ -38,7 +38,7 @@ class PepperMediaStreamAudioTrackHost : public PepperMediaStreamTrackHostBase {
class AudioSink : public MediaStreamAudioSink {
public:
explicit AudioSink(PepperMediaStreamAudioTrackHost* host);
- virtual ~AudioSink();
+ ~AudioSink() override;
// Enqueues a free buffer index into |buffers_| which will be used for
// sending audio samples to plugin.
@@ -64,11 +64,11 @@ class PepperMediaStreamAudioTrackHost : public PepperMediaStreamTrackHostBase {
// MediaStreamAudioSink overrides:
// These two functions should be called on the audio thread.
- virtual void OnData(const int16* audio_data,
- int sample_rate,
- int number_of_channels,
- int number_of_frames) override;
- virtual void OnSetFormat(const media::AudioParameters& params) override;
+ void OnData(const int16* audio_data,
+ int sample_rate,
+ int number_of_channels,
+ int number_of_frames) override;
+ void OnSetFormat(const media::AudioParameters& params) override;
// Unowned host which is available during the AudioSink's lifespan.
// It is mainly used in the main thread. But the audio thread will use
@@ -148,10 +148,10 @@ class PepperMediaStreamAudioTrackHost : public PepperMediaStreamTrackHostBase {
DISALLOW_COPY_AND_ASSIGN(AudioSink);
};
- virtual ~PepperMediaStreamAudioTrackHost();
+ ~PepperMediaStreamAudioTrackHost() override;
// ResourceMessageHandler overrides:
- virtual int32_t OnResourceMessageReceived(
+ int32_t OnResourceMessageReceived(
const IPC::Message& msg,
ppapi::host::HostMessageContext* context) override;
@@ -161,13 +161,13 @@ class PepperMediaStreamAudioTrackHost : public PepperMediaStreamTrackHostBase {
const ppapi::MediaStreamAudioTrackShared::Attributes& attributes);
// PepperMediaStreamTrackHostBase overrides:
- virtual void OnClose() override;
+ void OnClose() override;
// MediaStreamBufferManager::Delegate overrides:
- virtual void OnNewBufferEnqueued() override;
+ void OnNewBufferEnqueued() override;
// ResourceHost overrides:
- virtual void DidConnectPendingHostToResource() override;
+ void DidConnectPendingHostToResource() override;
blink::WebMediaStreamTrack track_;
« no previous file with comments | « content/renderer/pepper/pepper_media_device_manager.h ('k') | content/renderer/pepper/pepper_media_stream_track_host_base.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698