Virtual destructors should have virtual keyword.

chrome/browser/sync/engine/all_status.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.
 //
 // The AllStatus object watches various sync engine components and aggregates
 // the status of all of them into one place.
 
 #ifndef CHROME_BROWSER_SYNC_ENGINE_ALL_STATUS_H_
 #define CHROME_BROWSER_SYNC_ENGINE_ALL_STATUS_H_
 #pragma once
@@ skipping 12 matching lines @@
 class Syncer;
 class SyncerThread;
 struct AuthWatcherEvent;
 struct ServerConnectionEvent;
 
 class AllStatus : public SyncEngineEventListener {
   friend class ScopedStatusLock;
  public:
 
   AllStatus();
-  ~AllStatus();
+  virtual ~AllStatus();
 
   void HandleServerConnectionEvent(const ServerConnectionEvent& event);
 
   void HandleAuthWatcherEvent(const AuthWatcherEvent& event);
 
   virtual void OnSyncEngineEvent(const SyncEngineEvent& event);
 
   sync_api::SyncManager::Status status() const;
 
   void SetNotificationsEnabled(bool notifications_enabled);
@@ skipping 21 matching lines @@
  public:
   explicit ScopedStatusLock(AllStatus* allstatus);
   ~ScopedStatusLock();
  protected:
   AllStatus* allstatus_;
 };
 
 }  // namespace browser_sync
 
 #endif  // CHROME_BROWSER_SYNC_ENGINE_ALL_STATUS_H_