Virtual destructors should have virtual keyword.

net/tools/flip_server/sm_connection.cc

 // Copyright (c) 2009 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.
 
 #include "net/tools/flip_server/sm_connection.h"
 
 #include <errno.h>
 #include <netinet/tcp.h>
 #include <sys/socket.h>
 
 #include <list>
 #include <string>
 
 #include "net/tools/flip_server/constants.h"
 #include "net/tools/flip_server/flip_config.h"
 #include "net/tools/flip_server/http_interface.h"
 #include "net/tools/flip_server/spdy_interface.h"
 #include "net/tools/flip_server/spdy_ssl.h"
 #include "net/tools/flip_server/streamer_interface.h"
 
 namespace net {
 
 // static
 bool SMConnection::force_spdy_ = false;
 
+DataFrame::~DataFrame() {
+  if (delete_when_done)
+    delete[] data;
+}
+
 SMConnection::SMConnection(EpollServer* epoll_server,
                            SSLState* ssl_state,
                            MemoryCache* memory_cache,
                            FlipAcceptor* acceptor,
                            std::string log_prefix)
     : last_read_time_(0),
       fd_(-1),
       events_(0),
       registered_in_epoll_server_(false),
       initialized_(false),
@@ skipping 616 matching lines @@
 SMConnection* SMConnection::NewSMConnection(EpollServer* epoll_server,
                                             SSLState *ssl_state,
                                             MemoryCache* memory_cache,
                                             FlipAcceptor *acceptor,
                                             std::string log_prefix) {
   return new SMConnection(epoll_server, ssl_state, memory_cache,
                           acceptor, log_prefix);
 }
 
 }  // namespace net
-
-