Virtual destructors should have virtual keyword.

chrome/browser/importer/importer_host.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_IMPORTER_IMPORTER_HOST_H_
 #define CHROME_BROWSER_IMPORTER_IMPORTER_HOST_H_
 #pragma once
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
@@ skipping 63 matching lines @@
       const importer::SourceProfile& source_profile,
       Profile* target_profile,
       uint16 items,
       ProfileWriter* writer,
       bool first_run);
 
   // Cancels the import process.
   virtual void Cancel();
 
  protected:
-  ~ImporterHost();
+  virtual ~ImporterHost();
 
   // Returns true if importer should import to bookmark bar.
   bool ShouldImportToBookmarkBar(bool first_run);
 
   // Make sure that Firefox isn't running, if import browser is Firefox. Show
   // to the user a dialog that notifies that is necessary to close Firefox
   // prior to continue.
   // |source_profile| - importer profile to import.
   // |items| - specifies which data to import (bitmask of importer::ImportItem).
   // |first_run| - true if this method is being called during first run.
@@ skipping 63 matching lines @@
   // The observer that we need to notify about changes in the import process.
   importer::ImporterProgressObserver* observer_;
 
   // Firefox profile lock.
   scoped_ptr<FirefoxProfileLock> firefox_lock_;
 
   DISALLOW_COPY_AND_ASSIGN(ImporterHost);
 };
 
 #endif  // CHROME_BROWSER_IMPORTER_IMPORTER_HOST_H_