Virtual destructors should have virtual keyword.

chrome/browser/ui/cocoa/download/download_item_mac.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 CHROME_BROWSER_UI_COCOA_DOWNLOAD_DOWNLOAD_ITEM_MAC_H_
 #define CHROME_BROWSER_UI_COCOA_DOWNLOAD_DOWNLOAD_ITEM_MAC_H_
 #pragma once
 
 #import <Cocoa/Cocoa.h>
 
@@ skipping 15 matching lines @@
 // model. The owning object (DownloadItemController) must explicitly call
 // |LoadIcon| if it wants to display the icon associated with this download.
 
 class DownloadItemMac : DownloadItem::Observer {
  public:
   // DownloadItemMac takes ownership of |download_model|.
   DownloadItemMac(BaseDownloadItemModel* download_model,
                   DownloadItemController* controller);
 
   // Destructor.
-  ~DownloadItemMac();
+  virtual ~DownloadItemMac();
 
   // DownloadItem::Observer implementation
   virtual void OnDownloadUpdated(DownloadItem* download);
   virtual void OnDownloadOpened(DownloadItem* download);
 
   BaseDownloadItemModel* download_model() { return download_model_.get(); }
 
   // Asynchronous icon loading support.
   void LoadIcon();
 
@@ skipping 11 matching lines @@
   // For canceling an in progress icon request.
   CancelableRequestConsumerT<int, 0> icon_consumer_;
 
   // Stores the last known path where the file will be saved.
   FilePath lastFilePath_;
 
   DISALLOW_COPY_AND_ASSIGN(DownloadItemMac);
 };
 
 #endif  // CHROME_BROWSER_UI_COCOA_DOWNLOAD_DOWNLOAD_ITEM_MAC_H_