Virtual destructors should have virtual keyword.

chrome/renderer/safe_browsing/malware_dom_details.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.
 //
 // MalwareDOMDetails iterates over a document's frames and gathers
 // interesting URLs such as those of scripts and frames. When done, it sends
 // them to the MalwareDetails that requested them.
 
 #ifndef CHROME_RENDERER_SAFE_BROWSING_MALWARE_DOM_DETAILS_H_
 #define CHROME_RENDERER_SAFE_BROWSING_MALWARE_DOM_DETAILS_H_
@@ skipping 19 matching lines @@
 
 namespace safe_browsing {
 
 // There is one MalwareDOMDetails per RenderView.
 class MalwareDOMDetails : public RenderViewObserver {
  public:
   // An upper limit on the number of nodes we collect. Not const for the test.
   static uint32 kMaxNodes;
 
   static MalwareDOMDetails* Create(RenderView* render_view);
-  ~MalwareDOMDetails();
+  virtual ~MalwareDOMDetails();
 
   // Begins extracting resource urls for the page currently loaded in
   // this object's RenderView.
   void ExtractResources(
       std::vector<SafeBrowsingHostMsg_MalwareDOMDetails_Node>* resources);
 
  private:
   // Creates a MalwareDOMDetails for the specified RenderView.
   // The MalwareDOMDetails should be destroyed prior to destroying
   // the RenderView.
@@ skipping 12 matching lines @@
 
   // Non-owned pointer to the view that we will extract features from.
   RenderView* render_view_;
 
   DISALLOW_COPY_AND_ASSIGN(MalwareDOMDetails);
 };
 
 }  // namespace safe_browsing
 
 #endif  // CHROME_RENDERER_SAFE_BROWSING_MALWARE_DOM_DETAILS_H_