M118 public #204
Merged
M118 public #204
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…current m118 (canvaskit/0.38.2-1452-gbd56a010b6)
commit 3e85749702f44e21fdca2285b8c3c6a692e50516 Author: Robert Phillips <[email protected]> Date: Tue Aug 8 15:08:28 2023 -0400 Propagate protected-ness to dependant draws For example, when blurring a protected image we need to create protected surfaces and SFCs to accommodate the protectedness. Additionally, having an API-level entry point to create a protected SkSurface seems like it would be useful to any client trying to work with protected images or trying to create protected content. Change-Id: I191f50ec1e04175c3e243de177ba21e0970f2a58 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/738160 Reviewed-by: Brian Osman <[email protected]> Reviewed-by: Jim Van Verth <[email protected]> Commit-Queue: Robert Phillips <[email protected]>
commit d9eb1f0a0c53b6ea4f67a97f58fb8c54367bfbda Author: Brian Osman <[email protected]> Date: Thu Aug 17 19:54:20 2023 +0000 Remove GrBackendSurfaceMutableState Clients are now using skgpu::MutableTextureState directly Bug: b/296240795 Change-Id: Ia310e876d49835ba8046f69372a8db0be30d14ff Reviewed-on: https://skia-review.googlesource.com/c/skia/+/744256 Reviewed-by: Greg Daniel <[email protected]> Commit-Queue: Brian Osman <[email protected]>
commit ac39b12fd8357fbe88cf8ff29d7ade1773b2cb2b Author: John Stiles <[email protected]> Date: Fri Aug 25 16:23:37 2023 -0400 Remove GrBackendApi::kDawn. Dawn is only supported in Graphite, so we don't need a Ganesh backend API enum for it. Our backend-API enums now contain a `kUnsupported` enumerant which is returned when calling `ContextTypeBackend` with an unsupported context type. Bug: b/297077133 Change-Id: If42280750988f4a1d1f1485eb1fcf065fdb126eb Reviewed-on: https://skia-review.googlesource.com/c/skia/+/747657 Reviewed-by: Michael Ludwig <[email protected]> Commit-Queue: John Stiles <[email protected]> Reviewed-by: Brian Osman <[email protected]>
commit 2cf149beae3fdb70f689ab3b64b138f1fdd5bd0c Author: Kevin Lubick <[email protected]> Date: Mon Aug 14 10:36:08 2023 -0400 Remove bridge code for legacy GL GrBackendSurface code While doing this, I ran into a small #include issue with GrGLGpu, so I enforced IWYU on it. Client CLs: - https://crrev.com/c/4739911 - http://cl/553620143 - http://cl/553628593 - http://cl/553637942 - http://cl/553705311 - http://cl/553821763 - http://cl/553837992 - http://ag/24343000 - http://ag/24343454 - http://ag/24420576 - http://ag/24445187 - flutter/engine#44334 - flutter/engine#44682 Change-Id: If07cbe3a60593810911c147b8a81e30459623f28 Bug: b/293490566 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/741437 Reviewed-by: Greg Daniel <[email protected]> Reviewed-by: Brian Osman <[email protected]> Commit-Queue: Kevin Lubick <[email protected]>
Without obvious replacements, unlike the others.
Concerning "backend_texture" and "backend_surface_mutable_state" init.
Merged
|
@kyamagu I just remember one important reason why I didn't want to merge m118 - skia-python/docs/tutorial/canvas.rst Line 155 in dcba39e |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is just posted for RFC . M119 is already out, and there are two removed constructors which means about a dozen of new m118:REVISIT skips were inserted to get CI to pass.
See README.m118 for details.
Since this is a lost of existing functionality compared to v117b3, I'd say let's see if m119 can be better.