@shopify/react-native-skia 1.5.3 → 1.5.4

package/cpp/skia/include/ports/SkFontScanner_FreeType.h

@@ -0,0 +1,15 @@
+/*
+* Copyright 2024 Google Inc.
+*
+* Use of this source code is governed by a BSD-style license that can be
+* found in the LICENSE file.
+*/
+
+#ifndef SKFONTSCANNER_FREETYPE_H_
+#define SKFONTSCANNER_FREETYPE_H_
+
+#include "include/core/SkFontScanner.h"
+
+SK_API std::unique_ptr<SkFontScanner> SkFontScanner_Make_FreeType();
+
+#endif // SKFONTSCANNER_FREETYPE_H_

package/cpp/skia/include/private/SkExif.h

@@ -9,6 +9,7 @@
 #define SkExif_DEFINED
 
 #include "include/codec/SkEncodedOrigin.h"
+#include "include/core/SkRefCnt.h"
 #include "include/private/base/SkAPI.h"
 
 #include <cstdint>
@@ -50,6 +51,19 @@ struct Metadata {
  */
 void SK_API Parse(Metadata& metadata, const SkData* data);
 
+/*
+ * Write exif data that includes the values in |metadata| and returns it as SkData
+ * untruncated. Return nullptr if a write to the data stream fails or if
+ * |metadata.fHdrHeadroom| has value.
+ * This function cannot write an IFD entry based on the HdrHeadroom value because
+ * the information of maker33 and maker48 are lost in decoding.
+ * For metadata that belongs in a subIFD the function will write it to one and
+ * store it after the root IFD.
+ * Data that does not fit within kSizeEntry, is appended to the end of the data,
+ * after the subIFD if it exists.
+ */
+sk_sp<SkData> WriteExif(Metadata& metadata);
+
 }  // namespace SkExif
 
 #endif

package/cpp/skia/include/private/SkPathRef.h

@@ -8,6 +8,7 @@
 #ifndef SkPathRef_DEFINED
 #define SkPathRef_DEFINED
 
+#include "include/core/SkArc.h"
 #include "include/core/SkPoint.h"
 #include "include/core/SkRect.h"
 #include "include/core/SkRefCnt.h"
@@ -63,6 +64,7 @@ public:
         kGeneral,
         kOval,
         kRRect,
+        kArc,
     };
 
     SkPathRef(SkSpan<const SkPoint> points, SkSpan<const uint8_t> verbs,
@@ -78,6 +80,9 @@ public:
         // The next two values don't matter unless fType is kOval or kRRect
         fRRectOrOvalIsCCW = false;
         fRRectOrOvalStartIdx = 0xAC;
+        fArcOval.setEmpty();
+        fArcStartAngle = fArcSweepAngle = 0.0f;
+        fArcType = SkArc::Type::kArc;
         SkDEBUGCODE(fEditorsAttached.store(0);)
 
         this->computeBounds();  // do this now, before we worry about multiple owners/threads
@@ -160,6 +165,10 @@ public:
             fPathRef->setIsRRect(isCCW, start);
         }
 
+        void setIsArc(const SkArc& arc) {
+            fPathRef->setIsArc(arc);
+        }
+
         void setBounds(const SkRect& rect) { fPathRef->setBounds(rect); }
 
     private:
@@ -249,6 +258,16 @@ public:
 
     bool isRRect(SkRRect* rrect, bool* isCCW, unsigned* start) const;
 
+    bool isArc(SkArc* arc) const {
+        if (fType == PathType::kArc) {
+            if (arc) {
+                *arc = SkArc::Make(fArcOval, fArcStartAngle, fArcSweepAngle, fArcType);
+            }
+        }
+
+        return fType == PathType::kArc;
+    }
+
     bool hasComputedBounds() const {
         return !fBoundsIsDirty;
     }
@@ -365,6 +384,9 @@ private:
         // The next two values don't matter unless fType is kOval or kRRect
         fRRectOrOvalIsCCW = false;
         fRRectOrOvalStartIdx = 0xAC;
