RESOLVED FIXED 224366
[WPE][GTK] Use g_object_notify_by_pspec() whenever possible 
https://bugs.webkit.org/show_bug.cgi?id=224366
Summary [WPE][GTK] Use g_object_notify_by_pspec() whenever possible
Adrian Perez
Reported 2021-04-09 05:52:37 PDT
In the WebKit code base, every single time we tell GObject to emit a “notify::<propetyname>” signal we use “g_object_notify()”, which will do a lookup for the property given its name. Ideally the code should use “g_object_notify_by_pspec()” and avoid those lookups altogether.
Attachments
Patch (153.21 KB, patch)
2021-04-10 14:36 PDT, Adrian Perez 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Adrian Perez
Comment 1 2021-04-10 14:36:25 PDT
Created attachment 425689 [details] Patch
EWS Watchlist
Comment 2 2021-04-10 14:37:29 PDT
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See https://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API
EWS
Comment 3 2021-04-12 01:13:24 PDT
Committed r275816 (236386@main): <https://commits.webkit.org/236386@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 425689 [details].
Note You need to log in before you can comment on or make changes to this bug.