@shopify/react-native-skia 0.1.192 → 0.1.193

package/cpp/skia/include/config/SkUserConfig.h

@@ -1,4 +1,3 @@
-
 /*
  * Copyright 2006 The Android Open Source Project
  *
@@ -6,7 +5,6 @@
  * found in the LICENSE file.
  */
 
-
 #ifndef SkUserConfig_DEFINED
 #define SkUserConfig_DEFINED
 
@@ -16,9 +14,9 @@
 
     Below are optional defines that add, subtract, or change default behavior
     in Skia. Your port can locally edit this file to enable/disable flags as
-    you choose, or these can be delared on your command line (i.e. -Dfoo).
+    you choose, or these can be declared on your command line (i.e. -Dfoo).
 
-    By default, this include file will always default to having all of the flags
+    By default, this #include file will always default to having all the flags
     commented out, so including it will have no effect.
 */
 
@@ -32,58 +30,92 @@
     By default, these mutually exclusive flags are defined in SkTypes.h,
     based on the presence or absence of NDEBUG, but that decision can be changed
     here.
- */
+*/
 //#define SK_DEBUG
 //#define SK_RELEASE
 
 /*  To write debug messages to a console, skia will call SkDebugf(...) following
     printf conventions (e.g. const char* format, ...). If you want to redirect
     this to something other than printf, define yours here
- */
+*/
 //#define SkDebugf(...)  MyFunction(__VA_ARGS__)
 
-/*
- *  To specify a different default font cache limit, define this. If this is
- *  undefined, skia will use a built-in value.
- */
+/* Skia has both debug and release asserts. When an assert fails SK_ABORT will
+   be used to report an abort message. SK_ABORT is expected not to return. Skia
+   provides a default implementation which will print the message with SkDebugf
+   and then call sk_abort_no_print.
+*/
+//#define SK_ABORT(message, ...)
+
+/* To specify a different default font strike cache memory limit, define this. If this is
+   undefined, skia will use a built-in value.
+*/
 //#define SK_DEFAULT_FONT_CACHE_LIMIT   (1024 * 1024)
 
-/*
- *  To specify the default size of the image cache, undefine this and set it to
- *  the desired value (in bytes). SkGraphics.h as a runtime API to set this
- *  value as well. If this is undefined, a built-in value will be used.
- */
+/* To specify a different default font strike cache count limit, define this. If this is
+   undefined, skia will use a built-in value.
+*/
+// #define SK_DEFAULT_FONT_CACHE_COUNT_LIMIT   2048
+
+/* To specify the default size of the image cache, undefine this and set it to
+   the desired value (in bytes). SkGraphics.h as a runtime API to set this
+   value as well. If this is undefined, a built-in value will be used.
+*/
 //#define SK_DEFAULT_IMAGE_CACHE_LIMIT (1024 * 1024)
 
 /*  Define this to set the upper limit for text to support LCD. Values that
     are very large increase the cost in the font cache and draw slower, without
     improving readability. If this is undefined, Skia will use its default
     value (e.g. 48)
- */
+*/
 //#define SK_MAX_SIZE_FOR_LCDTEXT     48
 
 /*  Change the kN32_SkColorType ordering to BGRA to work in X windows.
- */
+*/
 //#define SK_R32_SHIFT    16
 
-
-/* Determines whether to build code that supports the GPU backend. Some classes
+/* Determines whether to build code that supports the Ganesh GPU backend. Some classes
    that are not GPU-specific, such as SkShader subclasses, have optional code
-   that is used allows them to interact with the GPU backend. If you'd like to
-   omit this code set SK_SUPPORT_GPU to 0. This also allows you to omit the gpu
-   directories from your include search path when you're not building the GPU
-   backend. Defaults to 1 (build the GPU code).
- */
-//#define SK_SUPPORT_GPU 1
+   that is used allows them to interact with this GPU backend. If you'd like to
+   include this code, include -DSK_GANESH in your cflags or uncomment below.
+   Defaults to not set (No Ganesh GPU backend).
+   This define affects the ABI of Skia, so make sure it matches the client which uses
+   the compiled version of Skia.
+*/
+//#define SK_GANESH
 
 /* Skia makes use of histogram logging macros to trace the frequency of
- * events. By default, Skia provides no-op versions of these macros.
- * Skia consumers can provide their own definitions of these macros to
- * integrate with their histogram collection backend.
- */
+   events. By default, Skia provides no-op versions of these macros.
+   Skia consumers can provide their own definitions of these macros to
+   integrate with their histogram collection backend.
+*/
 //#define SK_HISTOGRAM_BOOLEAN(name, sample)
 //#define SK_HISTOGRAM_ENUMERATION(name, sample, enum_size)
 //#define SK_HISTOGRAM_EXACT_LINEAR(name, sample, value_max)
 //#define SK_HISTOGRAM_MEMORY_KB(name, sample)
 
