From 3690ebe09a8c3cea0fd7d9ece8f5b8d8ebc65c19 Mon Sep 17 00:00:00 2001 From: "hans@chromium.org" Date: Wed, 25 May 2011 09:08:19 +0000 Subject: 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 --- webkit/appcache/appcache_host.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'webkit/appcache/appcache_host.h') diff --git a/webkit/appcache/appcache_host.h b/webkit/appcache/appcache_host.h index a06cdd3..0f8ac47 100644 --- a/webkit/appcache/appcache_host.h +++ b/webkit/appcache/appcache_host.h @@ -48,7 +48,7 @@ class AppCacheHost : public AppCacheStorage::Delegate, AppCacheHost(int host_id, AppCacheFrontend* frontend, AppCacheService* service); - ~AppCacheHost(); + virtual ~AppCacheHost(); // Adds/removes an observer, the AppCacheHost does not take // ownership of the observer. -- cgit v1.1