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

Unified Diff: chrome/browser/media/webrtc_rtp_dump_handler_unittest.cc

Issue 665323004: Standardize usage of virtual/override/final in chrome/browser/media (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: chrome/browser/media/webrtc_rtp_dump_handler_unittest.cc
diff --git a/chrome/browser/media/webrtc_rtp_dump_handler_unittest.cc b/chrome/browser/media/webrtc_rtp_dump_handler_unittest.cc
index 78ad5bee007fb5d13caaa7883e9fe30eba97ffa6..eed30b5b9d954d7b74731dedbe134aaa5a7e006b 100644
--- a/chrome/browser/media/webrtc_rtp_dump_handler_unittest.cc
+++ b/chrome/browser/media/webrtc_rtp_dump_handler_unittest.cc
@@ -28,17 +28,17 @@ class FakeDumpWriter : public WebRtcRtpDumpWriter {
max_size_reached_callback_(max_size_reached_callback),
end_dump_success_(end_dump_success) {}
- virtual void WriteRtpPacket(const uint8* packet_header,
- size_t header_length,
- size_t packet_length,
- bool incoming) override {
+ void WriteRtpPacket(const uint8* packet_header,
+ size_t header_length,
+ size_t packet_length,
+ bool incoming) override {
current_dump_size_ += header_length;
if (current_dump_size_ > max_dump_size_)
max_size_reached_callback_.Run();
}
- virtual void EndDump(RtpDumpType type,
- const EndDumpCallback& finished_callback) override {
+ void EndDump(RtpDumpType type,
+ const EndDumpCallback& finished_callback) override {
bool incoming_sucess = end_dump_success_;
bool outgoing_success = end_dump_success_;

Powered by Google App Engine
This is Rietveld 408576698