Make GrGpuResources register with GrResourceCache2 after fully constructed.

include/gpu/GrGpuResource.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 GrGpuResource_DEFINED
 #define GrGpuResource_DEFINED
 
@@ skipping 72 matching lines @@
     GrResourceCacheEntry* getCacheEntry() { return fCacheEntry; }
 
     /**
      * Gets an id that is unique for this GrCacheable object. It is static in that it does
      * not change when the content of the GrCacheable object changes. This will never return
      * 0.
      */
     uint32_t getUniqueID() const { return fUniqueID; }
 
 protected:
+

[robertphillips, 2014/08/26 18:49:25]

"by every GrGpuObject" -> "at the end of every GrGpuObject-derived class'
constructor" ?
object (typo)

[bsalomon, 2014/08/26 20:56:45]

Hm, there are intermediate derived classes that shouldn't call this. That's why
I called out "fully constructed".

+    // This must be called by every GrGpuObject. It should be called once the objet is fully

[robertphillips, 2014/08/26 18:49:25]

constructor (typo)

[bsalomon, 2014/08/26 20:56:45]

Done.

+    // initialized (i.e. not in a base class construtor).
+    void registerWithCache();
+
     GrGpuResource(GrGpu*, bool isWrapped);
     virtual ~GrGpuResource();
 
     bool isInCache() const { return NULL != fCacheEntry; }
 
     GrGpu* getGpu() const { return fGpu; }
 
     // Derived classes should always call their parent class' onRelease
     // and onAbandon methods in their overrides.
     virtual void onRelease() {};
@@ skipping 35 matching lines @@
     uint32_t                fFlags;
 
     mutable int32_t         fRefCnt;
     GrResourceCacheEntry*   fCacheEntry;  // NULL if not in cache
     const uint32_t          fUniqueID;
 
     typedef SkNoncopyable INHERITED;
 };
 
 #endif