+/* Skia tries to make use of some non-standard C++ language extensions.
+   By default, Skia provides msvc and clang/gcc versions of these macros.
+   Skia consumers can provide their own definitions of these macros to
+   integrate with their own compilers and build system.
+*/
+//#define SK_UNUSED [[maybe_unused]]
+//#define SK_WARN_UNUSED_RESULT [[nodiscard]]
+//#define SK_ALWAYS_INLINE inline __attribute__((always_inline))
+//#define SK_NEVER_INLINE __attribute__((noinline))
+//#define SK_PRINTF_LIKE(A, B) __attribute__((format(printf, (A), (B))))
+//#define SK_NO_SANITIZE(A) __attribute__((no_sanitize(A)))
+//#define SK_TRIVIAL_ABI [[clang::trivial_abi]]
+
+/*
+ * If compiling Skia as a DLL, public APIs should be exported. Skia will set
+ * SK_API to something sensible for Clang and MSVC, but if clients need to
+ * customize it for their build system or compiler, they may.
+ * If a client needs to use SK_API (e.g. overriding SK_ABORT), then they
+ * *must* define their own, the default will not be defined prior to loading
+ * this file.
+ */
+//#define SK_API __declspec(dllexport)
+
+
 #endif

package/cpp/skia/include/core/SkBitmap.h

@@ -8,24 +8,32 @@
 #ifndef SkBitmap_DEFINED
 #define SkBitmap_DEFINED
 
+#include "include/core/SkAlphaType.h"
 #include "include/core/SkColor.h"
 #include "include/core/SkImageInfo.h"
-#include "include/core/SkMatrix.h"
 #include "include/core/SkPixmap.h"
 #include "include/core/SkPoint.h"
+#include "include/core/SkRect.h"
 #include "include/core/SkRefCnt.h"
-#include "include/core/SkShader.h"
-#include "include/core/SkTileMode.h"
+#include "include/core/SkSamplingOptions.h"
+#include "include/core/SkSize.h"
+#include "include/core/SkTypes.h"
+#include "include/private/base/SkCPUTypes.h"
+#include "include/private/base/SkDebug.h"
+
+#include <cstddef>
+#include <cstdint>
 
-class SkBitmap;
 class SkColorSpace;
-struct SkMask;
+class SkImage;
+class SkMatrix;
 class SkMipmap;
-struct SkIRect;
-struct SkRect;
 class SkPaint;
 class SkPixelRef;
 class SkShader;
+enum SkColorType : int;
+enum class SkTileMode;
+struct SkMask;
 
 /** \class SkBitmap
     SkBitmap describes a two-dimensional raster pixel array. SkBitmap is built on
@@ -766,11 +774,10 @@ public:
         treated as opaque. If colorType() is kAlpha_8_SkColorType, then RGB is ignored.
 
         @param c            unpremultiplied color
-        @param colorSpace   SkColorSpace of c
 
         example: https://fiddle.skia.org/c/@Bitmap_eraseColor
     */
-    void eraseColor(SkColor4f c, SkColorSpace* colorSpace = nullptr) const;
+    void eraseColor(SkColor4f) const;
 
     /** Replaces pixel values with c, interpreted as being in the sRGB SkColorSpace.
         All pixels contained by bounds() are affected. If the colorType() is
@@ -810,11 +817,9 @@ public:
 
         @param c            unpremultiplied color
         @param area         rectangle to fill
-        @param colorSpace   SkColorSpace of c
 
         example: https://fiddle.skia.org/c/@Bitmap_erase
     */
