WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
302896
[WPE][GTK] WebKitFaviconDatabase.cpp:202:47: error: 'this' argument to member function 'leakRef' has type 'WTF::RefPtr<_cairo_surface> const', but function is not marked const
https://bugs.webkit.org/show_bug.cgi?id=302896
Summary
[WPE][GTK] WebKitFaviconDatabase.cpp:202:47: error: 'this' argument to member...
Michael Catanzaro
Reported
2025-11-20 14:37:31 PST
Our changes to WebKitFaviconDatabase have broken the build on s390x. Once I have successfully ensured the build succeeds, I will use this bug to submit a fix. /builddir/build/BUILD/webkitgtk-2.51.2-build/webkitgtk-2.51.2/Source/WebKit/UIProcess/API/glib/WebKitFaviconDatabase.cpp:202:47: error: 'this' argument to member function 'leakRef' has type 'WTF::RefPtr<_cairo_surface> const', but function is not marked const 202 | g_task_return_pointer(task.get(), icon.leakRef(), reinterpret_cast<GDestroyNotify>(cairo_surface_destroy)); | ^~~~ /builddir/build/BUILD/webkitgtk-2.51.2-build/webkitgtk-2.51.2/webkitgtk-6.0/redhat-linux-build/WTF/Headers/wtf/RefPtr.h:69:8: note: 'leakRef' declared here 69 | T* leakRef() WARN_UNUSED_RETURN; | ^ 1 error generated.
Attachments
Add attachment
proposed patch, testcase, etc.
Michael Catanzaro
Comment 1
2025-11-21 13:37:03 PST
Pull request:
https://github.com/WebKit/WebKit/pull/54344
EWS
Comment 2
2025-11-24 09:15:07 PST
Committed
303498@main
(36f65d4be38b): <
https://commits.webkit.org/303498@main
> Reviewed commits have been landed. Closing PR #54344 and removing active labels.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug