@shopify/react-native-skia 1.5.6 → 1.5.8

package/android/CMakeLists.txt

@@ -4,7 +4,7 @@ cmake_minimum_required(VERSION 3.4.1)
 set (CMAKE_VERBOSE_MAKEFILE ON)
 set (CMAKE_CXX_STANDARD 17)
 set(SK_GRAPHITE OFF)
-set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -DSK_BUILD_FOR_ANDROID -DFOLLY_NO_CONFIG=1 -DFOLLY_HAVE_CLOCK_GETTIME=1 -DFOLLY_HAVE_MEMRCHR=1 -DFOLLY_USE_LIBCPP=1 -DFOLLY_MOBILE=1 -DON_ANDROID -DONANDROID")
+set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -DSK_BUILD_FOR_ANDROID -DSK_IMAGE_READ_PIXELS_DISABLE_LEGACY_API -DFOLLY_NO_CONFIG=1 -DFOLLY_HAVE_CLOCK_GETTIME=1 -DFOLLY_HAVE_MEMRCHR=1 -DFOLLY_USE_LIBCPP=1 -DFOLLY_MOBILE=1 -DON_ANDROID -DONANDROID")
 set (PACKAGE_NAME "rnskia")
 set (SKIA_LIB "skia")
 set (SKIA_SVG_LIB "svg")

package/android/cpp/jni/include/JniSkiaBaseView.h

@@ -33,8 +33,8 @@ protected:
     _skiaAndroidView->surfaceAvailable(surface, width, height);
   }
 
-  virtual void surfaceSizeChanged(int width, int height) {
-    _skiaAndroidView->surfaceSizeChanged(width, height);
+  virtual void surfaceSizeChanged(jobject surface, int width, int height) {
+    _skiaAndroidView->surfaceSizeChanged(surface, width, height);
   }
 
   virtual void surfaceDestroyed() { _skiaAndroidView->surfaceDestroyed(); }

package/android/cpp/jni/include/JniSkiaDomView.h

@@ -51,8 +51,8 @@ protected:
     JniSkiaBaseView::surfaceAvailable(surface, width, height);
   }
 
-  void surfaceSizeChanged(int width, int height) override {
-    JniSkiaBaseView::surfaceSizeChanged(width, height);
+  void surfaceSizeChanged(jobject surface, int width, int height) override {
+    JniSkiaBaseView::surfaceSizeChanged(surface, width, height);
   }
 
   void surfaceDestroyed() override { JniSkiaBaseView::surfaceDestroyed(); }

package/android/cpp/jni/include/JniSkiaPictureView.h

@@ -53,8 +53,8 @@ protected:
     JniSkiaBaseView::surfaceAvailable(surface, width, height);
   }
 
-  void surfaceSizeChanged(int width, int height) override {
-    JniSkiaBaseView::surfaceSizeChanged(width, height);
+  void surfaceSizeChanged(jobject surface, int width, int height) override {
+    JniSkiaBaseView::surfaceSizeChanged(surface, width, height);
   }
 
   void surfaceDestroyed() override { JniSkiaBaseView::surfaceDestroyed(); }

package/android/cpp/rnskia-android/OpenGLContext.h

@@ -125,11 +125,15 @@ public:
 #endif
   }
 
+  // TODO: remove width, height
   std::unique_ptr<WindowContext> MakeWindow(ANativeWindow *window, int width,
                                             int height) {
-    return std::make_unique<OpenGLWindowContext>(this, window, width, height);
+    return std::make_unique<OpenGLWindowContext>(
+        _directContext.get(), _glDisplay.get(), _glContext.get(), window);
   }
 
+  GrDirectContext *getDirectContext() { return _directContext.get(); }
+
 private:
   EGLConfig _glConfig;
   std::unique_ptr<gl::Display> _glDisplay;

package/android/cpp/rnskia-android/OpenGLWindowContext.cpp

