Add SkNVRefCnt, prune down SkPicture's size

include/core/SkRefCnt.h

 
 /*
  * Copyright 2006 The Android Open Source Project
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 
 #ifndef SkRefCnt_DEFINED
@@ skipping 228 matching lines @@
 private:
     T*  fObj;
 };
 // Can't use the #define trick below to guard a bare SkAutoTUnref(...) because it's templated. :(
 
 class SkAutoUnref : public SkAutoTUnref<SkRefCnt> {
 public:
     SkAutoUnref(SkRefCnt* obj) : SkAutoTUnref<SkRefCnt>(obj) {}
 };
 #define SkAutoUnref(...) SK_REQUIRE_LOCAL_VAR(SkAutoUnref)
 

[tomhudson, 2014/11/19 18:59:06]

Can we put something like a short version of the CL description here as a class
header comment? Since there's an abbreviation in the class name, the purpose of
the class isn't clear, and even spelling out the abbreviation leaves it
inobvious.

[mtklein, 2014/11/19 19:19:14]

Done.

+template <typename Derived>
+class SkNVRefCnt : SkNoncopyable {
+public:
+    SkNVRefCnt() : fRefCnt(1) {}
+
+    // Implementation is pretty much the same as SkRefCntBase. All required barriers are the same:
+    //   - unique() needs acquire when it returns true, and no barrier if it returns false;
+    //   - ref() doesn't need any barrier;
+    //   - unref() needs a release barrier, and an acquire if it's going to call delete.
+
+    bool unique() const { return 1 == sk_acquire_load(&fRefCnt); }
+    void    ref() const { sk_atomic_inc(&fRefCnt); }
+    void  unref() const { if (1 == sk_atomic_dec(&fRefCnt)) { SkDELETE((const Derived*)this); } }
+private:
+    mutable int32_t fRefCnt;
+};
+
 #endif