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
[Skia] Fix support for custom cursors with GTK #26388
[Skia] Fix support for custom cursors with GTK #26388
Conversation
EWS run on previous version of this PR (hash 45b3f88) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left one comment, please take a look.
45b3f88
to
1de06bd
Compare
EWS run on previous version of this PR (hash 1de06bd) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@TingPing my apologies for the churn, I think in my earlier review I wasn't clear about needing to change the accessor used for the pixel data π
1de06bd
to
2ab2574
Compare
EWS run on current version of this PR (hash 2ab2574) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
However you can construct a GdkTexture/cairo_surface from it and then create a GdkPixbuf off that, so I took this approach.
@TingPing That's a clever solution, and I like it that makes the code somewhat more consistent and easier to follow. Thanks!
https://bugs.webkit.org/show_bug.cgi?id=271613 Reviewed by Adrian Perez de Castro. * Source/WebCore/platform/graphics/gtk/GdkSkiaUtilities.cpp: (WebCore::skiaImageToGdkPixbuf): * Source/WebCore/platform/graphics/gtk/GdkSkiaUtilities.h: * Source/WebCore/platform/graphics/gtk/ImageAdapterGtk.cpp: (WebCore::ImageAdapter::gdkPixbuf): (WebCore::ImageAdapter::gdkTexture): * Source/WebCore/platform/gtk/CursorGtk.cpp: (WebCore::createCustomCursor): destroy notify was not being called. * Source/WebCore/platform/graphics/skia/GraphicsContextSkia.cpp: (WebCore::GraphicsContextSkia::~GraphicsContextSkia): Canonical link: https://commits.webkit.org/276737@main
2ab2574
to
ee489d2
Compare
Committed 276737@main (ee489d2): https://commits.webkit.org/276737@main Reviewed commits have been landed. Closing PR #26388 and removing active labels. |
ee489d2
2ab2574