-    void erase(SkColor4f c, SkColorSpace* colorSpace, const SkIRect& area) const;
     void erase(SkColor4f c, const SkIRect& area) const;
 
     /** Replaces pixel values inside area with c. interpreted as being in the sRGB
@@ -1168,23 +1173,18 @@ public:
         example: https://fiddle.skia.org/c/@Bitmap_peekPixels
     */
     bool peekPixels(SkPixmap* pixmap) const;
-    sk_sp<SkShader> makeShader(SkTileMode tmx, SkTileMode tmy, const SkSamplingOptions&,
-                               const SkMatrix* = nullptr) const;
 
+    /**
+     *  Make a shader with the specified tiling, matrix and sampling.
+     */
+    sk_sp<SkShader> makeShader(SkTileMode tmx, SkTileMode tmy, const SkSamplingOptions&,
+                               const SkMatrix* localMatrix = nullptr) const;
     sk_sp<SkShader> makeShader(SkTileMode tmx, SkTileMode tmy, const SkSamplingOptions& sampling,
-                               const SkMatrix& localMatrix) const {
-        return this->makeShader(tmx, tmy, sampling, &localMatrix);
-    }
-
-    sk_sp<SkShader> makeShader(const SkSamplingOptions& sampling,
-                               const SkMatrix* localMatrix = nullptr) const {
-        return this->makeShader(SkTileMode::kClamp, SkTileMode::kClamp, sampling, localMatrix);
-    }
-
+                               const SkMatrix& lm) const;
+    /** Defaults to clamp in both X and Y. */
+    sk_sp<SkShader> makeShader(const SkSamplingOptions& sampling, const SkMatrix& lm) const;
     sk_sp<SkShader> makeShader(const SkSamplingOptions& sampling,
-                               const SkMatrix& localMatrix) const {
-        return this->makeShader(sampling, &localMatrix);
-    }
+                               const SkMatrix* lm = nullptr) const;
 
     /**
      *  Returns a new image from the bitmap. If the bitmap is marked immutable, this will

package/cpp/skia/include/core/SkBlurTypes.h

@@ -8,8 +8,6 @@
 #ifndef SkBlurTypes_DEFINED
 #define SkBlurTypes_DEFINED
 
-#include "include/core/SkTypes.h"
-
 enum SkBlurStyle : int {
     kNormal_SkBlurStyle,  //!< fuzzy inside and outside
     kSolid_SkBlurStyle,   //!< solid inside, fuzzy outside

package/cpp/skia/include/core/SkCanvas.h

@@ -12,6 +12,7 @@
 #include "include/core/SkClipOp.h"
 #include "include/core/SkColor.h"
 #include "include/core/SkFontTypes.h"
+#include "include/core/SkImageFilter.h"
 #include "include/core/SkImageInfo.h"
 #include "include/core/SkM44.h"
 #include "include/core/SkMatrix.h"
@@ -26,13 +27,13 @@
 #include "include/core/SkString.h"
 #include "include/core/SkSurfaceProps.h"
 #include "include/core/SkTypes.h"
-#include "include/private/SkDeque.h"
-#include "include/private/SkMacros.h"
+#include "include/private/base/SkCPUTypes.h"
+#include "include/private/base/SkDeque.h"
 
+#include <cstdint>
 #include <cstring>
 #include <memory>
 #include <optional>
-#include <vector>
 
 #ifndef SK_SUPPORT_LEGACY_GETTOTALMATRIX
 #define SK_SUPPORT_LEGACY_GETTOTALMATRIX
@@ -44,34 +45,39 @@ class GlyphRunList;
 }
 
 class AutoLayerForImageFilter;
-class GrBackendRenderTarget;
 class GrRecordingContext;
+
 class SkBaseDevice;
 class SkBitmap;
+class SkBlender;
 class SkData;
 class SkDrawable;
-struct SkDrawShadowRec;
 class SkFont;
 class SkImage;
-class SkImageFilter;
+class SkMesh;
 class SkPaintFilterCanvas;
 class SkPath;
 class SkPicture;
 class SkPixmap;
-class SkRegion;
 class SkRRect;
-struct SkRSXform;
-class SkMesh;
+class SkRegion;
+class SkShader;
 class SkSpecialImage;
 class SkSurface;
 class SkSurface_Base;
 class SkTextBlob;
 class SkVertices;
+struct SkDrawShadowRec;
+struct SkRSXform;
 
 namespace skgpu::graphite { class Recorder; }
 namespace sktext::gpu { class Slug; }
 namespace SkRecords { class Draw; }
 
+#if defined(SK_BUILD_FOR_ANDROID_FRAMEWORK) && defined(SK_GANESH)
+class GrBackendRenderTarget;
+#endif
+
 // TODO:
 // This is not ideal but Chrome is depending on a forward decl of GrSlug here.
 // It should be removed once Chrome has migrated to sktext::gpu::Slug.
@@ -650,7 +656,7 @@ public:
         SkRect bounds suggests but does not define layer size. To clip drawing to
         a specific rectangle, use clipRect().
 
-        alpha of zero is fully transparent, 255 is fully opaque.
+        alpha of zero is fully transparent, 1.0f is fully opaque.
 
         Call restoreToCount() with returned value to restore this and subsequent saves.
 
@@ -660,7 +666,11 @@ public:
 
         example: https://fiddle.skia.org/c/@Canvas_saveLayerAlpha
     */
