dbus: ProperySet destructor must be virtual

dbus/property.h

 // Copyright (c) 2012 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.
 
 #ifndef DBUS_PROPERTY_H_
 #define DBUS_PROPERTY_H_
 #pragma once
 
 #include <map>
 #include <string>
@@ skipping 179 matching lines @@
   // remote object that these properties are for, care should be taken to
   // ensure that this object does not outlive the lifetime of the proxy;
   // |interface| specifies the D-Bus interface of these properties, and
   // |property_changed_callback| specifies the callback for when properties
   // are changed, this may be a NULL callback.
   PropertySet(ObjectProxy* object_proxy, const std::string& interface,
               PropertyChangedCallback property_changed_callback);
 
   // 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,
   // and the pointer to the |property| member of the structure. This will
   // call the PropertyBase::Init method.
   void RegisterProperty(const std::string& name, PropertyBase* property);
 
   // Call after construction to connect property change notification
   // signals. Sub-classes may override to use different D-Bus signals.
   void ConnectSignals();
@@ skipping 201 matching lines @@
   T value_;
 
   // Weak pointer factory as D-Bus callbacks may last longer than these
   // objects.
   base::WeakPtrFactory<Property<T> > weak_ptr_factory_;
 };
 
 }  // namespace dbus
 
 #endif  // DBUS_PROPERTY_H_