Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(592)

Unified Diff: skia/ext/refptr_unittest.cc

Issue 11418217: Add skia::RefPtr class to wrap ref counted classes from Skia. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: nit+rebase Created 8 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « skia/ext/refptr.h ('k') | skia/ext/vector_canvas.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: skia/ext/refptr_unittest.cc
diff --git a/skia/ext/refptr_unittest.cc b/skia/ext/refptr_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..1d63ed1b1e929b1041783a24c6f3fd32a854b968
--- /dev/null
+++ b/skia/ext/refptr_unittest.cc
@@ -0,0 +1,125 @@
+// Copyright 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.
+
+#include "skia/ext/refptr.h"
+
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace skia {
+namespace {
+
+TEST(RefPtrTest, ReferenceCounting) {
+ SkRefCnt* ref = new SkRefCnt();
+ EXPECT_EQ(1, ref->getRefCnt());
+
+ {
+ // Adopt the reference from the caller on creation.
+ RefPtr<SkRefCnt> refptr1 = AdoptRef(ref);
+ EXPECT_EQ(1, ref->getRefCnt());
+ EXPECT_EQ(1, refptr1->getRefCnt());
+
+ EXPECT_EQ(ref, &*refptr1);
+ EXPECT_EQ(ref, refptr1.get());
+
+ {
+ // Take a second reference for the second instance.
+ RefPtr<SkRefCnt> refptr2(refptr1);
+ EXPECT_EQ(2, ref->getRefCnt());
+
+ RefPtr<SkRefCnt> refptr3;
+ EXPECT_FALSE(refptr3);
+
+ // Take a third reference for the third instance.
+ refptr3 = refptr1;
+ EXPECT_EQ(3, ref->getRefCnt());
+
+ // Same object, so should have the same refcount.
+ refptr2 = refptr3;
+ EXPECT_EQ(3, ref->getRefCnt());
+
+ // Drop the object from refptr2, so it should lose its reference.
+ EXPECT_TRUE(refptr2);
+ refptr2.clear();
+ EXPECT_EQ(2, ref->getRefCnt());
+
+ EXPECT_FALSE(refptr2);
+ EXPECT_EQ(NULL, refptr2.get());
+
+ EXPECT_TRUE(refptr3);
+ EXPECT_EQ(2, refptr3->getRefCnt());
+ EXPECT_EQ(ref, &*refptr3);
+ EXPECT_EQ(ref, refptr3.get());
+ }
+
+ // Drop a reference when the third object is destroyed.
+ EXPECT_EQ(1, ref->getRefCnt());
+ }
+}
+
+TEST(RefPtrTest, Construct) {
+ SkRefCnt* ref = new SkRefCnt();
+ EXPECT_EQ(1, ref->getRefCnt());
+
+ // Adopt the reference from the caller on creation.
+ RefPtr<SkRefCnt> refptr1(AdoptRef(ref));
+ EXPECT_EQ(1, ref->getRefCnt());
+ EXPECT_EQ(1, refptr1->getRefCnt());
+
+ EXPECT_EQ(ref, &*refptr1);
+ EXPECT_EQ(ref, refptr1.get());
+
+ RefPtr<SkRefCnt> refptr2(refptr1);
+ EXPECT_EQ(2, ref->getRefCnt());
+}
+
+TEST(RefPtrTest, DeclareAndAssign) {
+ SkRefCnt* ref = new SkRefCnt();
+ EXPECT_EQ(1, ref->getRefCnt());
+
+ // Adopt the reference from the caller on creation.
+ RefPtr<SkRefCnt> refptr1 = AdoptRef(ref);
+ EXPECT_EQ(1, ref->getRefCnt());
+ EXPECT_EQ(1, refptr1->getRefCnt());
+
+ EXPECT_EQ(ref, &*refptr1);
+ EXPECT_EQ(ref, refptr1.get());
+
+ RefPtr<SkRefCnt> refptr2 = refptr1;
+ EXPECT_EQ(2, ref->getRefCnt());
+}
+
+TEST(RefPtrTest, Assign) {
+ SkRefCnt* ref = new SkRefCnt();
+ EXPECT_EQ(1, ref->getRefCnt());
+
+ // Adopt the reference from the caller on creation.
+ RefPtr<SkRefCnt> refptr1;
+ refptr1 = AdoptRef(ref);
+ EXPECT_EQ(1, ref->getRefCnt());
+ EXPECT_EQ(1, refptr1->getRefCnt());
+
+ EXPECT_EQ(ref, &*refptr1);
+ EXPECT_EQ(ref, refptr1.get());
+
+ RefPtr<SkRefCnt> refptr2;
+ refptr2 = refptr1;
+ EXPECT_EQ(2, ref->getRefCnt());
+}
+
+class Subclass : public SkRefCnt {};
+
+TEST(RefPtrTest, Upcast) {
+ RefPtr<Subclass> child = AdoptRef(new Subclass());
+ EXPECT_EQ(1, child->getRefCnt());
+
+ RefPtr<SkRefCnt> parent = child;
+ EXPECT_TRUE(child);
+ EXPECT_TRUE(parent);
+
+ EXPECT_EQ(2, child->getRefCnt());
+ EXPECT_EQ(2, parent->getRefCnt());
+}
+
+} // namespace
+} // namespace skia
« no previous file with comments | « skia/ext/refptr.h ('k') | skia/ext/vector_canvas.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698