diff options
author | hans@chromium.org <hans@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-05-25 09:08:19 +0000 |
---|---|---|
committer | hans@chromium.org <hans@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-05-25 09:08:19 +0000 |
commit | 3690ebe09a8c3cea0fd7d9ece8f5b8d8ebc65c19 (patch) | |
tree | b26f7f81e48a95473eb4af5304301e9d8cd22fb8 /chrome/browser/net/net_log_logger.h | |
parent | eef99b6591d82399096abdcee07dd67359eec036 (diff) | |
download | chromium_src-3690ebe09a8c3cea0fd7d9ece8f5b8d8ebc65c19.zip chromium_src-3690ebe09a8c3cea0fd7d9ece8f5b8d8ebc65c19.tar.gz chromium_src-3690ebe09a8c3cea0fd7d9ece8f5b8d8ebc65c19.tar.bz2 |
Virtual destructors should have virtual keyword.
Make sure user-declared virtual destructors always have the virtual keyword.
The Clang style-check plugin will check for this soon.
No functionality change: virtual is only added
to destructors that are already implicitly virtual.
Also fix a couple of in-line destructor definitions.
BUG=83408
TEST=none
Review URL: http://codereview.chromium.org/7064033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@86587 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/net/net_log_logger.h')
-rw-r--r-- | chrome/browser/net/net_log_logger.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/chrome/browser/net/net_log_logger.h b/chrome/browser/net/net_log_logger.h index b1440b7..9f3ff75 100644 --- a/chrome/browser/net/net_log_logger.h +++ b/chrome/browser/net/net_log_logger.h @@ -23,7 +23,7 @@ class NetLogLogger : public ChromeNetLog::ThreadSafeObserver { // Otherwise, writes to |log_path|. Uses one line per entry, for // easy parsing. explicit NetLogLogger(const FilePath &log_path); - ~NetLogLogger(); + virtual ~NetLogLogger(); // ThreadSafeObserver implementation: virtual void OnAddEntry(net::NetLog::EventType type, @@ -39,4 +39,3 @@ class NetLogLogger : public ChromeNetLog::ThreadSafeObserver { }; #endif // CHROME_BROWSER_NET_NET_LOG_LOGGER_H_ - |