diff options
author | akalin@chromium.org <akalin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-06-07 09:05:12 +0000 |
---|---|---|
committer | akalin@chromium.org <akalin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-06-07 09:05:12 +0000 |
commit | 8ee0492feeb4f8b10b965eb0603531acf51fec23 (patch) | |
tree | 0fe3ab210bd80f2664084876454d6e657e52183a /sync/internal_api | |
parent | 3ed32b6a198b97b8918123f1c45e224903f0db06 (diff) | |
download | chromium_src-8ee0492feeb4f8b10b965eb0603531acf51fec23.zip chromium_src-8ee0492feeb4f8b10b965eb0603531acf51fec23.tar.gz chromium_src-8ee0492feeb4f8b10b965eb0603531acf51fec23.tar.bz2 |
Make some more code use "if (weak_ptr)" style
This is a revert of r204036 and part of r203996.
BUG=245942
TBR=darin@chromium.org
Review URL: https://chromiumcodereview.appspot.com/16571012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@204773 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'sync/internal_api')
-rw-r--r-- | sync/internal_api/public/util/weak_handle.h | 10 | ||||
-rw-r--r-- | sync/internal_api/public/util/weak_handle_unittest.cc | 4 |
2 files changed, 7 insertions, 7 deletions
diff --git a/sync/internal_api/public/util/weak_handle.h b/sync/internal_api/public/util/weak_handle.h index 24fccbc..c299be7 100644 --- a/sync/internal_api/public/util/weak_handle.h +++ b/sync/internal_api/public/util/weak_handle.h @@ -208,7 +208,7 @@ class WeakHandleCore template <typename U> void DoCall0(void (U::*fn)(void)) const { CHECK(IsOnOwnerThread()); - if (!Get().get()) { + if (!Get()) { return; } (Get().get()->*fn)(); @@ -218,7 +218,7 @@ class WeakHandleCore void DoCall1(void (U::*fn)(A1), typename ParamTraits<A1>::ForwardType a1) const { CHECK(IsOnOwnerThread()); - if (!Get().get()) { + if (!Get()) { return; } (Get().get()->*fn)(a1); @@ -229,7 +229,7 @@ class WeakHandleCore typename ParamTraits<A1>::ForwardType a1, typename ParamTraits<A2>::ForwardType a2) const { CHECK(IsOnOwnerThread()); - if (!Get().get()) { + if (!Get()) { return; } (Get().get()->*fn)(a1, a2); @@ -241,7 +241,7 @@ class WeakHandleCore typename ParamTraits<A2>::ForwardType a2, typename ParamTraits<A3>::ForwardType a3) const { CHECK(IsOnOwnerThread()); - if (!Get().get()) { + if (!Get()) { return; } (Get().get()->*fn)(a1, a2, a3); @@ -254,7 +254,7 @@ class WeakHandleCore typename ParamTraits<A3>::ForwardType a3, typename ParamTraits<A4>::ForwardType a4) const { CHECK(IsOnOwnerThread()); - if (!Get().get()) { + if (!Get()) { return; } (Get().get()->*fn)(a1, a2, a3, a4); diff --git a/sync/internal_api/public/util/weak_handle_unittest.cc b/sync/internal_api/public/util/weak_handle_unittest.cc index c8ba1ef..56dfe7f 100644 --- a/sync/internal_api/public/util/weak_handle_unittest.cc +++ b/sync/internal_api/public/util/weak_handle_unittest.cc @@ -97,7 +97,7 @@ TEST_F(WeakHandleTest, InitializedAfterDestroy) { h = b.AsWeakHandle(); } EXPECT_TRUE(h.IsInitialized()); - EXPECT_FALSE(h.Get().get()); + EXPECT_FALSE(h.Get()); } TEST_F(WeakHandleTest, InitializedAfterInvalidate) { @@ -105,7 +105,7 @@ TEST_F(WeakHandleTest, InitializedAfterInvalidate) { WeakHandle<Base> h = b.AsWeakHandle(); b.Kill(); EXPECT_TRUE(h.IsInitialized()); - EXPECT_FALSE(h.Get().get()); + EXPECT_FALSE(h.Get()); } TEST_F(WeakHandleTest, Call) { |