Virtual destructors should have virtual keyword.

chrome/browser/net/passive_log_collector.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_NET_PASSIVE_LOG_COLLECTOR_H_
 #define CHROME_BROWSER_NET_PASSIVE_LOG_COLLECTOR_H_
 #pragma once
 
 #include <deque>
 #include <string>
@@ skipping 72 matching lines @@
 
     // Appends all the captured entries to |out|. The ordering is undefined.
     virtual void AppendAllEntries(ChromeNetLog::EntryList* out) const = 0;
   };
 
   // This source tracker is intended for TYPE_NONE. All entries go into a
   // circular buffer, and there is no concept of live/dead requests.
   class GlobalSourceTracker : public SourceTrackerInterface {
    public:
     GlobalSourceTracker();
-    ~GlobalSourceTracker();
+    virtual ~GlobalSourceTracker();
 
     // SourceTrackerInterface implementation:
     virtual void OnAddEntry(const ChromeNetLog::Entry& entry);
     virtual void Clear();
     virtual void AppendAllEntries(ChromeNetLog::EntryList* out) const;
 
    private:
     typedef std::deque<ChromeNetLog::Entry> CircularEntryList;
     CircularEntryList entries_;
     DISALLOW_COPY_AND_ASSIGN(GlobalSourceTracker);
@@ skipping 232 matching lines @@
     explicit HttpStreamJobTracker(PassiveLogCollector* parent);
 
    private:
     virtual Action DoAddEntry(const ChromeNetLog::Entry& entry,
                               SourceInfo* out_info);
     DISALLOW_COPY_AND_ASSIGN(HttpStreamJobTracker);
   };
 
 
   PassiveLogCollector();
-  ~PassiveLogCollector();
+  virtual ~PassiveLogCollector();
 
   // ThreadSafeObserver implementation:
   virtual void OnAddEntry(net::NetLog::EventType type,
                           const base::TimeTicks& time,
                           const net::NetLog::Source& source,
                           net::NetLog::EventPhase phase,
                           net::NetLog::EventParameters* params);
 
   // Clears all of the passively logged data.
   void Clear();
@@ skipping 31 matching lines @@
   SourceTrackerInterface* trackers_[net::NetLog::SOURCE_COUNT];
 
   // The count of how many events have flowed through this log. Used to set the
   // "order" field on captured events.
   uint32 num_events_seen_;
 
   DISALLOW_COPY_AND_ASSIGN(PassiveLogCollector);
 };
 
 #endif  // CHROME_BROWSER_NET_PASSIVE_LOG_COLLECTOR_H_