| commit | 64ccf236e6657cf0a85506c56d4ba226fdea4765 | [log] [tgz] |
|---|---|---|
| author | mgiuca <[email protected]> | Mon Nov 10 06:44:23 2014 |
| committer | Commit bot <[email protected]> | Mon Nov 10 06:44:47 2014 |
| tree | 32a2887d6f35225748b3ae93193fdb306575bdd0 | |
| parent | d2b6be4e2ef7a38d503deb1f89ca96b16c85effb [diff] [blame] |
ObserverList::HasObserver now takes a const pointer. Updated all clients that wrap HasObserver and expose it to also take a const pointer. This change avoids further propagation of non-constness (which was previously required when using HasObserver methods). BUG=429588 [email protected],[email protected],[email protected],[email protected],[email protected],[email protected] Review URL: https://codereview.chromium.org/697953002 Cr-Commit-Position: refs/heads/master@{#303428}
diff --git a/base/observer_list_unittest.cc b/base/observer_list_unittest.cc index 11f59be..65ef934 100644 --- a/base/observer_list_unittest.cc +++ b/base/observer_list_unittest.cc
@@ -182,6 +182,9 @@ observer_list.AddObserver(&a); observer_list.AddObserver(&b); + EXPECT_TRUE(observer_list.HasObserver(&a)); + EXPECT_FALSE(observer_list.HasObserver(&c)); + FOR_EACH_OBSERVER(Foo, observer_list, Observe(10)); observer_list.AddObserver(&evil);