[release/10.0.1xx-sr10] Update SkiaSharp to 4.150.1

[kubaflo]

Note

Are you waiting for the changes in this PR to be merged?
It would be very helpful if you could test the resulting artifacts from this PR and let us know in a comment if this change resolves your issue. Thank you!

Summary

Manual Skia payload backport for .NET MAUI 10.0.101, targeting release/10.0.1xx-sr10.

• Backports Update SkiaSharp to 4.150.1 and HarfBuzzSharp to 14.2.1.1 #36255 from immutable inflight/candidate commit 08a2078a086a496195f5b9b39fa0888567d4e183.
• Updates SkiaSharp 3.116.1 to 4.150.1, HarfBuzzSharp 8.3.0.1 to 14.2.1.1, Svg.Skia 2.0.0.4 to 5.1.1, and the required supporting dependencies and source ports.
• Includes only the SkiaSharpAppIconTools.cs SKPath.AddCircle compatibility hunk from Fix inflight/current build breaks (duplicate MAUIX2017, obsolete AddCircle, Assert.IsTrue) #36787 / 81c9eca0f98a1b0c0f3b147d2ac120fb2444e2df.
• Migrates the affected Graphics callers to the SkiaSharp 4 SKPathBuilder, SKTextAlign, and SKSamplingOptions APIs so normal warnings-as-errors builds remain clean.
• Excludes Fix inflight/current build breaks (duplicate MAUIX2017, obsolete AddCircle, Assert.IsTrue) #36787's unrelated SourceGen, integration-test, and other build-break fixes.
• Keeps the version bump from 10.0.100 to 10.0.101 out of this PR.

Validation

• The target-relative diff is 17 files, 73 insertions, and 84 deletions.
• Microsoft.Maui.BuildTasks.slnf builds successfully with zero errors.
• Graphics.Skia.csproj builds all target frameworks with warnings as errors and zero warnings/errors.
• All 372 Graphics.Tests pass under the normal warnings-as-errors configuration.
• All 339 focused ResizetizeImagesTests pass.

Release risk

_SkiaSharpNativeAssetsVersion remains unchanged. Before shipping, the release owner must confirm a matching SkiaSharp 4.150.1 native-assets/PDB build or record an explicit release waiver.

Review gate

This agent-authored release PR must remain unmerged until two distinct non-bot MAUI maintainers with write access, other than the PR author, approve the current head SHA.

[github-actions]

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/dotnet/maui/main/eng/scripts/get-maui-pr.sh | bash -s -- 37731

Or

• Run remotely in PowerShell:

iex "& { $(irm https://raw.githubusercontent.com/dotnet/maui/main/eng/scripts/get-maui-pr.ps1) } 37731"

[azure-pipelines]

Azure Pipelines:
Successfully started running 1 pipeline(s).
There may be pipelines that require an authorized user to comment /azp run to run.

[kubaflo]

/azp run

[azure-pipelines]

Azure Pipelines:
Successfully started running 3 pipeline(s).

[Copilot]

Pull request overview

Backports SkiaSharp 4.150.1 and related dependency updates to the MAUI SR10 branch, with graphics and Resizetizer compatibility changes.

Changes:

• Updates SkiaSharp, HarfBuzzSharp, Svg.Skia, and supporting packages.
• Ports graphics and Resizetizer code to newer SkiaSharp APIs.
• Refreshes SVG tests, fixtures, and rendering diagnostics.

Reviewed changes

Copilot reviewed 12 out of 15 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Reviewed changes
src/SingleProject/Resizetizer/test/UnitTests/ResizetizeImagesTests.cs	Updates valid and invalid SVG test coverage.
src/SingleProject/Resizetizer/test/UnitTests/images/invalid.svg	Adds malformed SVG test coverage.
src/SingleProject/Resizetizer/src/TizenSplashUpdater.cs	Removes obsolete filter-quality configuration.
src/SingleProject/Resizetizer/src/SkiaSharpTools.cs	Removes obsolete filter-quality configuration.
src/SingleProject/Resizetizer/src/SkiaSharpAppIconTools.cs	Applies the SKPath.AddCircle compatibility update.
src/SingleProject/Resizetizer/src/ResizetizerPackages.projitems	Packages updated SVG assemblies.
src/Graphics/tests/Graphics.Tests/ImageAssert.cs	Corrects threshold diagnostics; the scenario tolerance remains 0.07 despite the stated 0.2 change.
src/Graphics/src/Graphics.Skia/SKPaintExtensions.cs	Removes obsolete paint members.
src/Graphics/src/Graphics.Skia/SkiaTextLayout.cs	Ports text layout away from obsolete APIs.
src/Graphics/src/Graphics.Skia/SkiaCanvasStateService.cs	Updates default font setup.
src/Graphics/src/Graphics.Skia/SkiaCanvasState.cs	Uses SKFont for text state.
eng/Versions.props	Updates SkiaSharp versions; matching 4.150.1 native symbols or a release waiver is still required.
eng/NuGetVersions.targets	Updates SVG dependency versions.

[kubaflo]

