Update SkiaSharp from 2.88.8 to 3.119.2 (#7582)

* Initial plan

* Update SkiaSharp from 2.88.8 to 3.119.2 and fix breaking API changes

Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>

* Use high-quality SKCubicResampler.Mitchell for resize operations

Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>

* Address review feedback: fix csproj comment and add pixel pointer validation

Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>

Author: Copilot

eng/Versions.props

@@ -19,7 +19,7 @@
     <MicrosoftExtensionsObjectPoolVersion>9.0.4</MicrosoftExtensionsObjectPoolVersion>
     <MicrosoftExtensionsOptionsVersion>9.0.4</MicrosoftExtensionsOptionsVersion>
     <NuGetVersion>6.9.1</NuGetVersion>
-    <SkiaSharpVersion>2.88.8</SkiaSharpVersion>
+    <SkiaSharpVersion>3.119.2</SkiaSharpVersion>
     <SystemCodeDomVersion>9.0.4</SystemCodeDomVersion>
     <SystemCollectionsImmutableVersion>9.0.4</SystemCollectionsImmutableVersion>
     <SystemConfigurationConfigurationManagerVersion>9.0.4</SystemConfigurationConfigurationManagerVersion>

src/Microsoft.ML.ImageAnalytics/MLImage.cs

@@ -155,14 +155,18 @@ public byte[] GetBGRPixels
         /// <summary>
         /// Gets the image pixel data.
         /// </summary>
-        public ReadOnlySpan<byte> Pixels
+        public unsafe ReadOnlySpan<byte> Pixels
         {
             get
             {
                 ThrowInvalidOperationExceptionIfDisposed();
                 Debug.Assert(_image.Info.BytesPerPixel == 4);
 
-                return _image.GetPixelSpan();
+                var pixelsPtr = _image.GetPixels();
+                if (pixelsPtr == IntPtr.Zero || _image.ByteCount <= 0)
+                    throw new InvalidOperationException("Pixel data is unavailable.");
+
+                return new ReadOnlySpan<byte>(pixelsPtr.ToPointer(), _image.ByteCount);
             }
         }
 
@@ -305,7 +309,7 @@ internal MLImage CloneWithResizing(int width, int height, ImageResizeMode mode)
             return new MLImage(image);
         }
 
-        private SKBitmap ResizeFull(int width, int height) => _image.Resize(new SKSizeI(width, height), SKFilterQuality.None);
+        private SKBitmap ResizeFull(int width, int height) => _image.Resize(new SKSizeI(width, height), new SKSamplingOptions(SKFilterMode.Nearest));
 
         private SKBitmap ResizeWithPadding(int width, int height)
         {
@@ -334,9 +338,9 @@ private SKBitmap ResizeWithPadding(int width, int height)
             SKRect destRect = new SKRect(destX, destY, destX + destWidth, destY + destHeight);
 
             using SKCanvas canvas = new SKCanvas(destBitmap);
-            using SKPaint paint = new SKPaint() { FilterQuality = SKFilterQuality.High };
+            using SKImage image = SKImage.FromBitmap(_image);
 
-            canvas.DrawBitmap(_image, srcRect, destRect, paint);
+            canvas.DrawImage(image, srcRect, destRect, new SKSamplingOptions(SKCubicResampler.Mitchell));
 
             return destBitmap;
         }
@@ -391,9 +395,9 @@ private SKBitmap ResizeWithCrop(int width, int height, ImageResizeMode mode)
             SKRect destRect = new SKRect(destX, destY, destX + destWidth, destY + destHeight);
 
             using SKCanvas canvas = new SKCanvas(dst);
-            using SKPaint paint = new SKPaint() { FilterQuality = SKFilterQuality.High };
+            using SKImage image = SKImage.FromBitmap(_image);
 
-            canvas.DrawBitmap(_image, srcRect, destRect, paint);
+            canvas.DrawImage(image, srcRect, destRect, new SKSamplingOptions(SKCubicResampler.Mitchell));
 
             return dst;
         }
@@ -415,7 +419,6 @@ internal MLImage CloneWithGrayscale()
             using SKPaint paint = new SKPaint()
             {
                 ColorFilter = _grayscaleColorMatrix,
-                FilterQuality = SKFilterQuality.High
             };
 
             SKBitmap destBitmap = new SKBitmap(_image.Width, _image.Height, isOpaque: true);

src/Microsoft.ML.ImageAnalytics/Microsoft.ML.ImageAnalytics.csproj

@@ -11,7 +11,9 @@
     <PackageReference Include="System.Buffers" />
     <PackageReference Include="System.Runtime.CompilerServices.Unsafe" />
 
-    <!-- SkiaSharp reference Windows, UWP, and MacOS native dependencies packages. No need to explicity reference it here. -->
+    <PackageReference Include="SkiaSharp" />
+
+    <!-- SkiaSharp already includes Windows, UWP, and macOS native dependency packages; only the Linux native assets package needs to be referenced explicitly here. -->
     <PackageReference Include="SkiaSharp.NativeAssets.Linux.NoDependencies" />
   </ItemGroup>