drawinfo carries bufferinfo

include/gpu/GrGpuResourceRef.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 GrGpuResourceRef_DEFINED
 #define GrGpuResourceRef_DEFINED
 
@@ skipping 141 matching lines @@
 private:
     typedef GrGpuResourceRef INHERITED;
 };
 
 /**
  * This is similar to GrTGpuResourceRef but can only be in the pending IO state. It never owns a
  * ref.
  */
 template <typename T, GrIOType IO_TYPE> class GrPendingIOResource : SkNoncopyable {
 public:
-    GrPendingIOResource(T* resource) : fResource(resource) {
-        if (NULL != fResource) {
+    GrPendingIOResource(T* resource = NULL) {
+        this->set(resource);
+    }
+
+    void set(T* resource) {

[bsalomon, 2014/11/18 20:48:31]

Shouldn't this check whether the resource was already set and if so release the
pending io?

Also, this looks an awful lot like SkAutoTUnref which to me argues for nameing
the reset() rather than set().

+        fResource = resource;
+        if (NULL != resource) {
             switch (IO_TYPE) {
                 case kRead_GrIOType:
-                    fResource->addPendingRead();
+                    resource->addPendingRead();
                     break;
                 case kWrite_GrIOType:
-                    fResource->addPendingWrite();
+                    resource->addPendingWrite();
                     break;
                 case kRW_GrIOType:
-                    fResource->addPendingRead();
-                    fResource->addPendingWrite();
+                    resource->addPendingRead();
+                    resource->addPendingWrite();
                     break;
             }
         }
     }
 
     ~GrPendingIOResource() {
         if (NULL != fResource) {
             switch (IO_TYPE) {
                 case kRead_GrIOType:
                     fResource->completedRead();
                     break;
                 case kWrite_GrIOType:
                     fResource->completedWrite();
                     break;
                 case kRW_GrIOType:
                     fResource->completedRead();
                     fResource->completedWrite();
                     break;
             }
         }
     }
 
     T* get() const { return fResource; }
 
 private:
     T* fResource;
 };
 #endif