Virtual destructors should have virtual keyword.

remoting/protocol/jingle_session_manager.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_JINGLE_SESSION_MANAGER_H_
 #define REMOTING_PROTOCOL_JINGLE_SESSION_MANAGER_H_
 
 #include <list>
 #include <string>
 
@@ skipping 24 matching lines @@
 // ContentDescription used for chromoting sessions. It contains the information
 // from the content description stanza in the session intialization handshake.
 //
 // This class also provides a type abstraction so that the Chromotocol Session
 // interface does not need to depend on libjingle.
 class ContentDescription : public cricket::ContentDescription {
  public:
   explicit ContentDescription(const CandidateSessionConfig* config,
                               const std::string& auth_token,
                               scoped_refptr<net::X509Certificate> certificate);
-  ~ContentDescription();
+  virtual ~ContentDescription();
 
   const CandidateSessionConfig* config() const {
     return candidate_config_.get();
   }
 
   const std::string& auth_token() const { return auth_token_; }
 
   scoped_refptr<net::X509Certificate> certificate() const {
     return certificate_;
   }
@@ skipping 96 matching lines @@
   scoped_ptr<crypto::RSAPrivateKey> private_key_;
 
   DISALLOW_COPY_AND_ASSIGN(JingleSessionManager);
 };
 
 }  // namespace protocol
 
 }  // namespace remoting
 
 #endif  // REMOTING_PROTOCOL_JINGLE_SESSION_MANAGER_H_