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

M130 public #265

Merged
merged 10 commits into from
Sep 20, 2024
Merged

M130 public #265

merged 10 commits into from
Sep 20, 2024

Conversation

HinTak
Copy link
Collaborator

@HinTak HinTak commented Sep 17, 2024

This is inclusive of #260 . The m130 change is small - SkColorFilter::filterColor was removed and is now emulated.

…u/vk/GrVkTypes.h

See the diff:

$ diff -ruBb skia-m128/include/gpu/ganesh/vk/GrVkBackendSurface.h skia-m129/include/gpu/ganesh/vk/GrVkBackendSurface.h
--- skia-m128/include/gpu/ganesh/vk/GrVkBackendSurface.h	2024-08-10 19:16:09.000000000 +0100
+++ skia-m129/include/gpu/ganesh/vk/GrVkBackendSurface.h	2024-08-21 16:38:24.000000000 +0100
@@ -8,15 +8,19 @@
 #ifndef GrVkBackendSurface_DEFINED
 #define GrVkBackendSurface_DEFINED

-#include "include/gpu/vk/GrVkTypes.h"
-#include "include/gpu/vk/VulkanTypes.h"
 #include "include/private/base/SkAPI.h"
+#include "include/private/gpu/vk/SkiaVulkan.h"

 #include <string_view>

 class GrBackendFormat;
 class GrBackendTexture;
 class GrBackendRenderTarget;
+struct GrVkImageInfo;
+
+namespace skgpu {
+struct VulkanYcbcrConversionInfo;
+}

 namespace GrBackendFormats {
Based on the just removed code in the "Remove SkColorFilter::filterColor" commit:
google/skia@2727e4e

    This has been marked deprecated for some time, and all callers
    have migrated to filterColor4f.

Milestone 130
-------------
  * `SkColorFilter::filterColor` has been removed. Please use `SkColorFilter::filterColor4f` instead.
@HinTak HinTak mentioned this pull request Sep 18, 2024
@HinTak HinTak merged commit 165602f into kyamagu:main Sep 20, 2024
13 checks passed
@HinTak HinTak deleted the m130-public branch September 20, 2024 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant