Steal refs from other TextBatch in onCombineIfPossible

src/gpu/GrAtlasTextContext.cpp

 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 #include "GrAtlasTextContext.h"
 
 #include "GrBatch.h"
 #include "GrBatchFontCache.h"
@@ skipping 78 matching lines @@
 
 static inline GrColor skcolor_to_grcolor_nopremultiply(SkColor c) {
     unsigned r = SkColorGetR(c);
     unsigned g = SkColorGetG(c);
     unsigned b = SkColorGetB(c);
     return GrColorPackRGBA(r, g, b, 0xff);
 }
 
 };
 
-// TODO
-// Distance field text in textblobs
-
 GrAtlasTextContext::GrAtlasTextContext(GrContext* context,
                                        GrDrawContext* drawContext,
                                        const SkSurfaceProps& surfaceProps)
     : INHERITED(context, drawContext, surfaceProps)
     , fDistanceAdjustTable(SkNEW(DistanceAdjustTable)) {
     // We overallocate vertices in our textblobs based on the assumption that A8 has the greatest
     // vertexStride
     SK_COMPILE_ASSERT(kGrayTextVASize >= kColorTextVASize && kGrayTextVASize >= kLCDTextVASize,
                       vertex_attribute_changed);
     fCurrStrike = NULL;
@@ skipping 1819 matching lines @@
         // is greater than or equal to the number of geometries (and at least
         // kMinGeometryAllocated).
         int newAllocSize = GrNextPow2(newGeoCount);
         int currAllocSize = SkTMax<int>(kMinGeometryAllocated, GrNextPow2(fGeoCount));
 
         if (newAllocSize > currAllocSize) {
             fGeoData.realloc(newAllocSize);
         }
 
         memcpy(&fGeoData[fGeoCount], that->fGeoData.get(), that->fGeoCount * sizeof(Geometry));
-        for (int i = fGeoCount; i < newGeoCount; ++i) {
-            fGeoData[i].fBlob->ref();
+        // We steal the ref on the blobs from the other TextBatch and set its count to 0 so that
+        // it doesn't try to unref them.
+#ifdef SK_DEBUG
+        for (int i = 0; i < that->fGeoCount; ++i) {
+            that->fGeoData.get()[i].fBlob = (Blob*)0x1;
         }
+#endif
+        that->fGeoCount = 0;
         fGeoCount = newGeoCount;
 
         this->joinBounds(that->bounds());
         return true;
     }
 
     // TODO just use class params
     // TODO trying to figure out why lcd is so whack
     GrGeometryProcessor* setupDfProcessor(const SkMatrix& viewMatrix, SkColor filteredColor,
                                           GrColor color, GrTexture* texture) {
@@ skipping 316 matching lines @@
             gTextContext->createDrawTextBlob(rt, clip, grPaint, skPaint, viewMatrix, text,
                                              static_cast<size_t>(textLen), 0, 0, noClip));
 
     SkScalar transX = static_cast<SkScalar>(random->nextU());
     SkScalar transY = static_cast<SkScalar>(random->nextU());
     const GrAtlasTextBlob::Run::SubRunInfo& info = blob->fRuns[0].fSubRunInfo[0];
     return gTextContext->createBatch(blob, info, textLen, 0, 0, color, transX, transY, skPaint);
 }
 
 #endif