SkRecord: text blobs

src/core/SkRemote.h

Index: src/core/SkRemote.h
diff --git a/src/core/SkRemote.h b/src/core/SkRemote.h
index b33309449c78483b6f0ca17bad0b1ce96883c231..fcfa1f94a29d50ecbd50ca445aa977ce8624dde3 100644
--- a/src/core/SkRemote.h
+++ b/src/core/SkRemote.h
@@ -17,6 +17,7 @@ class SkCanvas;
 class SkMatrix;
 class SkPath;
 class SkShader;
+class SkTextBlob;
 class SkXfermode;
 
 // TODO: document
@@ -65,19 +66,20 @@ namespace SkRemote {
     struct Encoder {
         virtual ~Encoder() {}
 
-        virtual ID define(const SkMatrix&) = 0;
-        virtual ID define(const Misc&)     = 0;
-        virtual ID define(const SkPath&)   = 0;
-        virtual ID define(const Stroke&)   = 0;
-        virtual ID define(SkPathEffect*)   = 0;
-        virtual ID define(SkShader*)       = 0;
-        virtual ID define(SkXfermode*)     = 0;
-        virtual ID define(SkMaskFilter*)   = 0;
-        virtual ID define(SkColorFilter*)  = 0;
-        virtual ID define(SkRasterizer*)   = 0;
-        virtual ID define(SkDrawLooper*)   = 0;
-        virtual ID define(SkImageFilter*)  = 0;
-        virtual ID define(SkAnnotation*)   = 0;
+        virtual ID define(const SkMatrix&)   = 0;
+        virtual ID define(const Misc&)       = 0;
+        virtual ID define(const SkPath&)     = 0;
+        virtual ID define(const Stroke&)     = 0;
+        virtual ID define(const SkTextBlob*) = 0;
+        virtual ID define(SkPathEffect*)     = 0;
+        virtual ID define(SkShader*)         = 0;
+        virtual ID define(SkXfermode*)       = 0;
+        virtual ID define(SkMaskFilter*)     = 0;
+        virtual ID define(SkColorFilter*)    = 0;
+        virtual ID define(SkRasterizer*)     = 0;
+        virtual ID define(SkDrawLooper*)     = 0;
+        virtual ID define(SkImageFilter*)    = 0;
+        virtual ID define(SkAnnotation*)     = 0;
 
         virtual void undefine(ID) = 0;
 
@@ -93,9 +95,11 @@ namespace SkRemote {
 
         virtual void setMatrix(ID matrix) = 0;
 
-        virtual void   clipPath(ID path, SkRegion::Op, bool aa) = 0;
-        virtual void   fillPath(ID path, CommonIDs)             = 0;
-        virtual void strokePath(ID path, CommonIDs, ID stroke)  = 0;
+        virtual void   clipPath(ID path, SkRegion::Op, bool aa)           = 0;
+        virtual void   fillPath(ID path, CommonIDs)                       = 0;
+        virtual void strokePath(ID path, CommonIDs, ID stroke)            = 0;
+        virtual void   fillText(ID text, ID offset, CommonIDs)            = 0;

[hal.canary, 2015/11/04 15:14:44]

I really don't like this. Offset should be a SkPoint value.  64 bits without the
need to define. 

[mtklein, 2015/11/04 15:31:51]

Done.

+        virtual void strokeText(ID text, ID offset, CommonIDs, ID stroke) = 0;
     };
 
     // None of these factories take ownership of their arguments.