commit 82e082a3a037ab638c187f72dfb24951341f5d13
author keybuk@chromium.org <keybuk@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Fri Feb 17 10:01:18 2012 +0900
committer Qijiang Fan <fqj@google.com> Thu Jun 04 12:24:52 2020 +0900
tree 0f6fe783da863c29ec60da92ded22f272976ae25
parent 93255477756a5d8e671490616d7b9a031fda9eaf

dbus: ProperySet destructor must be virtual

clang requires us to have a destructor for this "complex" class,
but that destructor will need to be virtual later.

BUG=none
TEST=git try -b linux_chromeos_clang

Change-Id: Ia0efd34f205ab090cd7f2092d47fc0288b1f4e0c


Review URL: http://codereview.chromium.org/9420027

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@122417 0039d316-1c4b-4281-b951-d872f2087c98


CrOS-Libchrome-Original-Commit: 8acfaf570e7b0af382825605ec5134ebc97adfc8

dbus/property.h

diff --git a/dbus/property.h b/dbus/property.h
index 47b118f..e90d2a2 100644
--- a/dbus/property.h
+++ b/dbus/property.h
@@ -197,7 +197,7 @@
 
   // Destructor; we don't hold on to any references or memory that needs
   // explicit clean-up, but clang thinks we might.
-  ~PropertySet();
+  virtual ~PropertySet();
 
   // Registers a property, generally called from the subclass constructor;
   // pass the |name| of the property as used in method calls and signals,