@@ -16,74 +16,45 @@ namespace RNSkia {
 
 sk_sp<SkSurface> OpenGLWindowContext::getSurface() {
   if (_skSurface == nullptr) {
-
-    struct ReleaseContext {
-      std::unique_ptr<gl::Surface> surface = nullptr;
-    };
-
-    if (!_window) {
-      throw std::runtime_error("No native window provided");
-    }
-    auto releaseCtx = new ReleaseContext();
-    releaseCtx->surface =
-        _context->_glDisplay->makeWindowSurface(_context->_glConfig, _window);
-    if (!releaseCtx->surface) {
-      throw std::runtime_error("Failed to create window surface");
-    }
-    _glSurface = releaseCtx->surface.get();
-
-    // Now make this one current
-    auto success = _context->_glContext->makeCurrent(releaseCtx->surface.get());
-    if (!success) {
-      throw std::runtime_error("Failed to make window surface current");
-    }
-
-    // Set up parameters for the render target so that it
-    // matches the underlying OpenGL context.
-    GrGLFramebufferInfo fboInfo;
-
-    // We pass 0 as the framebuffer id, since the
-    // underlying Skia GrGlGpu will read this when wrapping the context in the
-    // render target and the GrGlGpu object.
-    fboInfo.fFBOID = 0;
-    fboInfo.fFormat = 0x8058; // GL_RGBA8
-
+    _glContext->makeCurrent(_glSurface.get());
     GLint stencil;
     glGetIntegerv(GL_STENCIL_BITS, &stencil);
 
     GLint samples;
     glGetIntegerv(GL_SAMPLES, &samples);
 
-    auto colorType = kN32_SkColorType;
+    auto colorType = kRGBA_8888_SkColorType;
 
     auto maxSamples =
-        _context->_directContext->maxSurfaceSampleCountForColorType(colorType);
+        _directContext->maxSurfaceSampleCountForColorType(colorType);
 
     if (samples > maxSamples) {
       samples = maxSamples;
     }
 
-    auto renderTarget = GrBackendRenderTargets::MakeGL(_width, _height, samples,
-                                                       stencil, fboInfo);
-
-    SkSurfaceProps props(0, kUnknown_SkPixelGeometry);
-
-    // Create surface object
+    GrGLFramebufferInfo fbInfo;
+    fbInfo.fFBOID = 0;
+    fbInfo.fFormat = GR_GL_RGBA8;
+    // fbInfo.fProtected =
+    // skgpu::Protected(fDisplayParams.fCreateProtectedNativeBackend);
+
+    auto width = ANativeWindow_getWidth(_window);
+    auto height = ANativeWindow_getHeight(_window);
+    auto backendRT =
+        GrBackendRenderTargets::MakeGL(width, height, samples, stencil, fbInfo);
+    sk_sp<SkColorSpace> colorSpace(nullptr);
+    SkSurfaceProps surfaceProps(0, kRGB_H_SkPixelGeometry);
     _skSurface = SkSurfaces::WrapBackendRenderTarget(
-        _context->_directContext.get(), renderTarget,
-        kBottomLeft_GrSurfaceOrigin, colorType, nullptr, &props,
-        [](void *addr) {
-          auto releaseCtx = reinterpret_cast<ReleaseContext *>(addr);
-          delete releaseCtx;
-        },
-        reinterpret_cast<void *>(releaseCtx));
+        _directContext, backendRT, kBottomLeft_GrSurfaceOrigin,
+        kRGBA_8888_SkColorType, colorSpace, &surfaceProps);
   }
   return _skSurface;
 }
 
 void OpenGLWindowContext::present() {
-  _context->_glContext->makeCurrent(_glSurface);
-  _context->_directContext->flushAndSubmit();
+  _glContext->makeCurrent(_glSurface.get());
+  // TODO: is flushAndSubmit needed here?
+  _directContext->flushAndSubmit();
   _glSurface->present();
 }
 

package/android/cpp/rnskia-android/OpenGLWindowContext.h

