Virtual destructors should have virtual keyword.

content/renderer/render_process_impl.h

 // Copyright (c) 2010 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 CONTENT_RENDERER_RENDER_PROCESS_IMPL_H_
 #define CONTENT_RENDERER_RENDER_PROCESS_IMPL_H_
 #pragma once
 
 #include "base/timer.h"
 #include "content/renderer/render_process.h"
 #include "native_client/src/shared/imc/nacl_imc.h"
 
 namespace skia {
 class PlatformCanvas;
 }
 
 // Implementation of the RenderProcess interface for the regular browser.
 // See also MockRenderProcess which implements the active "RenderProcess" when
 // running under certain unit tests.
 class RenderProcessImpl : public RenderProcess {
  public:
   RenderProcessImpl();
-  ~RenderProcessImpl();
+  virtual ~RenderProcessImpl();
 
   // RenderProcess implementation.
   virtual skia::PlatformCanvas* GetDrawingCanvas(TransportDIB** memory,
                                                  const gfx::Rect& rect);
   virtual void ReleaseTransportDIB(TransportDIB* memory);
   virtual bool UseInProcessPlugins() const;
   virtual bool HasInitializedMediaLibrary() const;
 
   // Like UseInProcessPlugins(), but called before RenderProcess is created
   // and does not allow overriding by tests. This just checks the command line
@@ skipping 33 matching lines @@
 
   // TransportDIB sequence number
   uint32 transport_dib_next_sequence_number_;
 
   bool in_process_plugins_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderProcessImpl);
 };
 
 #endif  // CONTENT_RENDERER_RENDER_PROCESS_IMPL_H_