Virtual destructors should have virtual keyword.

chrome/browser/net/load_timing_observer.h

 // Copyright (c) 2010 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_LOAD_TIMING_OBSERVER_H_
 #define CHROME_BROWSER_NET_LOAD_TIMING_OBSERVER_H_
 #pragma once
 
 #include "base/gtest_prod_util.h"
 #include "base/hash_tables.h"
@@ skipping 42 matching lines @@
     base::TimeTicks dns_start;
     base::TimeTicks dns_end;
   };
 
   struct SocketRecord {
     base::TimeTicks ssl_start;
     base::TimeTicks ssl_end;
   };
 
   LoadTimingObserver();
-  ~LoadTimingObserver();
+  virtual ~LoadTimingObserver();
 
   URLRequestRecord* GetURLRequestRecord(uint32 source_id);
 
   // 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);
 
@@ skipping 43 matching lines @@
   HTTPStreamJobToRecordMap http_stream_job_to_record_;
   ConnectJobToRecordMap connect_job_to_record_;
   SocketToRecordMap socket_to_record_;
   uint32 last_connect_job_id_;
   ConnectJobRecord last_connect_job_record_;
 
   DISALLOW_COPY_AND_ASSIGN(LoadTimingObserver);
 };
 
 #endif  // CHROME_BROWSER_NET_LOAD_TIMING_OBSERVER_H_