@@ -31,14 +31,15 @@
 
 namespace RNSkia {
 
-class OpenGLContext;
-
 class OpenGLWindowContext : public WindowContext {
 public:
-  OpenGLWindowContext(OpenGLContext *context, ANativeWindow *window, int width,
-                      int height)
-      : _context(context), _window(window), _width(width), _height(height) {
+  OpenGLWindowContext(GrDirectContext *directContext, gl::Display *display,
+                      gl::Context *glContext, ANativeWindow *window)
+      : _directContext(directContext), _display(display), _glContext(glContext),
+        _window(window) {
     ANativeWindow_acquire(_window);
+    auto config = display->chooseConfig();
+    _glSurface = display->makeWindowSurface(config, _window);
   }
 
   ~OpenGLWindowContext() {
@@ -51,23 +52,19 @@ public:
 
   void present() override;
 
-  void resize(int width, int height) override {
-    _skSurface = nullptr;
-    _width = width;
-    _height = height;
-  }
+  int getWidth() override { return ANativeWindow_getWidth(_window); };
 
-  int getWidth() override { return _width; };
+  int getHeight() override { return ANativeWindow_getHeight(_window); };
 
-  int getHeight() override { return _height; };
+  void resize(int width, int height) override { _skSurface = nullptr; }
 
 private:
-  OpenGLContext *_context;
+  GrDirectContext *_directContext;
+  gl::Display *_display;
+  gl::Context *_glContext = nullptr;
   ANativeWindow *_window;
   sk_sp<SkSurface> _skSurface = nullptr;
-  gl::Surface *_glSurface = nullptr;
-  int _width = 0;
-  int _height = 0;
+  std::unique_ptr<gl::Surface> _glSurface = nullptr;
 };
 
 } // namespace RNSkia

package/android/cpp/rnskia-android/RNSkAndroidPlatformContext.h

@@ -152,6 +152,12 @@ public:
 #endif
   }
 
+#if !defined(SK_GRAPHITE)
+  GrDirectContext *getDirectContext() override {
+    return OpenGLContext::getInstance().getDirectContext();
+  }
+#endif
+
   sk_sp<SkFontMgr> createFontMgr() override {
     return SkFontMgr_New_Android(nullptr);
   }

package/android/cpp/rnskia-android/RNSkAndroidView.h

@@ -15,7 +15,7 @@ public:
 
   virtual void surfaceDestroyed() = 0;
 
-  virtual void surfaceSizeChanged(int width, int height) = 0;
+  virtual void surfaceSizeChanged(jobject surface, int width, int height) = 0;
 
   virtual float getPixelDensity() = 0;
 
@@ -50,9 +50,9 @@ public:
         ->surfaceDestroyed();
   }
 
