Split out GrBitmapTextContext from GrTextContext.

include/gpu/GrTextContext.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 GrTextContext_DEFINED
 #define GrTextContext_DEFINED
 
 #include "GrContext.h"
 #include "GrGlyph.h"
 #include "GrPaint.h"
 
 class GrContext;
-class GrTextStrike;
+class GrDrawTarget;
 class GrFontScaler;
-class GrDrawTarget;
 
+/*
+ * This class wraps the state for a single text render
+ */
 class GrTextContext {
 public:
     GrTextContext(GrContext*, const GrPaint&);

[bsalomon, 2013/10/14 19:32:47]

private?

[jvanverth1, 2013/10/16 17:56:29]

Can't make it private (the child classes need access), but made protected.

-    ~GrTextContext();
+    virtual ~GrTextContext();
 
-    void drawPackedGlyph(GrGlyph::PackedID, GrFixed left, GrFixed top,
-                         GrFontScaler*);
+    virtual void drawPackedGlyph(GrGlyph::PackedID, GrFixed left, GrFixed top,
+                                 GrFontScaler*) = 0;
 
-    void flush();   // optional; automatically called by destructor
+    virtual void flush() = 0;   // optional

[bsalomon, 2013/10/14 19:32:47]

the comment now seems weird... as though overriding it were optional, which
isn't case. Are there external callers? If not let's just make it private.

[jvanverth1, 2013/10/16 17:56:29]

Just removed it, no one seems to be using it.

 
-private:
-    GrPaint         fPaint;
-    GrContext*      fContext;
-    GrDrawTarget*   fDrawTarget;
+protected:
+    GrPaint                fPaint;
+    GrContext*             fContext;
+    GrDrawTarget*          fDrawTarget;
 
-    GrFontScaler*   fScaler;
-    GrTextStrike*   fStrike;
-
-    inline void flushGlyphs();
-    void setupDrawTarget();
-
-    enum {
-        kMinRequestedGlyphs      = 1,
-        kDefaultRequestedGlyphs  = 64,
-        kMinRequestedVerts       = kMinRequestedGlyphs * 4,
-        kDefaultRequestedVerts   = kDefaultRequestedGlyphs * 4,
-    };
-
-    SkPoint*                fVertices;
-    int32_t                 fMaxVertices;
-    GrTexture*              fCurrTexture;
-    int                     fCurrVertex;
-
-    SkIRect                 fClipRect;
-    GrContext::AutoMatrix   fAutoMatrix;
+    SkIRect                fClipRect;
+    GrContext::AutoMatrix  fAutoMatrix;
 };
 
 #endif