-    int saveLayerAlpha(const SkRect* bounds, U8CPU alpha);
+    int saveLayerAlphaf(const SkRect* bounds, float alpha);
+    // Helper that accepts an int between 0 and 255, and divides it by 255.0
+    int saveLayerAlpha(const SkRect* bounds, U8CPU alpha) {
+        return this->saveLayerAlphaf(bounds, alpha * (1.0f / 255));
+    }
 
     /** \enum SkCanvas::SaveLayerFlagsSet
         SaveLayerFlags provides options that may be used in any combination in SaveLayerRec,
@@ -2173,7 +2183,7 @@ public:
 
     ///////////////////////////////////////////////////////////////////////////
 
-#if defined(SK_BUILD_FOR_ANDROID_FRAMEWORK) && SK_SUPPORT_GPU
+#if defined(SK_BUILD_FOR_ANDROID_FRAMEWORK) && defined(SK_GANESH)
     // These methods exist to support WebView in Android Framework.
     SkIRect topLayerBounds() const;
     GrBackendRenderTarget topLayerBackendRenderTarget() const;
@@ -2290,7 +2300,7 @@ protected:
 
     virtual void onDiscard();
 
-#if (SK_SUPPORT_GPU || defined(SK_GRAPHITE_ENABLED))
+#if (defined(SK_GANESH) || defined(SK_GRAPHITE))
     /** Experimental
      */
     virtual sk_sp<sktext::gpu::Slug> onConvertGlyphRunListToSlug(
@@ -2352,6 +2362,13 @@ private:
 
     // Encapsulate state needed to restore from saveBehind()
     struct BackImage {
+        // Out of line to avoid including SkSpecialImage.h
+        BackImage(sk_sp<SkSpecialImage>, SkIPoint);
+        BackImage(const BackImage&);
+        BackImage(BackImage&&);
+        BackImage& operator=(const BackImage&);
+        ~BackImage();
+
         sk_sp<SkSpecialImage> fImage;
         SkIPoint              fLoc;
     };
@@ -2406,7 +2423,7 @@ private:
         fSurfaceBase = sb;
     }
     friend class SkSurface_Base;
-    friend class SkSurface_Gpu;
+    friend class SkSurface_Ganesh;
 
     SkIRect fClipRestrictionRect = SkIRect::MakeEmpty();
     int fClipRestrictionSaveCount = -1;
@@ -2440,7 +2457,7 @@ private:
     SkCanvas& operator=(SkCanvas&&) = delete;
     SkCanvas& operator=(const SkCanvas&) = delete;
 
