Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SkiaSharp updates, SkiaSharp 3.0 render tests enabled #15255

Merged
merged 8 commits into from
Apr 20, 2024

Conversation

maxkatz6
Copy link
Member

@maxkatz6 maxkatz6 commented Apr 6, 2024

What does the pull request do?

  1. Updates SkiaSharp to 2.88.8-preview
  2. Removes skia compat methods, as these are compatible now
  3. Adds SkiaSharp 3.0 render tests msbuild property.

@maxkatz6 maxkatz6 added area-skia backport-candidate-11.1.x Consider this PR for backporting to 11.1 branch labels Apr 6, 2024
@maxkatz6 maxkatz6 requested review from kekekeks and Gillibald April 6, 2024 02:07
@maxkatz6
Copy link
Member Author

maxkatz6 commented Apr 6, 2024

For some reason some tests are failing with System.InvalidOperationException : Could not create glyphTypeface, which is weird. Locally I don't have this issue.

@kekekeks
Copy link
Member

kekekeks commented Apr 8, 2024

What would happen if Skia and Skia3 render tests are running in parallel (e. g. from an IDE runner)?

@maxkatz6 maxkatz6 changed the title [WIP] SkiaSharp updates, SkiaSharp 3.0 render tests enabled SkiaSharp updates, SkiaSharp 3.0 render tests enabled Apr 20, 2024
@maxkatz6
Copy link
Member Author

@kekekeks removed Skia3 unit test project. But kept a msbuild property, so one could run tests with it manually.

@maxkatz6 maxkatz6 merged commit 8c7754f into master Apr 20, 2024
2 of 10 checks passed
@maxkatz6 maxkatz6 deleted the skia-sharp-update branch April 20, 2024 01:01
maxkatz6 added a commit that referenced this pull request Apr 20, 2024
* Use 2.88.8-preview.1.1 skiasharp

* Use SKImageFilter directly, as we don't need compat anymore

* Add SkiaSharp 3 render tests

* Enable SkiaSharp 3 tests on CI

* Add IncludeLinuxSkia in Skia tests

* Update Skia version, remove SkiaCompat

* Remove Skia3 test project, reuse single Skia render test project, so they won't conflict
@maxkatz6 maxkatz6 added backported-11.1.x and removed backport-candidate-11.1.x Consider this PR for backporting to 11.1 branch labels Apr 20, 2024
@maxkatz6 maxkatz6 mentioned this pull request Apr 25, 2024
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants