BitmapTextBatch and BitmapTextBlob

src/gpu/GrGlyph.h

 /*
  * Copyright 2010 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef GrGlyph_DEFINED
 #define GrGlyph_DEFINED
 
+#include "GrBatchAtlas.h"
 #include "GrRect.h"
 #include "GrTypes.h"
 
 #include "SkChecksum.h"
 #include "SkPath.h"
 
 class GrPlot;
 
 /*  Need this to be quad-state:
     - complete w/ image
     - just metrics
     - failed to get image, but has metrics
     - failed to get metrics
  */
 struct GrGlyph {
     enum MaskStyle {
         kCoverage_MaskStyle,
         kDistance_MaskStyle
     };
     
     typedef uint32_t PackedID;
 
+    // TODO either plot or AtlasID will be valid, not both

[bsalomon, 2015/03/24 16:27:53]

Is this b/c we have both batch and nonbatch impls of text rendering?

[joshualitt, 2015/03/25 14:13:37]

yes

+    GrBatchAtlas::AtlasID fID;
     GrPlot*      fPlot;
     SkPath*      fPath;
     PackedID     fPackedID;
     GrMaskFormat fMaskFormat;
     GrIRect16    fBounds;
     SkIPoint16   fAtlasLocation;
 
     void init(GrGlyph::PackedID packed, const SkIRect& bounds, GrMaskFormat format) {
+        fID = GrBatchAtlas::kInvalidAtlasID;
         fPlot = NULL;
         fPath = NULL;
         fPackedID = packed;
         fBounds.set(bounds);
         fMaskFormat = format;
         fAtlasLocation.set(0, 0);
     }
 
     void free() {
         if (fPath) {
@@ skipping 40 matching lines @@
     static inline const GrGlyph::PackedID& GetKey(const GrGlyph& glyph) {
         return glyph.fPackedID;
     }
 
     static inline uint32_t Hash(GrGlyph::PackedID key) {
         return SkChecksum::Murmur3(&key, sizeof(key));
     }
 };
 
 #endif