Branch: refs/heads/webkitglib/2.50
Home: https://github.com/WebKit/WebKit
Commit: b20741f230b83e70f13ecb96206b9f29ee63c3ca
https://github.com/WebKit/WebKit/commit/b20741f230b83e70f13ecb96206b9f29ee63c3ca
Author: Chris Dumez <[email protected]>
Date: 2026-03-30 (Mon, 30 Mar 2026)
Changed paths:
M Source/WTF/wtf/StdLibExtras.h
Log Message:
-----------
Cherry-pick 310026@main (5b0e5ea8452e).
https://bugs.webkit.org/show_bug.cgi?id=310777
WTF::contains(std::span, std::span) should return a bool, not a size_t
https://bugs.webkit.org/show_bug.cgi?id=310777
Reviewed by Ryosuke Niwa.
* Source/WTF/wtf/StdLibExtras.h:
(WTF::contains):
Canonical link: https://commits.webkit.org/310026@main
Canonical link: https://commits.webkit.org/298234.519@webkitglib/2.50
Commit: 182440c54241130b49767301e12b17122be0812a
https://github.com/WebKit/WebKit/commit/182440c54241130b49767301e12b17122be0812a
Author: Claudio Saavedra <[email protected]>
Date: 2026-03-31 (Tue, 31 Mar 2026)
Changed paths:
M Source/WebKit/UIProcess/API/glib/IconDatabase.cpp
Log Message:
-----------
Cherry-pick 310242@main (e284abe8a7fa).
https://bugs.webkit.org/show_bug.cgi?id=311112
[GLIB] WebKit::IconDatabase::~IconDatabase(): ASSERTION FAILED:
isMainRunLoop()
https://bugs.webkit.org/show_bug.cgi?id=311112
Reviewed by Adrian Perez de Castro.
The destructor for IconDatabase checks that it's being invoked from the
main loop
thread. But the many worker's lambdas in IconDatabase are capturing a
strong reference
to the object which, depending on timing, can be the last one alive,
so that when the lambda goes out of scope, it can end up destroying the
object from
the worker thread and hitting the assertion in the destructor.
This is particularly crashy for the worker lambda in
IconDatabase::loadIconsForPageURL().
We can fix this here by moving the protectedThis reference instead of
creating a new one,
that way the ownership of the reference goes back to the main loop thread.
This needs to be fixed for other methods in this class, but we will do that
in a subsequent patch as other lambda uses don't need to capture the icon
database object so the fix can be different.
* Source/WebKit/UIProcess/API/glib/IconDatabase.cpp:
(WebKit::IconDatabase::loadIconsForPageURL):
Canonical link: https://commits.webkit.org/310242@main
Canonical link: https://commits.webkit.org/298234.520@webkitglib/2.50
Compare: https://github.com/WebKit/WebKit/compare/75f5535a9a3b...182440c54241
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications
