Virtual destructors should have virtual keyword.

remoting/protocol/rtp_reader.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef REMOTING_PROTOCOL_RTP_READER_H_
 #define REMOTING_PROTOCOL_RTP_READER_H_
 
 #include "base/memory/scoped_ptr.h"
 #include "remoting/base/compound_buffer.h"
 #include "remoting/protocol/rtp_utils.h"
@@ skipping 30 matching lines @@
   uint32 extended_sequence_number_;
 };
 
 // RtpReader implements and RTP receiver. It reads packets from RTP
 // socket, parses them, calculates extended sequence number and then
 // passes them to a callback. It also collects statistics for RTCP
 // receiver reports, but doesn't send any RTCP packets itself.
 class RtpReader : public SocketReaderBase {
  public:
   RtpReader();
-  ~RtpReader();
+  virtual ~RtpReader();
 
   // The OnMessageCallback is called whenever a new message is received.
   // Ownership of the message is passed the callback.
   typedef Callback1<const RtpPacket*>::Type OnMessageCallback;
 
   // Initialize the reader and start reading. Must be called on the thread
   // |socket| belongs to. The callback will be called when a new message is
   // received. RtpReader owns |on_message_callback|, doesn't own
   // |socket|.
   void Init(net::Socket* socket, OnMessageCallback* on_message_callback);
@@ skipping 14 matching lines @@
   int total_packets_received_;
 
   DISALLOW_COPY_AND_ASSIGN(RtpReader);
 };
 
 }  // namespace protocol
 }  // namespace remoting
 
 
 #endif  // REMOTING_PROTOCOL_RTP_READER_H_