Stop doing at-exit cleanup of lazy pointers.

src/core/SkLazyPtr.h

 /*
  * Copyright 2014 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef SkLazyPtr_DEFINED
 #define SkLazyPtr_DEFINED
 
@@ skipping 23 matching lines @@
  *
  *  Foo* CreateFoo(int i) { return ...; }
  *  Foo* GetCachedFoo(Foo::Enum enumVal) {
  *      SK_DECLARE_STATIC_LAZY_PTR_ARRAY(Foo, Foo::kEnumCount, cachedFoos, CreateFoo);
  *      return cachedFoos[enumVal];
  *  }
  *
  *
  *  You can think of SK_DECLARE_STATIC_LAZY_PTR as a cheaper specialization of
  *  SkOnce.  There is no mutex or extra storage used past the pointer itself.
- *  In debug mode, each lazy pointer will be cleaned up at process exit so we
- *  can check that we've not leaked or freed them early.
  *
  *  We may call Create more than once, but all threads will see the same pointer
  *  returned from get().  Any extra calls to Create will be cleaned up.
  *
  *  These macros must be used in a global scope, not in function scope or as a class member.
  */
 
 #define SK_DECLARE_STATIC_LAZY_PTR(T, name, ...) \
     namespace {} static Private::SkLazyPtr<T, ##__VA_ARGS__> name
 
@@ skipping 51 matching lines @@
 template <typename T, T* (*Create)() = sk_new<T>, void (*Destroy)(T*) = sk_delete<T> >
 class SkLazyPtr {
 public:
     T* get() {
         // If fPtr has already been filled, we need a consume barrier when loading it.
         // If not, we need a release barrier when setting it.  try_cas will do that.
         T* ptr = (T*)sk_consume_load(&fPtr);
         return ptr ? ptr : try_cas<T*, Destroy>(&fPtr, Create());
     }
 
-#ifdef SK_DEVELOPER
-    // FIXME: We know we leak refs on some classes.  For now, let them leak.
-    void cleanup(SkFontConfigInterfaceDirect*) {}
-    template <typename U> void cleanup(U* ptr) { Destroy(ptr); }
-
-    ~SkLazyPtr() {
-        this->cleanup((T*)fPtr);
-        fPtr = NULL;
-    }
-#endif
-
 private:
     void* fPtr;
 };
 
 template <typename T> T* sk_new_arg(int i) { return SkNEW_ARGS(T, (i)); }
 
 // This has no constructor and must be zero-initalized (the macro above does this).
 template <typename T, int N, T* (*Create)(int) = sk_new_arg<T>, void (*Destroy)(T*) = sk_delete<T> >
 class SkLazyPtrArray {
 public:
     T* operator[](int i) {
         SkASSERT(i >= 0 && i < N);
         // If fPtr has already been filled, we need an consume barrier when loading it.
         // If not, we need a release barrier when setting it.  try_cas will do that.
         T* ptr = (T*)sk_consume_load(&fArray[i]);
         return ptr ? ptr : try_cas<T*, Destroy>(&fArray[i], Create(i));
     }
 
-#ifdef SK_DEVELOPER
-    ~SkLazyPtrArray() {
-        for (int i = 0; i < N; i++) {
-            Destroy((T*)fArray[i]);
-            fArray[i] = NULL;
-        }
-    }
-#endif
-
 private:
     void* fArray[N];
 };
 
 }  // namespace Private
 
 #endif//SkLazyPtr_DEFINED