fix: miss applying image_smoothing_enabled and image_smoothing_quality

Brooooooklyn · Aug 4, 2022 · 815bc91 · 815bc91
1 parent 110606b
commit 815bc91
Show file tree

Hide file tree

Showing 42 changed files with 63 additions and 6 deletions.
diff --git a/__test__/image-snapshot.ts b/__test__/image-snapshot.ts
@@ -32,7 +32,7 @@ export async function snapshotImage<C>(
   } catch {
     existed = false
   }
-  if (!existed) {
+  if (!existed || process.env.UPDATE_SNAPSHOT) {
     await fs.writeFile(p, image)
     t.pass()
   } else {

diff --git a/__test__/snapshots/arc.png b/__test__/snapshots/arc.png
diff --git a/__test__/snapshots/arcTo-colorful.png b/__test__/snapshots/arcTo-colorful.png
diff --git a/__test__/snapshots/arcTo.png b/__test__/snapshots/arcTo.png
diff --git a/__test__/snapshots/avif-output.avif b/__test__/snapshots/avif-output.avif
diff --git a/__test__/snapshots/bezierCurveTo-colorful.png b/__test__/snapshots/bezierCurveTo-colorful.png
diff --git a/__test__/snapshots/bezierCurveTo.png b/__test__/snapshots/bezierCurveTo.png
diff --git a/__test__/snapshots/clearRect.png b/__test__/snapshots/clearRect.png
diff --git a/__test__/snapshots/createRadialGradient.png b/__test__/snapshots/createRadialGradient.png
diff --git a/__test__/snapshots/draw-image-svg-noto-emoji.png b/__test__/snapshots/draw-image-svg-noto-emoji.png
diff --git a/__test__/snapshots/drawImage-another-Canvas.png b/__test__/snapshots/drawImage-another-Canvas.png
diff --git a/__test__/snapshots/drawImage-svg-resize.png b/__test__/snapshots/drawImage-svg-resize.png
diff --git a/__test__/snapshots/drawImage-svg.png b/__test__/snapshots/drawImage-svg.png
diff --git a/__test__/snapshots/drawImage.png b/__test__/snapshots/drawImage.png
diff --git a/__test__/snapshots/ellipse.png b/__test__/snapshots/ellipse.png
diff --git a/__test__/snapshots/fillText-COLRv1.png b/__test__/snapshots/fillText-COLRv1.png
diff --git a/__test__/snapshots/fillText.png b/__test__/snapshots/fillText.png
diff --git a/__test__/snapshots/filter-blur.png b/__test__/snapshots/filter-blur.png
diff --git a/__test__/snapshots/filter-brightness.png b/__test__/snapshots/filter-brightness.png
diff --git a/__test__/snapshots/filter-combine-contrast-brightness.png b/__test__/snapshots/filter-combine-contrast-brightness.png
diff --git a/__test__/snapshots/filter-contrast-ff.png b/__test__/snapshots/filter-contrast-ff.png
diff --git a/__test__/snapshots/filter-contrast.png b/__test__/snapshots/filter-contrast.png
diff --git a/__test__/snapshots/filter-drop-shadow.png b/__test__/snapshots/filter-drop-shadow.png
diff --git a/__test__/snapshots/filter-grayscale.png b/__test__/snapshots/filter-grayscale.png
diff --git a/__test__/snapshots/filter-hue-rotate.png b/__test__/snapshots/filter-hue-rotate.png
diff --git a/__test__/snapshots/filter-invert.png b/__test__/snapshots/filter-invert.png
diff --git a/__test__/snapshots/filter-opacity.png b/__test__/snapshots/filter-opacity.png
diff --git a/__test__/snapshots/filter-saturate.png b/__test__/snapshots/filter-saturate.png
diff --git a/__test__/snapshots/filter-save-restore.png b/__test__/snapshots/filter-save-restore.png
diff --git a/__test__/snapshots/filter-sepia.png b/__test__/snapshots/filter-sepia.png
diff --git a/__test__/snapshots/quadraticCurveTo.png b/__test__/snapshots/quadraticCurveTo.png
diff --git a/__test__/snapshots/should draw img.jpg b/__test__/snapshots/should draw img.jpg
diff --git a/__test__/snapshots/stroke-and-filling-jpeg.jpg b/__test__/snapshots/stroke-and-filling-jpeg.jpg
diff --git a/__test__/snapshots/strokeText.png b/__test__/snapshots/strokeText.png
diff --git a/__test__/snapshots/svg-transparent-background.png b/__test__/snapshots/svg-transparent-background.png
diff --git a/__test__/snapshots/transform-with-radial-gradient-x.png b/__test__/snapshots/transform-with-radial-gradient-x.png
diff --git a/__test__/snapshots/transform-with-radial-gradient.png b/__test__/snapshots/transform-with-radial-gradient.png
diff --git a/__test__/snapshots/webp-output.webp b/__test__/snapshots/webp-output.webp
diff --git a/skia-c/skia_c.cpp b/skia-c/skia_c.cpp
@@ -277,12 +277,26 @@ extern "C"
     CANVAS_CAST->drawColor(SkColor4f{r, g, b, a});
   }
 
-  void skiac_canvas_draw_image(skiac_canvas *c_canvas, skiac_bitmap *c_bitmap, float sx, float sy, float s_width, float s_height, float dx, float dy, float d_width, float d_height, skiac_paint *c_paint)
+  void skiac_canvas_draw_image(
+      skiac_canvas *c_canvas,
+      skiac_bitmap *c_bitmap,
+      float sx,
+      float sy,
+      float s_width,
+      float s_height,
+      float dx,
+      float dy,
+      float d_width,
+      float d_height,
+      bool enable_smoothing,
+      int filter_quality,
+      skiac_paint *c_paint)
   {
     const auto src_rect = SkRect::MakeXYWH(sx, sy, s_width, s_height);
     const auto dst_rect = SkRect::MakeXYWH(dx, dy, d_width, d_height);
     auto sk_image = SkImage::MakeFromBitmap(*BITMAP_CAST);
-    const auto sampling = SkSamplingOptions(SkCubicResampler::Mitchell());
+    auto fq = enable_smoothing ? filter_quality : 0;
+    const auto sampling = SamplingOptionsFromFQ(fq);
     auto paint = reinterpret_cast<const SkPaint *>(c_paint);
     CANVAS_CAST->drawImageRect(sk_image, src_rect, dst_rect, sampling, paint, SkCanvas::kFast_SrcRectConstraint);
   }

diff --git a/skia-c/skia_c.hpp b/skia-c/skia_c.hpp
@@ -245,7 +245,20 @@ extern "C"
   skiac_transform skiac_canvas_get_total_transform(skiac_canvas *c_canvas);
   skiac_matrix *skiac_canvas_get_total_transform_matrix(skiac_canvas *c_canvas);
   void skiac_canvas_draw_color(skiac_canvas *c_canvas, float r, float g, float b, float a);
-  void skiac_canvas_draw_image(skiac_canvas *c_canvas, skiac_bitmap *c_bitmap, float sx, float sy, float s_width, float s_height, float dx, float dy, float d_width, float d_height, skiac_paint *c_paint);
+  void skiac_canvas_draw_image(
+      skiac_canvas *c_canvas,
+      skiac_bitmap *c_bitmap,
+      float sx,
+      float sy,
+      float s_width,
+      float s_height,
+      float dx,
+      float dy,
+      float d_width,
+      float d_height,
+      bool enable_smoothing,
+      int filter_quality,
+      skiac_paint *c_paint);
   void skiac_canvas_draw_path(skiac_canvas *c_canvas, skiac_path *c_path, skiac_paint *c_paint);
   void skiac_canvas_draw_rect(
       skiac_canvas *c_canvas,

diff --git a/src/ctx.rs b/src/ctx.rs
@@ -711,11 +711,24 @@ impl Context {
         dy,
         d_width,
         d_height,
+        self.state.image_smoothing_enabled,
+        self.state.image_smoothing_quality,
         &drop_shadow_paint,
       );
     }
     self.surface.canvas.draw_image(
-      bitmap, sx, sy, s_width, s_height, dx, dy, d_width, d_height, &paint,
+      bitmap,
+      sx,
+      sy,
+      s_width,
+      s_height,
+      dx,
+      dy,
+      d_width,
+      d_height,
+      self.state.image_smoothing_enabled,
+      self.state.image_smoothing_quality,
+      &paint,
     );
 
     Ok(())

diff --git a/src/sk.rs b/src/sk.rs
@@ -347,6 +347,8 @@ mod ffi {
       dy: f32,
       d_width: f32,
       d_height: f32,
+      enable_smoothing: bool,
+      filter_quality: i32,
       paint: *mut skiac_paint,
     );
 
@@ -1251,6 +1253,7 @@ impl From<i32> for BlendMode {
   }
 }
 
+#[repr(i32)]
 #[derive(Copy, Clone, PartialEq, Eq, Hash, Debug)]
 pub enum FilterQuality {
   None = 0,
@@ -1993,11 +1996,25 @@ impl Canvas {
     dy: f32,
     d_width: f32,
     d_height: f32,
+    enable_smoothing: bool,
+    quality: FilterQuality,
     paint: &Paint,
   ) {
     unsafe {
       ffi::skiac_canvas_draw_image(
-        self.0, image, sx, sy, s_width, s_height, dx, dy, d_width, d_height, paint.0,
+        self.0,
+        image,
+        sx,
+        sy,
+        s_width,
+        s_height,
+        dx,
+        dy,
+        d_width,
+        d_height,
+        enable_smoothing,
+        quality as i32,
+        paint.0,
       );
     }
   }