Agreed this must remain a release gate. The current PR body already calls it out under Release risk and explicitly requires either a matching SkiaSharp 4.150.1 native-assets/PDB build or a release-owner waiver; the human-approval review gate also prevents treating this as shippable meanwhile. There is no verified replacement build or authorized waiver for the monitor to commit, so I am leaving this thread open for the release owner.

[PureWeen]

Adversarial review

An independent reviewer found a release-blocking regression.

❌ Build Regression — Updating _SkiaSharpVersion to 4.150.1 makes the ordinary Graphics.Skia and Graphics.Tests builds fail with 27 CS0618 errors. SkiaSharp 4.150.1 marks existing SKPath mutation methods, SKCanvas.DrawText(string, float, float, SKFont, SKPaint), and SKCanvas.DrawBitmap(..., SKPaint) obsolete, while this payload leaves their callers in SKGraphicsExtensions.cs and SkiaCanvas.cs unchanged. Directory.Build.props treats warnings as errors, and the Graphics unit-test pipeline does not override that setting.

Migrate those usages to the new SKPathBuilder, SKTextAlign, and SKSamplingOptions APIs, or add narrowly scoped servicing suppressions before this version update. This is anchored on the changed version property because the failing consumer files are outside the PR diff.

Flagged by: 3/3 reviewers after adversarial consensus.

Automated assessment complete: verified current head 9f573a85fb0272640806474ab329e50cb2e5797f; coordination sender session 20939cfd-13dd-40eb-83ee-cdb524c95e95.

[kubaflo]

@PureWeen Addressed in 8b42f957c0. Migrated the 27 affected calls to the SkiaSharp 4 APIs (SKPathBuilder, explicit SKTextAlign.Left, and explicit nearest/no-mipmap SKSamplingOptions) rather than suppressing obsoletion warnings. Graphics.Skia.csproj now builds all target frameworks with zero warnings/errors under the normal warnings-as-errors configuration, and all 372 Graphics.Tests pass without overriding TreatWarningsAsErrors. The PR description and validation section are updated. Ready for re-review.

[Copilot]

Pull request overview

Copilot reviewed 14 out of 17 changed files in this pull request and generated 1 comment.

[PureWeen]

Adversarial review

An independent reviewer found no new in-scope issues at commit 8b42f957c02e1ff8b890e61f74cefbb809397e25.

Three independent reviewers with adversarial consensus, plus the MAUI repository specialist, reviewed the SkiaSharp 4 migration, graphics behavior, package/config consumers, and Resizetizer updates. The specialist cross-platform snapshot concern and two independent concerns about sampling/path/text migration were rejected in a two-reviewer dispute pass because the current source did not establish a concrete regression.

Changed Graphics and Resizetizer test assets cover the updated SVG error path and Mac rendering baselines. Existing current-head feedback was checked and not duplicated.

[kubaflo]

@copilot addressed the Resizetizer signing-list feedback in 3d05f04 by adding Svg.SceneGraph.dll and Svg.Animation.dll to 3PartySHA2. This is ready for re-review; the separate native-assets/PDB release gate remains open.

[Copilot]

Pull request overview

Copilot reviewed 15 out of 18 changed files in this pull request and generated no new comments.

Suppressed comments (3)

eng/Versions.props:125

• The validation summary is stale for the current head: GitHub reports 18 changed files, 75 additions, and 84 deletions, rather than the stated 17/73/84 (the extra changes include the two modified Mac baseline PNGs). Please update the PR description so the release scope is accurate.

    <_SkiaSharpVersion>4.150.1</_SkiaSharpVersion>

src/Graphics/tests/Graphics.Tests/ImageAssert.cs:61

• The current PR description again says the per-scenario snapshot tolerance was raised from 0.07 to 0.2, but SkiaSharpScenarioTests.cs:43 still passes 0.07; this changed ImageAssert line only corrects the diagnostic text. Please either update the scenario call/tolerance or correct the description so it matches the behavior actually implemented.

		Assert.Fail($"Image was not equal. Error was {similarity.ErrorPixelPercentage}% ({similarity.AbsoluteError} pixels), which was higher than the threshold of {threshold}%. See {diffFilename}");

src/SingleProject/Resizetizer/test/UnitTests/ResizetizeImagesTests.cs:69

• Could this add a fixture containing an actual SVG <text> element and assert successful rasterization/content, rather than only adding link_out.svg? The package bump is needed because the older Svg.Skia path can call the SKPaint text members removed by SkiaSharp 4.x, but none of the current Resizetizer SVG fixtures contain <text>, so these tests would still pass while that MissingMethodException regression remains.

			[InlineData("link_out.svg")]

[PureWeen]

Adversarial review

An independent reviewer found no new high-confidence, in-scope issues at commit 3d05f047d86bcf3122a27255de70d1fbf52e8410.

Three independent reviewers with adversarial consensus, plus the MAUI repository specialist, reviewed the SkiaSharp 4 migration, Graphics rendering/text/path semantics, Resizetizer SVG handling, package payload/signing consumers, and the changed test assets. The path-builder ownership, SKFont migration, sampling behavior, package/signing propagation, and valid/malformed SVG coverage were verified. Existing current-head feedback was checked and not duplicated.