-  void surfaceSizeChanged(int width, int height) override {
+  void surfaceSizeChanged(jobject surface, int width, int height) override {
     std::static_pointer_cast<RNSkOpenGLCanvasProvider>(T::getCanvasProvider())
-        ->surfaceSizeChanged(width, height);
+        ->surfaceSizeChanged(surface, width, height);
     // This is only need for the first time to frame, this renderImmediate call
     // will invoke updateTexImage for the previous frame
     RNSkView::renderImmediate();

package/android/cpp/rnskia-android/RNSkOpenGLCanvasProvider.cpp

@@ -76,6 +76,10 @@ bool RNSkOpenGLCanvasProvider::renderToCanvas(
 
 void RNSkOpenGLCanvasProvider::surfaceAvailable(jobject jSurfaceTexture,
                                                 int width, int height) {
+  // If the surface is 0, we can skip it
+  if (width == 0 && height == 0) {
+    return;
+  }
   // Create renderer!
   JNIEnv *env = facebook::jni::Environment::current();
 
@@ -118,17 +122,22 @@ void RNSkOpenGLCanvasProvider::surfaceDestroyed() {
   }
 }
 
-void RNSkOpenGLCanvasProvider::surfaceSizeChanged(int width, int height) {
+void RNSkOpenGLCanvasProvider::surfaceSizeChanged(jobject jSurfaceTexture,
+                                                  int width, int height) {
   if (width == 0 && height == 0) {
     // Setting width/height to zero is nothing we need to care about when
     // it comes to invalidating the surface.
     return;
   }
 
-  // Recreate RenderContext surface based on size change???
-  _surfaceHolder->resize(width, height);
+  if (_surfaceHolder == nullptr) {
+    _surfaceHolder = nullptr;
+    surfaceAvailable(jSurfaceTexture, width, height);
+  } else {
+    _surfaceHolder->resize(width, height);
+  }
 
   // Redraw after size change
   _requestRedraw();
 }
-} // namespace RNSkia
+} // namespace RNSkia

package/android/cpp/rnskia-android/RNSkOpenGLCanvasProvider.h

@@ -31,7 +31,7 @@ public:
 
   void surfaceDestroyed();
 
-  void surfaceSizeChanged(int width, int height);
+  void surfaceSizeChanged(jobject jSurface, int width, int height);
 
 private:
   std::unique_ptr<WindowContext> _surfaceHolder = nullptr;

package/android/cpp/rnskia-android/gl/Display.h

@@ -35,6 +35,10 @@ public:
 
   bool isValid() const { return _display != EGL_NO_DISPLAY; }
 
+  void clearContext() {
+    eglMakeCurrent(_display, EGL_NO_SURFACE, EGL_NO_SURFACE, EGL_NO_CONTEXT);
+  }
+
   EGLConfig chooseConfig() {
 
     EGLint att[] = {EGL_RENDERABLE_TYPE,

package/android/src/main/java/com/shopify/reactnative/skia/SkiaBaseView.java

@@ -64,7 +64,7 @@ public abstract class SkiaBaseView extends ReactViewGroup implements TextureView
             return;
         }
         Log.i(tag, "onSurfaceTextureSizeChanged " + width + "/" + height);
-        surfaceSizeChanged(width, height);
+        surfaceSizeChanged(surface, width, height);
     }
 
     @Override
@@ -82,18 +82,18 @@ public abstract class SkiaBaseView extends ReactViewGroup implements TextureView
         return false;
     }
 
-    //private long _prevTimestamp = 0;
+    private long _prevTimestamp = 0;
     @Override
     public void onSurfaceTextureUpdated(SurfaceTexture surface) {
-//        long timestamp = surface.getTimestamp();
-//        long frameDuration = (timestamp - _prevTimestamp)/1000000;
-//        Log.i(tag, "onSurfaceTextureUpdated "+frameDuration+"ms");
-//        _prevTimestamp = timestamp;
+        long timestamp = surface.getTimestamp();
+        long frameDuration = (timestamp - _prevTimestamp)/1000000;
+        Log.i(tag, "onSurfaceTextureUpdated "+frameDuration+"ms");
+        _prevTimestamp = timestamp;
     }
 
     protected abstract void surfaceAvailable(Object surface, int width, int height);
 
-    protected abstract void surfaceSizeChanged(int width, int height);
+    protected abstract void surfaceSizeChanged(Object surface, int width, int height);
 
     protected abstract void surfaceDestroyed();
 

package/android/src/main/java/com/shopify/reactnative/skia/SkiaDomView.java

@@ -27,7 +27,7 @@ public class SkiaDomView extends SkiaBaseView {
 
     protected native void surfaceAvailable(Object surface, int width, int height);
 
-    protected native void surfaceSizeChanged(int width, int height);
+    protected native void surfaceSizeChanged(Object surface, int width, int height);
 
     protected native void surfaceDestroyed();
 
@@ -37,8 +37,6 @@ public class SkiaDomView extends SkiaBaseView {
 
     protected native void setDebugMode(boolean show);
 
-    protected native void updateTouchPoints(double[] points);
-
     protected native void registerView(int nativeId);
 
     protected native void unregisterView();

package/android/src/main/java/com/shopify/reactnative/skia/SkiaPictureView.java

@@ -26,7 +26,7 @@ public class SkiaPictureView extends SkiaBaseView {
 
     protected native void surfaceAvailable(Object surface, int width, int height);
 
-    protected native void surfaceSizeChanged(int width, int height);
+    protected native void surfaceSizeChanged(Object surface, int width, int height);
 
     protected native void surfaceDestroyed();
 
@@ -36,8 +36,6 @@ public class SkiaPictureView extends SkiaBaseView {
 
     protected native void setDebugMode(boolean show);
 
-    protected native void updateTouchPoints(double[] points);
-
     protected native void registerView(int nativeId);
 
     protected native void unregisterView();

package/cpp/api/JsiSkImage.h

@@ -92,7 +92,11 @@ public:
     image = DawnContext::getInstance().MakeRasterImage(image);
 #else
     if (image->isTextureBacked()) {
-      image = image->makeNonTextureImage();
+      auto grContext = getContext()->getDirectContext();
+      image = image->makeRasterImage(grContext);
+      if (!image) {
+        return nullptr;
+      }
     }
 #endif
     sk_sp<SkData> data;
@@ -121,6 +125,9 @@ public:
 
   JSI_HOST_FUNCTION(encodeToBytes) {
     auto data = encodeImageData(arguments, count);
+    if (!data) {
+      return jsi::Value::null();
+    }
 
     auto arrayCtor =
         runtime.global().getPropertyAsFunction(runtime, "Uint8Array");
@@ -141,6 +148,9 @@ public:
 
   JSI_HOST_FUNCTION(encodeToBase64) {
     auto data = encodeImageData(arguments, count);
+    if (!data) {
+      return jsi::Value::null();
+    }
 
     auto len = Base64::Encode(data->bytes(), data->size(), nullptr);
     auto buffer = std::string(len, 0);
@@ -182,10 +192,15 @@ public:
             .asObject(runtime)
             .getArrayBuffer(runtime);
     auto bfrPtr = reinterpret_cast<void *>(buffer.data(runtime));
-
-    if (!getObject()->readPixels(info, bfrPtr, bytesPerRow, srcX, srcY)) {
+#if defined(SK_GRAPHITE)
+    throw std::runtime_error("Not implemented yet");
+#else
+    auto grContext = getContext()->getDirectContext();
+    if (!getObject()->readPixels(grContext, info, bfrPtr, bytesPerRow, srcX,
+                                 srcY)) {
       return jsi::Value::null();
     }
+#endif
     return dest;
   }
 
@@ -193,7 +208,8 @@ public:
 #if defined(SK_GRAPHITE)
     auto rasterImage = DawnContext::getInstance().MakeRasterImage(getObject());
 #else
-    auto rasterImage = getObject()->makeNonTextureImage();
+    auto grContext = getContext()->getDirectContext();
+    auto rasterImage = getObject()->makeRasterImage(grContext);
 #endif
     return jsi::Object::createFromHostObject(
         runtime, std::make_shared<JsiSkImage>(getContext(), rasterImage));

package/cpp/rnskia/RNSkPlatformContext.h

@@ -147,6 +147,10 @@ public:
    */
   virtual sk_sp<SkImage> makeImageFromNativeBuffer(void *buffer) = 0;
 
+#if !defined(SK_GRAPHITE)
+  virtual GrDirectContext *getDirectContext() = 0;
+#endif
+
   virtual void releaseNativeBuffer(uint64_t pointer) = 0;
 
   virtual uint64_t makeNativeBuffer(sk_sp<SkImage> image) = 0;

package/cpp/rnskia/RNSkView.h

@@ -90,7 +90,8 @@ public:
   RNSkOffscreenCanvasProvider(std::shared_ptr<RNSkPlatformContext> context,
                               std::function<void()> requestRedraw, float width,
                               float height)
-      : RNSkCanvasProvider(requestRedraw), _width(width), _height(height) {
+      : RNSkCanvasProvider(requestRedraw), _context(context), _width(width),
+        _height(height) {
     _surface = context->makeOffscreenSurface(_width, _height);
     _pd = context->getPixelDensity();
   }
@@ -113,7 +114,8 @@ public:
         _surface->recorder()->snap().get());
     return DawnContext::getInstance().MakeRasterImage(image);
 #else
-    return image->makeNonTextureImage();
+    auto grContext = _context->getDirectContext();
+    return image->makeRasterImage(grContext);
 #endif
   }
 
@@ -140,6 +142,7 @@ private:
   float _height;
   float _pd = 1.0f;
   sk_sp<SkSurface> _surface;
+  std::shared_ptr<RNSkPlatformContext> _context;
 };
 
 enum RNSkDrawingMode { Default, Continuous };

package/ios/RNSkia-iOS/MetalContext.h

@@ -37,6 +37,8 @@ public:
                                                 height);
   }
 
+  GrDirectContext *getDirectContext() { return _context.skContext.get(); }
+
 private:
   friend class RNSkia::RNSkiOSPlatformContext;
   id<MTLDevice> _device;

package/ios/RNSkia-iOS/RNSkiOSPlatformContext.h

@@ -76,6 +76,9 @@ public:
 
   void raiseError(const std::exception &err) override;
   sk_sp<SkSurface> makeOffscreenSurface(int width, int height) override;
+#if !defined(SK_GRAPHITE)
+  GrDirectContext *getDirectContext() override;
+#endif
   sk_sp<SkFontMgr> createFontMgr() override;
 
   void willInvalidateModules() {

package/ios/RNSkia-iOS/RNSkiOSPlatformContext.mm

@@ -192,6 +192,12 @@ sk_sp<SkImage> RNSkiOSPlatformContext::makeImageFromNativeBuffer(void *buffer) {
 #endif
 }
 
+#if !defined(SK_GRAPHITE)
+GrDirectContext *RNSkiOSPlatformContext::getDirectContext() {
+  return MetalContext::getInstance().getDirectContext();
+}
+#endif
+
 sk_sp<SkFontMgr> RNSkiOSPlatformContext::createFontMgr() {
   return SkFontMgr_New_CoreText(nullptr);
 }

package/ios/RNSkia-iOS/SkiaMetalSurfaceFactory.h

@@ -107,8 +107,6 @@ public:
     _skSurface = nullptr;
   }
 
-  void resize(int width, int height) override { _skSurface = nullptr; }
-
   int getWidth() override {
     return _layer.frame.size.width * _layer.contentsScale;
   };
@@ -117,6 +115,8 @@ public:
     return _layer.frame.size.height * _layer.contentsScale;
   };
 
+  void resize(int width, int height) override { _skSurface = nullptr; }
+
 private:
   SkiaMetalContext *_context;
   sk_sp<SkSurface> _skSurface = nullptr;

package/package.json

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

package/react-native-skia.podspec

@@ -9,8 +9,8 @@ use_graphite = ENV['SK_GRAPHITE'] == '1'
 
 # Set preprocessor definitions based on GRAPHITE flag
 preprocessor_defs = use_graphite ? 
-  '$(inherited) SK_GRAPHITE=1' : 
-  '$(inherited) SK_METAL=1 SK_GANESH=1'
+  '$(inherited) SK_GRAPHITE=1 SK_IMAGE_READ_PIXELS_DISABLE_LEGACY_API=1' : 
+  '$(inherited) SK_METAL=1 SK_GANESH=1 SK_IMAGE_READ_PIXELS_DISABLE_LEGACY_API=1'
 
 # Define base frameworks
 base_frameworks = ['libs/ios/libskia.xcframework', 

package/src/renderer/__tests__/e2e/Image.spec.tsx

@@ -43,51 +43,17 @@ describe("Image loading from bundles", () => {
       },
       {
         data: Array.from(
-          loadImage("skia/__tests__/assets/oslo.jpg").encodeToBytes()
+          loadImage("skia/__tests__/assets/oslo-mini.jpg").encodeToBytes()
         ),
       }
     );
     expect(pixels).toBeDefined();
     expect(pixels).toEqual([
-      170, 186, 199, 255, 170, 186, 199, 255, 170, 186, 199, 255, 170, 186, 199,
+      171, 188, 198, 255, 171, 188, 198, 255, 171, 188, 198, 255, 171, 188, 198,
       255,
     ]);
   });
 
-  // it("should read pixels from an image using a preallocated buffer", async () => {
-  //   const pixels = await surface.eval(
-  //     (Skia, { colorType, alphaType, data }) => {
-  //       const image = Skia.Image.MakeImageFromEncoded(
-  //         Skia.Data.fromBytes(new Uint8Array(data))
-  //       )!;
-  //       const result = new Uint8Array(16);
-  //       image.readPixels(
-  //         0,
-  //         0,
-  //         {
-  //           width: 2,
-  //           height: 2,
-  //           colorType,
-  //           alphaType,
-  //         },
-  //         result
-  //       );
-  //       return result;
-  //     },
-  //     {
-  //       colorType: ColorType.RGBA_8888,
-  //       alphaType: AlphaType.Unpremul,
-  //       data: Array.from(
-  //         loadImage("skia/__tests__/assets/oslo.jpg").encodeToBytes()
-  //       ),
-  //     }
-  //   );
-  //   expect(pixels).toBeDefined();
-  //   expect(Array.from(pixels!)).toEqual([
-  //     170, 186, 199, 255, 170, 186, 199, 255, 170, 186, 199, 255, 170, 186, 199,
-  //     255,
-  //   ]);
-  // });
   it("should read pixels from a canvas", async () => {
     const pixels = await surface.eval(
       (Skia, { colorType, alphaType }) => {
@@ -108,32 +74,4 @@ describe("Image loading from bundles", () => {
     expect(pixels).toBeDefined();
     expect(Array.from(pixels!)).toEqual([255, 0, 0, 255]);
   });
-  // it("should read pixels from a canvas using a preallocated buffer", async () => {
-  //   const pixels = await surface.eval(
-  //     (Skia, { colorType, alphaType }) => {
-  //       const offscreen = Skia.Surface.MakeOffscreen(10, 10)!;
-  //       const canvas = offscreen.getCanvas();
-  //       canvas.drawColor(Skia.Color("red"));
-  // const result = new Uint8Array(4);
-  //        canvas.readPixels(0, 0, {
-  //         width: 1,
-  //         height: 1,
-  //         colorType,
-  //         alphaType,
-  //       }, result);
-  //     },
-  //     { colorType: ColorType.RGBA_8888, alphaType: AlphaType.Unpremul }
-  //   );
-  //   expect(pixels).toBeDefined();
-  //   expect(Array.from(pixels!)).toEqual([255, 0, 0, 255]);
-  // });
-  // This test should only run on CI because it will trigger a redbox.
-  // While this is fine on CI, it is undesirable on local dev.
-  // it("should not crash with an invalid viewTag", async () => {
-  //   const result = await surface.eval((Skia) => {
-  //     Skia.Image.MakeImageFromViewTag(-1);
-  //     return true;
-  //   });
-  //   expect(result).toBe(true);
-  // });
 });

package/android/src/paper/java/com/facebook/react/viewmanagers/SkiaDrawViewManagerInterface.java

@@ -1,18 +0,0 @@
-/**
-* This code was generated by [react-native-codegen](https://www.npmjs.com/package/react-native-codegen).
-*
-* Do not edit this file as changes may cause incorrect behavior and will be lost
-* once the code is regenerated.
-*
-* @generated by codegen project: GeneratePropsJavaInterface.js
-*/
-
-package com.facebook.react.viewmanagers;
-
-import android.view.View;
-import androidx.annotation.Nullable;
-
-public interface SkiaDrawViewManagerInterface<T extends View> {
-  void setMode(T view, @Nullable String value);
-  void setDebug(T view, boolean value);
-}