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
251266
[GTK] Use a weak reference of web view in webkitWebViewGetLoadDecisionForIcon
https://bugs.webkit.org/show_bug.cgi?id=251266
Summary
[GTK] Use a weak reference of web view in webkitWebViewGetLoadDecisionForIcon
Carlos Garcia Campos
Reported
2023-01-27 06:30:04 PST
We don't want to delay the dispose of a web view because an icon load decision is still pending, so let's use a weak reference instead of a strong one. This fixes memory leaks reported by unit tests with 2022 API now that favicons are enabled by default.
Attachments
Add attachment
proposed patch, testcase, etc.
Carlos Garcia Campos
Comment 1
2023-01-27 06:39:04 PST
Pull request:
https://github.com/WebKit/WebKit/pull/9228
EWS
Comment 2
2023-01-27 22:43:54 PST
Committed
259524@main
(5757289eb640): <
https://commits.webkit.org/259524@main
> Reviewed commits have been landed. Closing PR #9228 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