Virtual destructors should have virtual keyword.

net/tools/flip_server/simple_buffer.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/simple_buffer.h"
 #include "base/logging.h"
 
 // Some of the following member functions are marked inlined, even though they
 // are virtual. This may seem counter-intuitive, since virtual functions are
 // generally not eligible for inlining. Profiling results indicate that these
@@ skipping 19 matching lines @@
 SimpleBuffer::SimpleBuffer(int size)
   : write_idx_(0),
     read_idx_(0),
     storage_size_(size) {
   // Callers may try to allocate overly large blocks, but negative sizes are
   // obviously wrong.
   CHECK_GE(size, 0);
   storage_ = new char[size];
 }
 
+SimpleBuffer::~SimpleBuffer() {
+  delete[] storage_;
+}
+
+
 ////////////////////////////////////////////////////////////////////////////////
 
 int SimpleBuffer::ReadableBytes() const {
   return write_idx_ - read_idx_;
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 
 std::string SimpleBuffer::str() const {
   std::string s;
@@ skipping 144 matching lines @@
 // amount of data specified here is expected to
 // already be resident (as if it was Written)
 inline void SimpleBuffer::AdvanceWritablePtr(int amount_to_advance) {
   write_idx_ += amount_to_advance;
   if (write_idx_ > storage_size_) {
     write_idx_ = storage_size_;
   }
 }
 
 }  // namespace net
-