@shopify/react-native-skia 2.0.0-next.1 → 2.0.0-next.3

package/cpp/api/JsiSkCanvas.h

@@ -243,6 +243,12 @@ public:
 
     auto jsiPoints = arguments[1].asObject(runtime).asArray(runtime);
     auto pointsSize = jsiPoints.size(runtime);
+    
+    // Check if we have at least one point
+    if (pointsSize == 0) {
+      throw std::invalid_argument("Points array must not be empty");
+    }
+    
     points.reserve(pointsSize);
 
     for (int i = 0; i < pointsSize; i++) {
@@ -274,6 +280,12 @@ public:
 
     auto jsiCubics = arguments[0].asObject(runtime).asArray(runtime);
     auto cubicsSize = jsiCubics.size(runtime);
+    
+    // Validate cubic points - must be exactly 12 points
+    if (cubicsSize != 12) {
+      throw std::invalid_argument("Cubic points array must contain exactly 12 points");
+    }
+    
     cubics.reserve(cubicsSize);
     for (int i = 0; i < cubicsSize; i++) {
       std::shared_ptr<SkPoint> point = JsiSkPoint::fromValue(
@@ -284,6 +296,12 @@ public:
     if (count >= 2 && !arguments[1].isNull() && !arguments[1].isUndefined()) {
       auto jsiColors = arguments[1].asObject(runtime).asArray(runtime);
       auto colorsSize = jsiColors.size(runtime);
+      
+      // Validate colors array - must be exactly 4 colors
+      if (colorsSize != 4) {
+        throw std::invalid_argument("Colors array must contain exactly 4 colors");
+      }
+      
       colors.reserve(colorsSize);
       for (int i = 0; i < colorsSize; i++) {
         SkColor color = JsiSkColor::fromValue(
@@ -295,6 +313,12 @@ public:
     if (count >= 3 && !arguments[2].isNull() && !arguments[2].isUndefined()) {
       auto jsiTexs = arguments[2].asObject(runtime).asArray(runtime);
       auto texsSize = jsiTexs.size(runtime);
+      
+      // Validate textures array - must be exactly 4 points
+      if (texsSize != 4) {
+        throw std::invalid_argument("Texture coordinates array must contain exactly 4 points");
+      }
+      
       texs.reserve(texsSize);
       for (int i = 0; i < texsSize; i++) {
         auto point = JsiSkPoint::fromValue(
@@ -306,7 +330,8 @@ public:
     auto paint =
         count >= 4 ? JsiSkPaint::fromValue(runtime, arguments[4]) : nullptr;
     auto blendMode = static_cast<SkBlendMode>(arguments[3].asNumber());
-    _canvas->drawPatch(cubics.data(), colors.data(), texs.data(), blendMode,
+    _canvas->drawPatch(cubics.data(), colors.empty() ? nullptr : colors.data(), 
+                       texs.empty() ? nullptr : texs.data(), blendMode,
                        *paint);
     return jsi::Value::undefined();
   }
@@ -364,6 +389,12 @@ public:
 
     std::vector<SkGlyphID> glyphs;
     int glyphsSize = static_cast<int>(jsiGlyphs.size(runtime));
+    
+    // Validate that glyphs and positions arrays have the same size
+    if (glyphsSize != pointsSize) {
+      throw std::invalid_argument("Glyphs and positions arrays must have the same length");
+    }
+    
     glyphs.reserve(glyphsSize);
     for (int i = 0; i < glyphsSize; i++) {
       glyphs.push_back(jsiGlyphs.getValueAtIndex(runtime, i).asNumber());
@@ -522,11 +553,22 @@ public:
           runtime, rects.getValueAtIndex(runtime, i).asObject(runtime));
       skRects.push_back(*rect.get());
     }
+    
+    // Validate transforms and rects have the same size
+    if (xformsSize != rectsSize) {
+      throw std::invalid_argument("Transforms and rects arrays must have the same length");
+    }
 
     std::vector<SkColor> colors;
     if (count > 5 && !arguments[5].isUndefined()) {
       auto colorsArray = arguments[5].asObject(runtime).asArray(runtime);
       int colorsSize = static_cast<int>(colorsArray.size(runtime));
+      
+      // Validate colors array matches the size of sprites and transforms
+      if (colorsSize != rectsSize) {
+        throw std::invalid_argument("Colors array must have the same length as rects/transforms");
+      }
+      
       colors.reserve(colorsSize);
       for (int i = 0; i < colorsSize; i++) {
         // Convert from [r,g,b,a] in [0,1] to SkColor
@@ -551,7 +593,8 @@ public:
       sampling = SamplingOptionsFromValue(runtime, arguments[6]);
     }
     _canvas->drawAtlas(atlas.get(), xforms.data(), skRects.data(),
-                       colors.data(), skRects.size(), blendMode, sampling,
+                       colors.empty() ? nullptr : colors.data(), 
+                       skRects.size(), blendMode, sampling,
                        nullptr, paint.get());
 
     return jsi::Value::undefined();

package/cpp/api/JsiSkContourMeasure.h

@@ -34,7 +34,7 @@ public:
     SkPoint tangent;
     auto result = getObject()->getPosTan(dist, &position, &tangent);
     if (!result) {
-      throw jsi::JSError(runtime, "getSegment() failed");
+      throw jsi::JSError(runtime, "getPosTan() failed");
     }
     auto posTan = jsi::Array(runtime, 2);
     auto pos = jsi::Object::createFromHostObject(

package/cpp/api/JsiSkHostObjects.h

@@ -54,20 +54,21 @@ public:
 
   /**
    * Returns the underlying object exposed by this host object. This object
-   * should be wrapped in a shared pointer of some kind
+   * should be wrapped in a shared pointer of some kind.
+   * Throws if the object has been disposed.
    * @return Underlying object
    */
-  T getObject() { return _object; }
-  const T getObject() const { return _object; }
+  T getObject() { return validateObject(); }
+  const T getObject() const { return validateObject(); }
 
   /**
-   Updates the inner object with a new version of the object.
+   * Updates the inner object with a new version of the object.
    */
   void setObject(T object) { _object = object; }
 
   /**
-   Dispose function that can be exposed to JS by using the JSI_API_TYPENAME
-   macro
+   * Dispose function that can be exposed to JS by using the JSI_API_TYPENAME
+   * macro.
    */
   JSI_HOST_FUNCTION(dispose) {
     safeDispose();
@@ -76,12 +77,22 @@ public:
 
 protected:
   /**
-   Override to implement disposale of allocated resources like smart pointers
-   etc. This method will only be called once for each instance of this class.
+   * Override to implement disposal of allocated resources like smart pointers.
+   * This method will only be called once for each instance of this class.
    */
   virtual void releaseResources() = 0;
 
 private:
+  /**
+   * Validates that _object was not disposed and returns it.
+   */
+  T validateObject() const {
+    if (_isDisposed) {
+      throw std::runtime_error("Attempted to access a disposed object.");
+    }
+    return _object;
+  }
+
   void safeDispose() {
     if (!_isDisposed) {
       _isDisposed = true;
@@ -90,12 +101,12 @@ private:
   }
 
   /**
-   * Wrapped object
+   * Wrapped object.
    */
   T _object;
 
   /**
-   Resource disposed flag
+   * Resource disposed flag.
    */
   std::atomic<bool> _isDisposed = {false};
 };

package/cpp/api/JsiSkParagraph.h

@@ -24,19 +24,19 @@
 namespace RNSkia {
 
 namespace jsi = facebook::jsi;
-
 namespace para = skia::textlayout;
 
 /**
- Implementation of the Paragraph object in JSI
+ * Implementation of the Paragraph object in JSI
  */
-class JsiSkParagraph : public JsiSkHostObject {
+class JsiSkParagraph
+    : public JsiSkWrappingSharedPtrHostObject<para::Paragraph> {
 public:
   EXPORT_JSI_API_TYPENAME(JsiSkParagraph, Paragraph)
 
   JSI_HOST_FUNCTION(layout) {
     auto width = getArgumentAsNumber(runtime, arguments, count, 0);
-    _paragraph->layout(width);
+    getObject()->layout(width);
     return jsi::Value::undefined();
   }
 
@@ -45,34 +45,34 @@ public:
         getArgumentAsHostObject<JsiSkCanvas>(runtime, arguments, count, 0);
     auto x = getArgumentAsNumber(runtime, arguments, count, 1);
     auto y = getArgumentAsNumber(runtime, arguments, count, 2);
-    _paragraph->paint(jsiCanvas->getCanvas(), x, y);
+    getObject()->paint(jsiCanvas->getCanvas(), x, y);
     return jsi::Value::undefined();
   }
 
   JSI_HOST_FUNCTION(getHeight) {
-    return static_cast<double>(_paragraph->getHeight());
+    return static_cast<double>(getObject()->getHeight());
   }
 
   JSI_HOST_FUNCTION(getMaxWidth) {
-    return static_cast<double>(_paragraph->getMaxWidth());
+    return static_cast<double>(getObject()->getMaxWidth());
   }
 
   JSI_HOST_FUNCTION(getMaxIntrinsicWidth) {
-    return static_cast<double>(_paragraph->getMaxIntrinsicWidth());
+    return static_cast<double>(getObject()->getMaxIntrinsicWidth());
   }
 
   JSI_HOST_FUNCTION(getMinIntrinsicWidth) {
-    return static_cast<double>(_paragraph->getMinIntrinsicWidth());
+    return static_cast<double>(getObject()->getMinIntrinsicWidth());
   }
 
   JSI_HOST_FUNCTION(getLongestLine) {
-    return static_cast<double>(_paragraph->getLongestLine());
+    return static_cast<double>(getObject()->getLongestLine());
   }
 
   JSI_HOST_FUNCTION(getGlyphPositionAtCoordinate) {
     auto dx = getArgumentAsNumber(runtime, arguments, count, 0);
     auto dy = getArgumentAsNumber(runtime, arguments, count, 1);
-    auto result = _paragraph->getGlyphPositionAtCoordinate(dx, dy);
+    auto result = getObject()->getGlyphPositionAtCoordinate(dx, dy);
     return result.position;
   }
 
@@ -80,8 +80,8 @@ public:
     auto start = getArgumentAsNumber(runtime, arguments, count, 0);
     auto end = getArgumentAsNumber(runtime, arguments, count, 1);
     auto result =
-        _paragraph->getRectsForRange(start, end, para::RectHeightStyle::kTight,
-                                     para::RectWidthStyle::kTight);
+        getObject()->getRectsForRange(start, end, para::RectHeightStyle::kTight,
+                                      para::RectWidthStyle::kTight);
     auto returnValue = jsi::Array(runtime, result.size());
     for (size_t i = 0; i < result.size(); ++i) {
       returnValue.setValueAtIndex(
@@ -93,7 +93,7 @@ public:
 
   JSI_HOST_FUNCTION(getLineMetrics) {
     std::vector<para::LineMetrics> metrics;
-    _paragraph->getLineMetrics(metrics);
+    getObject()->getLineMetrics(metrics);
     auto returnValue = jsi::Array(runtime, metrics.size());
     auto height = 0;
     for (size_t i = 0; i < metrics.size(); ++i) {
@@ -110,7 +110,7 @@ public:
 
   JSI_HOST_FUNCTION(getRectsForPlaceholders) {
     std::vector<para::TextBox> placeholderInfos =
-        _paragraph->getRectsForPlaceholders();
+        getObject()->getRectsForPlaceholders();
     auto returnValue = jsi::Array(runtime, placeholderInfos.size());
     for (size_t i = 0; i < placeholderInfos.size(); ++i) {
       auto obj = jsi::Object(runtime);
@@ -124,12 +124,6 @@ public:
     return returnValue;
   }
 
-  JSI_HOST_FUNCTION(dispose) {
-    _paragraph = nullptr;
-
-    return jsi::Value::undefined();
-  }
-
   JSI_EXPORT_FUNCTIONS(JSI_EXPORT_FUNC(JsiSkParagraph, layout),
                        JSI_EXPORT_FUNC(JsiSkParagraph, paint),
                        JSI_EXPORT_FUNC(JsiSkParagraph, getMaxWidth),
@@ -146,12 +140,8 @@ public:
 
   explicit JsiSkParagraph(std::shared_ptr<RNSkPlatformContext> context,
                           para::ParagraphBuilder *paragraphBuilder)
-      : JsiSkHostObject(std::move(context)) {
-    _paragraph = paragraphBuilder->Build();
-  }
-
-public:
-  std::unique_ptr<para::Paragraph> _paragraph;
+      : JsiSkWrappingSharedPtrHostObject<para::Paragraph>(
+            std::move(context), std::move(paragraphBuilder->Build())) {}
 };
 
 } // namespace RNSkia

package/cpp/api/JsiSkPathEffect.h

@@ -26,7 +26,7 @@ public:
                                                    std::move(pathEffect)) {}
 
   EXPORT_JSI_API_TYPENAME(JsiSkPathEffect, PathEffect)
-  JSI_EXPORT_FUNCTIONS(JSI_EXPORT_FUNC(JsiSkColorFilter, dispose))
+  JSI_EXPORT_FUNCTIONS(JSI_EXPORT_FUNC(JsiSkPathEffect, dispose))
 };
 
 } // namespace RNSkia

package/cpp/api/JsiSkPictureFactory.h

@@ -45,7 +45,7 @@ public:
   JSI_EXPORT_FUNCTIONS(JSI_EXPORT_FUNC(JsiSkPictureFactory, MakePicture))
 
   explicit JsiSkPictureFactory(std::shared_ptr<RNSkPlatformContext> context)
-      : JsiSkHostObject(context) {}
+      : JsiSkHostObject(std::move(context)) {}
 };
 
 } // namespace RNSkia

package/cpp/api/JsiSkShaderFactory.h

@@ -84,14 +84,22 @@ public:
     SkPoint pts[] = {p1, p2};
 
     std::vector<SkColor> colors = getColors(runtime, arguments[2]);
+    auto colorsSize = colors.size();
+    if (colorsSize < 2) {
+      throw std::invalid_argument("colors must have at least 2 colors");
+    }
     std::vector<SkScalar> positions = getPositions(runtime, arguments[3]);
+    if (!positions.empty() && positions.size() != colorsSize) {
+      throw std::invalid_argument(
+          "positions must be empty or have the same size as colors");
+    }
     auto tileMode = getTileMode(arguments, 4, count);
     auto flag = getFlag(arguments, 6, count);
     auto localMatrix = getLocalMatrix(runtime, arguments, 5, count);
 
     sk_sp<SkShader> gradient = SkGradientShader::MakeLinear(
-        pts, colors.data(), positions.data(), static_cast<int>(colors.size()),
-        tileMode, flag, localMatrix);
+        pts, colors.data(), !positions.empty() ? positions.data() : nullptr,
+        static_cast<int>(colorsSize), tileMode, flag, localMatrix);
     return jsi::Object::createFromHostObject(
         runtime,
         std::make_shared<JsiSkShader>(getContext(), std::move(gradient)));
@@ -103,14 +111,23 @@ public:
     auto r = arguments[1].asNumber();
 
     std::vector<SkColor> colors = getColors(runtime, arguments[2]);
+    auto colorsSize = colors.size();
+    if (colorsSize < 2) {
+      throw std::invalid_argument("colors must have at least 2 colors");
+    }
     std::vector<SkScalar> positions = getPositions(runtime, arguments[3]);
+    if (!positions.empty() && positions.size() != colorsSize) {
+      throw std::invalid_argument(
+          "positions must be empty or the same size as colors");
+    }
     auto tileMode = getTileMode(arguments, 4, count);
     auto flag = getFlag(arguments, 6, count);
     auto localMatrix = getLocalMatrix(runtime, arguments, 5, count);
 
     sk_sp<SkShader> gradient = SkGradientShader::MakeRadial(
-        center, r, colors.data(), positions.data(),
-        static_cast<int>(colors.size()), tileMode, flag, localMatrix);
+        center, r, colors.data(),
+        !positions.empty() ? positions.data() : nullptr,
+        static_cast<int>(colorsSize), tileMode, flag, localMatrix);
     return jsi::Object::createFromHostObject(
         runtime,
         std::make_shared<JsiSkShader>(getContext(), std::move(gradient)));
@@ -120,7 +137,15 @@ public:
     auto x = arguments[0].asNumber();
     auto y = arguments[1].asNumber();
     std::vector<SkColor> colors = getColors(runtime, arguments[2]);
+    auto colorsSize = colors.size();
+    if (colorsSize < 2) {
+      throw std::invalid_argument("colors must have at least 2 colors");
+    }
     std::vector<SkScalar> positions = getPositions(runtime, arguments[3]);
+    if (!positions.empty() && positions.size() != colorsSize) {
+      throw std::invalid_argument(
+          "positions must be empty or the same size as colors");
+    }
     auto tileMode = getTileMode(arguments, 4, count);
     auto localMatrix = getLocalMatrix(runtime, arguments, 5, count);
     auto flag = getFlag(arguments, 6, count);
@@ -130,8 +155,9 @@ public:
                         ? 360
                         : arguments[8].asNumber();
     sk_sp<SkShader> gradient = SkGradientShader::MakeSweep(
-        x, y, colors.data(), positions.data(), static_cast<int>(colors.size()),
-        tileMode, startAngle, endAngle, flag, localMatrix);
+        x, y, colors.data(), !positions.empty() ? positions.data() : nullptr,
+        static_cast<int>(colorsSize), tileMode, startAngle, endAngle, flag,
+        localMatrix);
     return jsi::Object::createFromHostObject(
         runtime,
         std::make_shared<JsiSkShader>(getContext(), std::move(gradient)));
@@ -147,14 +173,23 @@ public:
     auto endRadius = arguments[3].asNumber();
 
     std::vector<SkColor> colors = getColors(runtime, arguments[4]);
+    auto colorsSize = colors.size();
+    if (colorsSize < 2) {
+      throw std::invalid_argument("colors must have at least 2 colors");
+    }
     std::vector<SkScalar> positions = getPositions(runtime, arguments[5]);
+    if (!positions.empty() && positions.size() != colorsSize) {
+      throw std::invalid_argument(
+          "positions must be empty or the same size as colors");
+    }
     auto tileMode = getTileMode(arguments, 6, count);
     auto localMatrix = getLocalMatrix(runtime, arguments, 7, count);
     auto flag = getFlag(arguments, 8, count);
 
     sk_sp<SkShader> gradient = SkGradientShader::MakeTwoPointConical(
-        start, startRadius, end, endRadius, colors.data(), positions.data(),
-        static_cast<int>(colors.size()), tileMode, flag, localMatrix);
+        start, startRadius, end, endRadius, colors.data(),
+        !positions.empty() ? positions.data() : nullptr,
+        static_cast<int>(colorsSize), tileMode, flag, localMatrix);
 
     return jsi::Object::createFromHostObject(
         runtime,

package/cpp/api/JsiSkVertices.h

@@ -70,6 +70,10 @@ public:
       if (count >= 3 && !arguments[2].isNull() && !arguments[2].isUndefined()) {
         auto jsiTexs = arguments[2].asObject(runtime).asArray(runtime);
         auto texsSize = jsiTexs.size(runtime);
+        if (texsSize != positionsSize) {
+          throw jsi::JSError(runtime, "The number of texture coordinates must "
+                                      "match the number of positions");
+        }
         texs.reserve(texsSize);
         for (int i = 0; i < texsSize; i++) {
           auto point = JsiSkPoint::fromValue(
@@ -81,6 +85,11 @@ public:
       if (count >= 4 && !arguments[3].isNull() && !arguments[3].isUndefined()) {
         auto jsiColors = arguments[3].asObject(runtime).asArray(runtime);
         auto colorsSize = jsiColors.size(runtime);
+        if (colorsSize != positionsSize) {
+          throw jsi::JSError(
+              runtime,
+              "The number of colors must match the number of positions");
+        }
         colors.reserve(colorsSize);
         for (int i = 0; i < colorsSize; i++) {
           SkColor color = JsiSkColor::fromValue(
@@ -132,9 +141,11 @@ public:
       //                    builder.indices());
       //                }
       //                auto vertices = builder.detach();
-      auto vertices = SkVertices::MakeCopy(
-          mode, positionsSize, positions.data(), texs.data(), colors.data(),
-          indicesSize, indices.data());
+      auto vertices =
+          SkVertices::MakeCopy(mode, positionsSize, positions.data(),
+                               texs.size() > 0 ? texs.data() : nullptr,
+                               colors.size() > 0 ? colors.data() : nullptr,
+                               indicesSize, indices.data());
       return jsi::Object::createFromHostObject(
           runtime,
           std::make_shared<JsiSkVertices>(context, std::move(vertices)));

package/cpp/api/recorder/Drawings.h

@@ -418,7 +418,7 @@ public:
         SkPaint shadowPaint;
         shadowPaint.setAntiAlias(true);
         shadowPaint.setColor(shadow.color.value_or(SK_ColorBLACK));
-        shadowPaint.setAlphaf(opacity);
+        shadowPaint.setAlphaf(opacity * shadowPaint.getAlphaf());
         shadowPaint.setMaskFilter(SkMaskFilter::MakeBlur(
             SkBlurStyle::kNormal_SkBlurStyle, shadow.blur, true));
 
@@ -442,7 +442,7 @@ public:
         SkPaint shadowPaint;
         shadowPaint.setAntiAlias(true);
         shadowPaint.setColor(shadow.color.value_or(SK_ColorBLACK));
-        shadowPaint.setAlphaf(opacity);
+        shadowPaint.setAlphaf(opacity * shadowPaint.getAlphaf());
         shadowPaint.setMaskFilter(SkMaskFilter::MakeBlur(
             SkBlurStyle::kNormal_SkBlurStyle, shadow.blur, true));
 
@@ -637,6 +637,16 @@ public:
   }
 
   void draw(DrawingCtx *ctx) {
+    // Validate colors array has exactly 4 colors if provided
+    if (props.colors.has_value() && props.colors.value().size() != 4) {
+      throw std::invalid_argument("Colors array for patch must have exactly 4 colors");
+    }
+    
+    // Validate texture array has exactly 4 points if provided
+    if (props.texture.has_value() && props.texture.value().size() != 4) {
+      throw std::invalid_argument("Texture coordinates array for patch must have exactly 4 points");
+    }
+
     // Determine default blend mode based on presence of colors
     SkBlendMode defaultBlendMode = props.colors.has_value()
                                        ? SkBlendMode::kDstOver
@@ -676,6 +686,15 @@ public:
   }
 
   void draw(DrawingCtx *ctx) {
+    // Validate array sizes
+    if (props.colors.has_value() && props.colors.value().size() != props.vertices.size()) {
+      throw std::invalid_argument("Colors array must have the same size as vertices array");
+    }
+    
+    if (props.textures.has_value() && props.textures.value().size() != props.vertices.size()) {
+      throw std::invalid_argument("Textures array must have the same size as vertices array");
+    }
+
     // Create vertices using MakeCopy
     auto vertices = SkVertices::MakeCopy(
         props.mode, static_cast<int>(props.vertices.size()),
@@ -874,9 +893,10 @@ public:
   }
 
   void draw(DrawingCtx *ctx) {
-    if (props.paragraph && props.paragraph->_paragraph) {
-      props.paragraph->_paragraph->layout(props.width);
-      props.paragraph->_paragraph->paint(ctx->canvas, props.x, props.y);
+    if (props.paragraph) {
+      auto paragraph = props.paragraph->getObject();
+      paragraph->layout(props.width);
+      paragraph->paint(ctx->canvas, props.x, props.y);
     }
   }
 };
@@ -908,6 +928,16 @@ public:
 
   void draw(DrawingCtx *ctx) {
     if (props.image) {
+      // Validate transforms and sprites have the same size
+      if (props.transforms.size() != props.sprites.size()) {
+        throw std::invalid_argument("transforms and sprites arrays must have the same length");
+      }
+      
+      // Validate colors array matches if provided
+      if (props.colors.has_value() && props.colors.value().size() != props.transforms.size()) {
+        throw std::invalid_argument("colors array must have the same length as transforms/sprites");
+      }
+      
       auto colors =
           props.colors.has_value() ? props.colors.value().data() : nullptr;
       auto blendMode = props.blendMode.value_or(SkBlendMode::kDstOver);

package/cpp/api/recorder/Shaders.h

@@ -239,6 +239,16 @@ public:
   }
 
   void pushShader(DrawingCtx *ctx) {
+    // Validate colors array has at least 2 colors
+    if (props.colors.size() < 2) {
+      throw std::invalid_argument("Colors array must have at least 2 colors");
+    }
+    
+    // Validate positions array matches colors array in size
+    if (props.positions.has_value() && props.positions.value().size() != props.colors.size()) {
+      throw std::invalid_argument("Positions array must have the same size as colors array");
+    }
+    
     SkMatrix m3 = processTransform(props.matrix, props.transform, props.origin);
     const SkPoint pts[2] = {props.start, props.end};
     auto shader = SkGradientShader::MakeLinear(
@@ -276,6 +286,16 @@ public:
   }
 
   void pushShader(DrawingCtx *ctx) {
+    // Validate colors array has at least 2 colors
+    if (props.colors.size() < 2) {
+      throw std::invalid_argument("Colors array must have at least 2 colors");
+    }
+    
+    // Validate positions array matches colors array in size
+    if (props.positions.has_value() && props.positions.value().size() != props.colors.size()) {
+      throw std::invalid_argument("Positions array must have the same size as colors array");
+    }
+    
     SkMatrix m3 = processTransform(props.matrix, props.transform, props.origin);
     auto shader = SkGradientShader::MakeRadial(
         props.center, props.radius, props.colors.data(),
@@ -314,6 +334,16 @@ public:
   }
 
   void pushShader(DrawingCtx *ctx) {
+    // Validate colors array has at least 2 colors
+    if (props.colors.size() < 2) {
+      throw std::invalid_argument("Colors array must have at least 2 colors");
+    }
+    
+    // Validate positions array matches colors array in size
+    if (props.positions.has_value() && props.positions.value().size() != props.colors.size()) {
+      throw std::invalid_argument("Positions array must have the same size as colors array");
+    }
+    
     SkMatrix m3 = processTransform(props.matrix, props.transform, props.origin);
     auto shader = SkGradientShader::MakeSweep(
         props.center.x(), props.center.y(), props.colors.data(),
@@ -355,6 +385,16 @@ public:
   }
 
   void pushShader(DrawingCtx *ctx) {
+    // Validate colors array has at least 2 colors
+    if (props.colors.size() < 2) {
+      throw std::invalid_argument("Colors array must have at least 2 colors");
+    }
+    
+    // Validate positions array matches colors array in size
+    if (props.positions.has_value() && props.positions.value().size() != props.colors.size()) {
+      throw std::invalid_argument("Positions array must have the same size as colors array");
+    }
+    
     SkMatrix m3 = processTransform(props.matrix, props.transform, props.origin);
     auto shader = SkGradientShader::MakeTwoPointConical(
         props.start, props.startRadius, props.end, props.endRadius,

package/cpp/rnskia/DawnContext.h

@@ -51,7 +51,7 @@ async_callback(void *c,
 class DawnContext {
 public:
   // TODO: remove
-  friend class RNSkiOSPlatformContext;
+  friend class RNSkApplePlatformContext;
 
   DawnContext(const DawnContext &) = delete;
   DawnContext &operator=(const DawnContext &) = delete;

package/cpp/skia/include/codec/SkCodec.h

@@ -412,6 +412,9 @@ public:
      *  If a scanline decode is in progress, scanline mode will end, requiring the client to call
      *  startScanlineDecode() in order to return to decoding scanlines.
      *
+     *  For certain codecs, reading into a smaller bitmap than the original dimensions may not
+     *  produce correct results (e.g. animated webp).
+     *
      *  @return Result kSuccess, or another value explaining the type of failure.
      */
     Result getPixels(const SkImageInfo& info, void* pixels, size_t rowBytes, const Options*);
@@ -812,6 +815,7 @@ protected:
     }
 
     virtual bool onGetGainmapCodec(SkGainmapInfo*, std::unique_ptr<SkCodec>*) { return false; }
+    virtual bool onGetGainmapInfo(SkGainmapInfo*) { return false; }
 
     // TODO(issues.skia.org/363544350): This API only works for JPEG images. Remove this API once
     // it is no longer used.
@@ -1053,8 +1057,9 @@ private:
     friend class PNGCodecGM;    // for fillIncompleteImage
     friend class SkSampledCodec;
     friend class SkIcoCodec;
-    friend class SkAndroidCodec; // for fEncodedInfo
-    friend class SkPDFBitmap; // for fEncodedInfo
+    friend class SkPngCodec;     // for onGetGainmapCodec
+    friend class SkAndroidCodec;  // for handleFrameIndex
+    friend class SkCodecPriv;     // for fEncodedInfo
 };
 
 namespace SkCodecs {