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

2 files changed, 11 insertions, 4 deletions

diff --git a/chrome/browser/webdata/web_database_table.cc b/chrome/browser/webdata/web_database_table.cc
new file mode 100644
index 0000000..1dada9d
--- /dev/null
+++ b/chrome/browser/webdata/web_database_table.cc
@@ -0,0 +1,10 @@
+// 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.
+
+#include "chrome/browser/webdata/web_database_table.h"
+
+WebDatabaseTable::~WebDatabaseTable() {
+  db_ = NULL;
+  meta_table_ = NULL;
+}
diff --git a/chrome/browser/webdata/web_database_table.h b/chrome/browser/webdata/web_database_table.h
index 5bb95b9..ea3c5ec 100644
--- a/chrome/browser/webdata/web_database_table.h
+++ b/chrome/browser/webdata/web_database_table.h
@@ -16,10 +16,7 @@ class WebDatabaseTable {
  protected:
   explicit WebDatabaseTable(sql::Connection* db, sql::MetaTable* meta_table)
       : db_(db), meta_table_(meta_table) {}
-  virtual ~WebDatabaseTable() {
-    db_ = NULL;
-    meta_table_ = NULL;
-  };
+  virtual ~WebDatabaseTable();
 
   // Attempts to initialize the table and returns true if successful.
   virtual bool Init() = 0;