-#if (SK_SUPPORT_GPU || defined(SK_GRAPHITE_ENABLED))
+#if (defined(SK_GANESH) || defined(SK_GRAPHITE))
     friend class sktext::gpu::Slug;
     /** Experimental
      * Convert a SkTextBlob to a sktext::gpu::Slug using the current canvas state.

package/cpp/skia/include/core/SkCapabilities.h

@@ -15,7 +15,7 @@
 namespace SkSL { struct ShaderCaps; }
 #endif
 
-#if defined(SK_GRAPHITE_ENABLED)
+#if defined(SK_GRAPHITE)
 namespace skgpu::graphite { class Caps; }
 #endif
 
@@ -28,7 +28,7 @@ public:
 #endif
 
 protected:
-#if defined(SK_GRAPHITE_ENABLED)
+#if defined(SK_GRAPHITE)
     friend class skgpu::graphite::Caps; // for ctor
 #endif
 

package/cpp/skia/include/core/SkColor.h

@@ -11,8 +11,10 @@
 #include "include/core/SkAlphaType.h"
 #include "include/core/SkScalar.h"
 #include "include/core/SkTypes.h"
+#include "include/private/base/SkCPUTypes.h"
 
 #include <array>
+#include <cstdint>
 
 /** \file SkColor.h
 

package/cpp/skia/include/core/SkColorPriv.h

@@ -9,9 +9,11 @@
 #define SkColorPriv_DEFINED
 
 #include "include/core/SkColor.h"
-#include "include/core/SkMath.h"
-#include "include/private/SkTPin.h"
-#include "include/private/SkTo.h"
+#include "include/private/base/SkMath.h"
+#include "include/private/base/SkTPin.h"
+#include "include/private/base/SkTo.h"
+
+#include <algorithm>
 
 /** Turn 0..255 into 0..256 by adding 1 at the half-way point. Used to turn a
     byte into a scale value, so that we can say scale * value >> 8 instead of
@@ -146,7 +148,20 @@ static SK_ALWAYS_INLINE uint32_t SkAlphaMulQ(uint32_t c, unsigned scale) {
 }
 
 static inline SkPMColor SkPMSrcOver(SkPMColor src, SkPMColor dst) {
-    return src + SkAlphaMulQ(dst, SkAlpha255To256(255 - SkGetPackedA32(src)));
+    uint32_t scale = SkAlpha255To256(255 - SkGetPackedA32(src));
+
+    uint32_t mask = 0xFF00FF;
+    uint32_t rb = (((dst & mask) * scale) >> 8) & mask;
+    uint32_t ag = (((dst >> 8) & mask) * scale) & ~mask;
+
+    rb += (src &  mask);
+    ag += (src & ~mask);
+
+    // Color channels (but not alpha) can overflow, so we have to saturate to 0xFF in each lane.
+    return std::min(rb & 0x000001FF, 0x000000FFU) |
+           std::min(ag & 0x0001FF00, 0x0000FF00U) |
+           std::min(rb & 0x01FF0000, 0x00FF0000U) |
+                   (ag & 0xFF000000);
 }
 
 #endif

package/cpp/skia/include/core/SkColorSpace.h

@@ -9,10 +9,13 @@
 #define SkColorSpace_DEFINED
 
 #include "include/core/SkRefCnt.h"
-#include "include/private/SkFixed.h"
-#include "include/private/SkOnce.h"
+#include "include/core/SkTypes.h"
+#include "include/private/base/SkFixed.h"
+#include "include/private/base/SkOnce.h"
 #include "modules/skcms/skcms.h"
-#include <memory>
+
+#include <cstddef>
+#include <cstdint>
 
 class SkData;
 
@@ -147,8 +150,7 @@ public:
     bool isNumericalTransferFn(skcms_TransferFunction* fn) const;
 
     /**
-     *  Returns true and sets |toXYZD50| if the color gamut can be described as a matrix.
-     *  Returns false otherwise.
+     *  Returns true and sets |toXYZD50|.
      */
     bool toXYZD50(skcms_Matrix3x3* toXYZD50) const;
 
@@ -160,23 +162,19 @@ public:
 
     /**
      *  Returns a color space with the same gamut as this one, but with a linear gamma.
-     *  For color spaces whose gamut can not be described in terms of XYZ D50, returns
-     *  linear sRGB.
      */
     sk_sp<SkColorSpace> makeLinearGamma() const;
 
     /**
-     *  Returns a color space with the same gamut as this one, with with the sRGB transfer
-     *  function. For color spaces whose gamut can not be described in terms of XYZ D50, returns
-     *  sRGB.
+     *  Returns a color space with the same gamut as this one, but with the sRGB transfer
+     *  function.
      */
     sk_sp<SkColorSpace> makeSRGBGamma() const;
 
     /**
      *  Returns a color space with the same transfer function as this one, but with the primary
-     *  colors rotated. For any XYZ space, this produces a new color space that maps RGB to GBR
-     *  (when applied to a source), and maps RGB to BRG (when applied to a destination). For other
-     *  types of color spaces, returns nullptr.
+     *  colors rotated. In other words, this produces a new color space that maps RGB to GBR
+     *  (when applied to a source), and maps RGB to BRG (when applied to a destination).
      *
      *  This is used for testing, to construct color spaces that have severe and testable behavior.
      */
