diff options
| author | caitkp@chromium.org <caitkp@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-08-26 19:15:33 +0000 |
|---|---|---|
| committer | caitkp@chromium.org <caitkp@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-08-26 19:15:33 +0000 |
| commit | 5e362e4700605d749f57cfe361449a6f067541ff (patch) | |
| tree | 120f168a2a56cc253d457c76b857e32dd0b49efb /chromeos/dbus/shill_client_helper.cc | |
| parent | 6ac748db5fe76c06d1560a040dac8a018b15d2a1 (diff) | |
| download | chromium_src-5e362e4700605d749f57cfe361449a6f067541ff.zip chromium_src-5e362e4700605d749f57cfe361449a6f067541ff.tar.gz chromium_src-5e362e4700605d749f57cfe361449a6f067541ff.tar.bz2 | |
Switch ObserverList::size() to ObserverList::might_have_observers() Pt.1
This is in preparation for making ObserverList::size() protected (see bug for details).
BUG=278526
TEST=no change
TBR=darin@chromium.org
Review URL: https://chromiumcodereview.appspot.com/23190045
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@219580 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chromeos/dbus/shill_client_helper.cc')
| -rw-r--r-- | chromeos/dbus/shill_client_helper.cc | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/chromeos/dbus/shill_client_helper.cc b/chromeos/dbus/shill_client_helper.cc index 61dcb4d..a0af3b8 100644 --- a/chromeos/dbus/shill_client_helper.cc +++ b/chromeos/dbus/shill_client_helper.cc @@ -180,9 +180,8 @@ ShillClientHelper::ShillClientHelper(dbus::Bus* bus, } ShillClientHelper::~ShillClientHelper() { - LOG_IF(ERROR, observer_list_.size() != 0u) - << "ShillClientHelper destroyed with active observers: " - << observer_list_.size(); + LOG_IF(ERROR, observer_list_.might_have_observers()) + << "ShillClientHelper destroyed with active observers"; } void ShillClientHelper::AddPropertyChangedObserver( @@ -203,7 +202,7 @@ void ShillClientHelper::RemovePropertyChangedObserver( void ShillClientHelper::MonitorPropertyChanged( const std::string& interface_name) { - if (observer_list_.size() > 0) { + if (observer_list_.might_have_observers()) { // Effectively monitor the PropertyChanged now. MonitorPropertyChangedInternal(interface_name); } else { |