From 695092fd7614c257866347f049e28ec327d7a3d3 Mon Sep 17 00:00:00 2001 From: "ziadh@chromium.org" Date: Mon, 2 Aug 2010 16:34:16 +0000 Subject: Add undeclared virtual destructors part 5 Preventative maintenance for abstract classes that do not declare virtual destructors. Base classes that do not declare their destructors as virtual could potentially lead to memory leaks. These files were discovered using the -Wnon-virtual-dtor flag in g++. r=jar BUG=47469 Review URL: http://codereview.chromium.org/3080013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54540 0039d316-1c4b-4281-b951-d872f2087c98 --- base/process_util.h | 3 +++ 1 file changed, 3 insertions(+) (limited to 'base/process_util.h') diff --git a/base/process_util.h b/base/process_util.h index 4cb954b..dbb0f13 100644 --- a/base/process_util.h +++ b/base/process_util.h @@ -254,6 +254,9 @@ class ProcessFilter { // Returns true to indicate set-inclusion and false otherwise. This method // should not have side-effects and should be idempotent. virtual bool Includes(const ProcessEntry& entry) const = 0; + + protected: + virtual ~ProcessFilter() {} }; // Returns the number of processes on the machine that are running from the -- cgit v1.1