+        fArcOval.setEmpty();
+        fArcStartAngle = fArcSweepAngle = 0.0f;
+        fArcType = SkArc::Type::kArc;
         if (numPoints > 0) {
             fPoints.reserve_exact(numPoints);
         }
@@ -497,6 +519,14 @@ private:
         fRRectOrOvalStartIdx = SkToU8(start);
     }
 
+    void setIsArc(const SkArc& arc) {
+        fType = PathType::kArc;
+        fArcOval = arc.fOval;
+        fArcStartAngle = arc.fStartAngle;
+        fArcSweepAngle = arc.fSweepAngle;
+        fArcType = arc.fType;
+    }
+
     // called only by the editor. Note that this is not a const function.
     SkPoint* getWritablePoints() {
         SkDEBUGCODE(this->validate();)
@@ -534,6 +564,12 @@ private:
     bool     fRRectOrOvalIsCCW;
     uint8_t  fRRectOrOvalStartIdx;
     uint8_t  fSegmentMask;
+    // If the path is an arc, these four variables store that information.
+    // We should just store an SkArc, but alignment would cost us 8 more bytes.
+    SkArc::Type fArcType;
+    SkRect      fArcOval;
+    SkScalar    fArcStartAngle;
+    SkScalar    fArcSweepAngle;
 
     friend class PathRefTest_Private;
     friend class ForceIsRRect_Private; // unit test isRRect

package/cpp/skia/include/private/base/SkFeatures.h

@@ -69,6 +69,10 @@
   #define SK_CPU_LOONGARCH 1
 #endif
 
+#if defined(__powerpc__) || defined (__powerpc64__)
+  #define SK_CPU_PPC 1
+#endif
+
 /**
  *  SK_CPU_SSE_LEVEL
  *

package/cpp/skia/include/private/base/SkSpan_impl.h

@@ -69,7 +69,7 @@ public:
     template <typename Integer, std::enable_if_t<std::is_integral_v<Integer>, bool> = true>
     constexpr SkSpan(T* ptr, Integer size) : fPtr{ptr}, fSize{SkToSizeT(size)} {
         SkASSERT(ptr || fSize == 0);  // disallow nullptr + a nonzero size
-        SkASSERT(fSize < kMaxSize);
+        SkASSERT(fSize < (std::numeric_limits<size_t>::max() / sizeof(T)));
     }
     template <typename U, typename = std::enable_if_t<std::is_same_v<const U, T>>>
     constexpr SkSpan(const SkSpan<U>& that) : fPtr(std::data(that)), fSize(std::size(that)) {}
@@ -118,8 +118,6 @@ public:
     }
 
 private:
-    static constexpr size_t kMaxSize = std::numeric_limits<size_t>::max() / sizeof(T);
-
     T* fPtr;
     size_t fSize;
 };

package/cpp/skia/modules/skcms/src/skcms_internals.h

@@ -39,6 +39,8 @@ extern "C" {
         // - Clang 18 runs into an ICE on armv7/androideabi with [[clang::musttail]].
         //   (http://crbug.com/1504548)
         // - Android RISC-V also runs into an ICE (b/314692534)
+        // - So does Linux ppc64le (https://github.com/llvm/llvm-project/issues/108014,
+        //   https://github.com/llvm/llvm-project/issues/98859)
         // - LoongArch developers indicate they had to turn it off
         // - Windows builds generate incorrect code with [[clang::musttail]] and crash mysteriously.
         //   (http://crbug.com/1505442)
@@ -47,6 +49,7 @@ extern "C" {
                                                  && !defined(__EMSCRIPTEN__) \
                                                  && !defined(__arm__) \
                                                  && !defined(__riscv) \
+                                                 && !defined(__powerpc__) \
                                                  && !defined(__loongarch__) \
                                                  && !defined(_WIN32) && !defined(__SYMBIAN32__)
             #define SKCMS_HAS_MUSTTAIL 1

package/cpp/skia/src/base/SkMathPriv.h

@@ -300,17 +300,10 @@ constexpr int SkPrevPow2_portable(int value) {
 
 ///////////////////////////////////////////////////////////////////////////////
 
-/**
- *  Return the smallest power-of-2 >= n.
- */
-static inline uint32_t GrNextPow2(uint32_t n) {
-    return n ? (1 << (32 - SkCLZ(n - 1))) : 1;
-}
-
 /**
  * Returns the next power of 2 >= n or n if the next power of 2 can't be represented by size_t.
  */
-static inline size_t GrNextSizePow2(size_t n) {
+constexpr size_t SkNextSizePow2(size_t n) {
     constexpr int kNumSizeTBits = 8 * sizeof(size_t);
     constexpr size_t kHighBitSet = size_t(1) << (kNumSizeTBits - 1);
 

package/ios/RNSkia-iOS/RNSkMetalCanvasProvider.h

@@ -30,8 +30,7 @@ public:
 
 private:
   std::shared_ptr<RNSkia::RNSkPlatformContext> _context;
-  float _width = -1;
-  float _height = -1;
+  std::unique_ptr<RNSkia::WindowContext> _ctx = nullptr;
 #pragma clang diagnostic push
 #pragma clang diagnostic ignored "-Wunguarded-availability-new"
   CAMetalLayer *_layer;

package/ios/RNSkia-iOS/RNSkMetalCanvasProvider.mm

@@ -1,7 +1,13 @@
 #import "RNSkMetalCanvasProvider.h"
-#include "MetalContext.h"
+
 #import "RNSkLog.h"
 
+#if defined(SK_GRAPHITE)
+#import "DawnContext.h"
+#else
+#import "MetalContext.h"
+#endif
+
 #pragma clang diagnostic push
 #pragma clang diagnostic ignored "-Wdocumentation"
 
@@ -30,19 +36,23 @@ RNSkMetalCanvasProvider::~RNSkMetalCanvasProvider() {}
 /**
  Returns the scaled width of the view
  */
-float RNSkMetalCanvasProvider::getScaledWidth() { return _width; };
+float RNSkMetalCanvasProvider::getScaledWidth() {
+  return _ctx ? _ctx->getWidth() : -1;
+};
 
 /**
  Returns the scaled height of the view
  */
-float RNSkMetalCanvasProvider::getScaledHeight() { return _height; };
+float RNSkMetalCanvasProvider::getScaledHeight() {
+  return _ctx ? _ctx->getHeight() : -1;
+};
 
 /**
  Render to a canvas
  */
 bool RNSkMetalCanvasProvider::renderToCanvas(
     const std::function<void(SkCanvas *)> &cb) {
-  if (_width <= 0 || _height <= 0) {
+  if (!_ctx) {
     return false;
   }
 
@@ -66,28 +76,24 @@ bool RNSkMetalCanvasProvider::renderToCanvas(
   // rendering and not wait until later - we've seen some example of memory
   // usage growing very fast in the simulator without this.
   @autoreleasepool {
-    id<CAMetalDrawable> currentDrawable = [_layer nextDrawable];
-    if (currentDrawable == nullptr) {
-      return false;
-    }
-    auto ctx = MetalContext::getInstance().MakeWindow(_layer, _width, _height);
-    auto skSurface = ctx->getSurface();
-    SkCanvas *canvas = skSurface->getCanvas();
+    auto surface = _ctx->getSurface();
+    auto canvas = surface->getCanvas();
     cb(canvas);
-
-    if (auto dContext = GrAsDirectContext(skSurface->recordingContext())) {
-      dContext->flushAndSubmit();
-    }
-
-    ctx->present();
+    _ctx->present();
   }
   return true;
 };
 
 void RNSkMetalCanvasProvider::setSize(int width, int height) {
   _layer.frame = CGRectMake(0, 0, width, height);
-  _width = width * _context->getPixelDensity();
-  _height = height * _context->getPixelDensity();
+  auto w = width * _context->getPixelDensity();
+  auto h = height * _context->getPixelDensity();
+#if defined(SK_GRAPHITE)
+  _ctx = RNSkia::DawnContext::getInstance().MakeWindow((__bridge void *)_layer,
+                                                       w, h);
+#else
+  _ctx = MetalContext::getInstance().MakeWindow(_layer, w, h);
+#endif
   _requestRedraw();
 }
 

package/ios/RNSkia-iOS/RNSkiOSPlatformContext.mm

@@ -5,7 +5,11 @@
 #include <thread>
 #include <utility>
 
+#if defined(SK_GRAPHITE)
+#include "DawnContext.h"
+#else
 #include "MetalContext.h"
+#endif
 #include "RNSkiOSVideo.h"
 
 #pragma clang diagnostic push
@@ -69,10 +73,17 @@ void RNSkiOSPlatformContext::releaseNativeBuffer(uint64_t pointer) {
 uint64_t RNSkiOSPlatformContext::makeNativeBuffer(sk_sp<SkImage> image) {
   // 0. If Image is not in BGRA, convert to BGRA as only BGRA is supported.
   if (image->colorType() != kBGRA_8888_SkColorType) {
+#if defined(SK_GRAPHITE)
+    SkImage::RequiredProperties requiredProps;
+    image = image->makeColorTypeAndColorSpace(
+        DawnContext::getInstance().getRecorder(), kBGRA_8888_SkColorType,
+        SkColorSpace::MakeSRGB(), requiredProps);
+#else
     // on iOS, 32_BGRA is the only supported RGB format for CVPixelBuffers.
     image = image->makeColorTypeAndColorSpace(
         MetalContext::getInstance()._context.skContext.get(),
         kBGRA_8888_SkColorType, SkColorSpace::MakeSRGB());
+#endif
     if (image == nullptr) {
       throw std::runtime_error(
           "Failed to convert image to BGRA_8888 colortype! Only BGRA_8888 "
@@ -152,8 +163,12 @@ RNSkiOSPlatformContext::createVideo(const std::string &url) {
 std::shared_ptr<WindowContext>
 RNSkiOSPlatformContext::makeContextFromNativeSurface(void *surface, int width,
                                                      int height) {
+#if defined(SK_GRAPHITE)
+  return DawnContext::getInstance().MakeWindow(surface, width, height);
+#else
   return MetalContext::getInstance().MakeWindow((__bridge CALayer *)surface,
                                                 width, height);
+#endif
 }
 
 void RNSkiOSPlatformContext::raiseError(const std::exception &err) {
@@ -162,11 +177,19 @@ void RNSkiOSPlatformContext::raiseError(const std::exception &err) {
 
 sk_sp<SkSurface> RNSkiOSPlatformContext::makeOffscreenSurface(int width,
                                                               int height) {
+#if defined(SK_GRAPHITE)
+  return DawnContext::getInstance().MakeOffscreen(width, height);
+#else
   return MetalContext::getInstance().MakeOffscreen(width, height);
+#endif
 }
 
 sk_sp<SkImage> RNSkiOSPlatformContext::makeImageFromNativeBuffer(void *buffer) {
+#if defined(SK_GRAPHITE)
+  return DawnContext::getInstance().MakeImageFromBuffer(buffer);
+#else
   return MetalContext::getInstance().MakeImageFromBuffer(buffer);
+#endif
 }
 
 sk_sp<SkFontMgr> RNSkiOSPlatformContext::createFontMgr() {

package/ios/RNSkia-iOS/SkiaMetalSurfaceFactory.h

@@ -11,9 +11,19 @@
 #pragma clang diagnostic ignored "-Wdocumentation"
 
 #import "include/core/SkCanvas.h"
+#import "include/core/SkColorSpace.h"
+
 #import <CoreMedia/CMSampleBuffer.h>
 #import <CoreVideo/CVMetalTextureCache.h>
+
+#import <include/gpu/ganesh/GrBackendSurface.h>
 #import <include/gpu/ganesh/GrDirectContext.h>
+#import <include/gpu/ganesh/SkImageGanesh.h>
+#import <include/gpu/ganesh/SkSurfaceGanesh.h>
+#import <include/gpu/ganesh/mtl/GrMtlBackendContext.h>
+#import <include/gpu/ganesh/mtl/GrMtlBackendSurface.h>
+#import <include/gpu/ganesh/mtl/GrMtlDirectContext.h>
+#import <include/gpu/ganesh/mtl/SkSurfaceMetal.h>
 
 #pragma clang diagnostic pop
 
@@ -26,9 +36,6 @@ class SkiaMetalSurfaceFactory {
   friend class IOSSkiaContext;
 
 public:
-  static sk_sp<SkSurface> makeWindowedSurface(SkiaMetalContext *context,
-                                              id<MTLTexture> texture, int width,
-                                              int height);
   static sk_sp<SkSurface> makeOffscreenSurface(id<MTLDevice> device,
                                                SkiaMetalContext *context,
                                                int width, int height);
@@ -76,9 +83,16 @@ public:
     }
 
     // Get the texture from the drawable
-    _skSurface = SkiaMetalSurfaceFactory::makeWindowedSurface(
-        _context, _currentDrawable.texture, _layer.drawableSize.width,
-        _layer.drawableSize.height);
+    GrMtlTextureInfo fbInfo;
+    fbInfo.fTexture.retain((__bridge void *)_currentDrawable.texture);
+
+    GrBackendRenderTarget backendRT = GrBackendRenderTargets::MakeMtl(
+        _layer.drawableSize.width, _layer.drawableSize.height, fbInfo);
+
+    _skSurface = SkSurfaces::WrapBackendRenderTarget(
+        _context->skContext.get(), backendRT, kTopLeft_GrSurfaceOrigin,
+        kBGRA_8888_SkColorType, nullptr, nullptr);
+
     return _skSurface;
   }
 
@@ -95,9 +109,9 @@ public:
 
   void resize(int width, int height) override { _skSurface = nullptr; }
 
-  int getWidth() override { return _layer.frame.size.width; };
+  int getWidth() override { return _layer.frame.size.width * _layer.contentsScale; };
 
-  int getHeight() override { return _layer.frame.size.height; };
+  int getHeight() override { return _layer.frame.size.height * _layer.contentsScale; };
 
 private:
   SkiaMetalContext *_context;

package/ios/RNSkia-iOS/SkiaMetalSurfaceFactory.mm

@@ -46,26 +46,6 @@ SkiaMetalSurfaceFactory::makeContext(SkiaMetalContext *context,
   return std::make_unique<IOSSkiaContext>(context, texture, width, height);
 }
 
-sk_sp<SkSurface> SkiaMetalSurfaceFactory::makeWindowedSurface(
-    SkiaMetalContext *context, id<MTLTexture> texture, int width, int height) {
-  GrMtlTextureInfo fbInfo;
-  fbInfo.fTexture.retain((__bridge void *)texture);
-
-  GrBackendRenderTarget backendRT =
-      GrBackendRenderTargets::MakeMtl(width, height, fbInfo);
-
-  auto skSurface = SkSurfaces::WrapBackendRenderTarget(
-      context->skContext.get(), backendRT, kTopLeft_GrSurfaceOrigin,
-      kBGRA_8888_SkColorType, nullptr, nullptr);
-
-  if (skSurface == nullptr || skSurface->getCanvas() == nullptr) {
-    RNSkia::RNSkLogger::logToConsole(
-        "Skia surface could not be created from parameters.");
-    return nullptr;
-  }
-  return skSurface;
-}
-
 sk_sp<SkSurface> SkiaMetalSurfaceFactory::makeOffscreenSurface(
     id<MTLDevice> device, SkiaMetalContext *context, int width, int height) {
 

package/libs/ios/libskshaper.xcframework/Info.plist

@@ -8,34 +8,34 @@
 			<key>BinaryPath</key>
 			<string>libskshaper.a</string>
 			<key>LibraryIdentifier</key>
-			<string>ios-arm64_arm64e</string>
+			<string>ios-arm64_arm64e_x86_64-simulator</string>
 			<key>LibraryPath</key>
 			<string>libskshaper.a</string>
 			<key>SupportedArchitectures</key>
 			<array>
 				<string>arm64</string>
 				<string>arm64e</string>
+				<string>x86_64</string>
 			</array>
 			<key>SupportedPlatform</key>
 			<string>ios</string>
+			<key>SupportedPlatformVariant</key>
+			<string>simulator</string>
 		</dict>
 		<dict>
 			<key>BinaryPath</key>
 			<string>libskshaper.a</string>
 			<key>LibraryIdentifier</key>
-			<string>ios-arm64_arm64e_x86_64-simulator</string>
+			<string>ios-arm64_arm64e</string>
 			<key>LibraryPath</key>
 			<string>libskshaper.a</string>
 			<key>SupportedArchitectures</key>
 			<array>
 				<string>arm64</string>
 				<string>arm64e</string>
-				<string>x86_64</string>
 			</array>
 			<key>SupportedPlatform</key>
 			<string>ios</string>
-			<key>SupportedPlatformVariant</key>
-			<string>simulator</string>
 		</dict>
 	</array>
 	<key>CFBundlePackageType</key>

package/libs/ios/libskunicode_core.xcframework/Info.plist

@@ -8,34 +8,34 @@
 			<key>BinaryPath</key>
 			<string>libskunicode_core.a</string>
 			<key>LibraryIdentifier</key>
-			<string>ios-arm64_arm64e</string>
+			<string>ios-arm64_arm64e_x86_64-simulator</string>
 			<key>LibraryPath</key>
 			<string>libskunicode_core.a</string>
 			<key>SupportedArchitectures</key>
 			<array>
 				<string>arm64</string>
 				<string>arm64e</string>
+				<string>x86_64</string>
 			</array>
 			<key>SupportedPlatform</key>
 			<string>ios</string>
+			<key>SupportedPlatformVariant</key>
+			<string>simulator</string>
 		</dict>
 		<dict>
 			<key>BinaryPath</key>
 			<string>libskunicode_core.a</string>
 			<key>LibraryIdentifier</key>
-			<string>ios-arm64_arm64e_x86_64-simulator</string>
+			<string>ios-arm64_arm64e</string>
 			<key>LibraryPath</key>
 			<string>libskunicode_core.a</string>
 			<key>SupportedArchitectures</key>
 			<array>
 				<string>arm64</string>
 				<string>arm64e</string>
-				<string>x86_64</string>
 			</array>
 			<key>SupportedPlatform</key>
 			<string>ios</string>
-			<key>SupportedPlatformVariant</key>
-			<string>simulator</string>
 		</dict>
 	</array>
 	<key>CFBundlePackageType</key>

package/libs/ios/libskunicode_libgrapheme.xcframework/Info.plist

@@ -8,34 +8,34 @@
 			<key>BinaryPath</key>
 			<string>libskunicode_libgrapheme.a</string>
 			<key>LibraryIdentifier</key>
-			<string>ios-arm64_arm64e</string>
+			<string>ios-arm64_arm64e_x86_64-simulator</string>
 			<key>LibraryPath</key>
 			<string>libskunicode_libgrapheme.a</string>
 			<key>SupportedArchitectures</key>
 			<array>
 				<string>arm64</string>
 				<string>arm64e</string>
+				<string>x86_64</string>
 			</array>
 			<key>SupportedPlatform</key>
 			<string>ios</string>
+			<key>SupportedPlatformVariant</key>
+			<string>simulator</string>
 		</dict>
 		<dict>
 			<key>BinaryPath</key>
 			<string>libskunicode_libgrapheme.a</string>
 			<key>LibraryIdentifier</key>
-			<string>ios-arm64_arm64e_x86_64-simulator</string>
+			<string>ios-arm64_arm64e</string>
 			<key>LibraryPath</key>
 			<string>libskunicode_libgrapheme.a</string>
 			<key>SupportedArchitectures</key>
 			<array>
 				<string>arm64</string>
 				<string>arm64e</string>
-				<string>x86_64</string>
 			</array>
 			<key>SupportedPlatform</key>
 			<string>ios</string>
-			<key>SupportedPlatformVariant</key>
-			<string>simulator</string>
 		</dict>
 	</array>
 	<key>CFBundlePackageType</key>

package/package.json

@@ -7,7 +7,7 @@
     "setup-skia-web": "./scripts/setup-canvaskit.js"
   },
   "title": "React Native Skia",
-  "version": "1.5.3",
+  "version": "1.5.4",
   "description": "High-performance React Native Graphics using Skia",
   "main": "lib/module/index.js",
   "react-native": "src/index.ts",