Virtual destructors should have virtual keyword.

remoting/jingle_glue/iq_request.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_JINGLE_GLUE_IQ_REQUEST_H_
 #define REMOTING_JINGLE_GLUE_IQ_REQUEST_H_
 
 #include <map>
 #include <string>
 #include <vector>
@@ skipping 214 matching lines @@
 //
 // This class is not threadsafe, and should only be used on the thread it is
 // created on.
 //
 // TODO(ajwong): Move into its own file, and rename to something better since
 // this is not actually a Request. Maybe SessionEstablishmentConnector.
 class SessionStartRequest : public sigslot::has_slots<> {
  public:
   SessionStartRequest(JavascriptIqRequest* request,
                       cricket::SessionManager* session_manager);
-  ~SessionStartRequest();
+  virtual ~SessionStartRequest();
 
   void Run();
 
  private:
   void OnResponse(const buzz::XmlElement* stanza);
 
   void OnOutgoingMessage(cricket::SessionManager* manager,
                          const buzz::XmlElement* stanza);
 
   scoped_ptr<JavascriptIqRequest> request_;
   cricket::SessionManager* session_manager_;
 
   DISALLOW_COPY_AND_ASSIGN(SessionStartRequest);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_JINGLE_GLUE_IQ_REQUEST_H_