@@ -190,15 +188,14 @@ public:
      *  in some cases: converting ICC fixed point to float, converting white point to D50,
      *  rounding decisions on transfer function and matrix.
      *
-     *  This does not consider a 2.2f exponential transfer function to be sRGB.  While these
+     *  This does not consider a 2.2f exponential transfer function to be sRGB. While these
      *  functions are similar (and it is sometimes useful to consider them together), this
      *  function checks for logical equality.
      */
     bool isSRGB() const;
 
     /**
-     *  Returns nullptr on failure.  Fails when we fallback to serializing ICC data and
-     *  the data is too large to serialize.
+     *  Returns a serialized representation of this color space.
      */
     sk_sp<SkData> serialize() const;
 
@@ -211,7 +208,7 @@ public:
     static sk_sp<SkColorSpace> Deserialize(const void* data, size_t length);
 
     /**
-     *  If both are null, we return true.  If one is null and the other is not, we return false.
+     *  If both are null, we return true. If one is null and the other is not, we return false.
      *  If both are non-null, we do a deeper compare.
      */
     static bool Equals(const SkColorSpace*, const SkColorSpace*);

package/cpp/skia/include/core/SkColorType.h

@@ -28,6 +28,7 @@ enum SkColorType : int {
     kBGRA_1010102_SkColorType, //!< 10 bits for blue, green, red; 2 bits for alpha; in 32-bit word
     kRGB_101010x_SkColorType,  //!< pixel with 10 bits each for red, green, blue; in 32-bit word
     kBGR_101010x_SkColorType,  //!< pixel with 10 bits each for blue, green, red; in 32-bit word
+    kBGR_101010x_XR_SkColorType, //!< pixel with 10 bits each for blue, green, red; in 32-bit word, extended range
     kGray_8_SkColorType,       //!< pixel with grayscale level in 8-bit byte
     kRGBA_F16Norm_SkColorType, //!< pixel with half floats in [0,1] for red, green, blue, alpha;
                                //   in 64-bit word

package/cpp/skia/include/core/SkContourMeasure.h

@@ -10,7 +10,7 @@
 
 #include "include/core/SkPath.h"
 #include "include/core/SkRefCnt.h"
-#include "include/private/SkTDArray.h"
+#include "include/private/base/SkTDArray.h"
 
 struct SkConic;
 

package/cpp/skia/include/core/SkCoverageMode.h

@@ -8,8 +8,6 @@
 #ifndef SkCoverageMode_DEFINED
 #define SkCoverageMode_DEFINED
 
-#include "include/core/SkTypes.h"
-
 /**
  *  Describes geometric operations (ala SkRegion::Op) that can be applied to coverage bytes.
  *  These can be thought of as variants of porter-duff (SkBlendMode) modes, but only applied

package/cpp/skia/include/core/SkCubicMap.h

@@ -9,6 +9,8 @@
 #define SkCubicMap_DEFINED
 
 #include "include/core/SkPoint.h"
+#include "include/core/SkScalar.h"
+#include "include/core/SkTypes.h"
 
 /**
  *  Fast evaluation of a cubic ease-in / ease-out curve. This is defined as a parametric cubic

package/cpp/skia/include/core/SkData.h

@@ -8,9 +8,12 @@
 #ifndef SkData_DEFINED
 #define SkData_DEFINED
 
-#include <stdio.h>
-
 #include "include/core/SkRefCnt.h"
+#include "include/private/base/SkAPI.h"
+#include "include/private/base/SkAssert.h"
+
+#include <cstdint>
+#include <cstdio>
 
 class SkStream;
 

package/cpp/skia/include/core/SkDataTable.h

@@ -8,8 +8,12 @@
 #ifndef SkDataTable_DEFINED
 #define SkDataTable_DEFINED
 
-#include "include/core/SkData.h"
-#include "include/private/SkTDArray.h"
+#include "include/core/SkRefCnt.h"
+#include "include/private/base/SkAPI.h"
+#include "include/private/base/SkAssert.h"
+
+#include <cstdint>
+#include <cstring>
 
 /**
  *  Like SkData, SkDataTable holds an immutable data buffer. The data buffer is