diff options
author | shess@chromium.org <shess@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-10-13 10:16:17 +0000 |
---|---|---|
committer | shess@chromium.org <shess@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-10-13 10:16:17 +0000 |
commit | af66dec755aabca3553ef796213e691d282c81e3 (patch) | |
tree | a2a17180531b4d21bdc5659b834b8364d8bb54b8 /base/memory | |
parent | 587869384ed4cf74d88974558500505f966adef0 (diff) | |
download | chromium_src-af66dec755aabca3553ef796213e691d282c81e3.zip chromium_src-af66dec755aabca3553ef796213e691d282c81e3.tar.gz chromium_src-af66dec755aabca3553ef796213e691d282c81e3.tar.bz2 |
[Mac] base::scoped_nsobject<>::autorelease()
To give up ownership of this object to the autorelease pool:
scoped_nsobject<MyClass> obj;
you could write:
[[obj retain] autorelease];
obj.reset();
or:
[obj.release() autorelease];
This CL makes it more explicit:
obj.autorelease();
BUG=none
Review URL: https://chromiumcodereview.appspot.com/11066079
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@161750 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/memory')
-rw-r--r-- | base/memory/scoped_nsobject.h | 5 | ||||
-rw-r--r-- | base/memory/scoped_nsobject_unittest.mm | 11 |
2 files changed, 16 insertions, 0 deletions
diff --git a/base/memory/scoped_nsobject.h b/base/memory/scoped_nsobject.h index 0656dbe..5d98e3f 100644 --- a/base/memory/scoped_nsobject.h +++ b/base/memory/scoped_nsobject.h @@ -96,6 +96,11 @@ class scoped_nsprotocol { return temp; } + // Shift reference to the autorelease pool to be released later. + NST autorelease() { + return [release() autorelease]; + } + private: NST object_; }; diff --git a/base/memory/scoped_nsobject_unittest.mm b/base/memory/scoped_nsobject_unittest.mm index e61c060..377a3de 100644 --- a/base/memory/scoped_nsobject_unittest.mm +++ b/base/memory/scoped_nsobject_unittest.mm @@ -5,6 +5,7 @@ #include <vector> #include "base/basictypes.h" +#include "base/mac/scoped_nsautorelease_pool.h" #include "base/memory/scoped_nsobject.h" #include "testing/gtest/include/gtest/gtest.h" @@ -40,6 +41,16 @@ TEST(ScopedNSObjectTest, ScopedNSObject) { ASSERT_TRUE(p1 != p5.get()); ASSERT_FALSE(p1 == p5); ASSERT_FALSE(p1 == p5.get()); + + scoped_nsobject<NSObject> p6 = p1; + ASSERT_EQ(3u, [p6 retainCount]); + { + base::mac::ScopedNSAutoreleasePool pool; + p6.autorelease(); + ASSERT_EQ(nil, p6.get()); + ASSERT_EQ(3u, [p1 retainCount]); + } + ASSERT_EQ(2u, [p1 retainCount]); } TEST(ScopedNSObjectTest, ScopedNSObjectInContainer) { |