Skip to content

[GTK][WPE] Add webkit_feature_list_find() to the API#1699

Open
pgorszkowski-igalia wants to merge 1 commit into
wpe-2.46from
pgorszkowski/2.46/add_webkit_feature_list_find_to_the_API
Open

[GTK][WPE] Add webkit_feature_list_find() to the API#1699
pgorszkowski-igalia wants to merge 1 commit into
wpe-2.46from
pgorszkowski/2.46/add_webkit_feature_list_find_to_the_API

Conversation

@pgorszkowski-igalia

@pgorszkowski-igalia pgorszkowski-igalia commented Jul 10, 2026

Copy link
Copy Markdown

https://bugs.webkit.org/show_bug.cgi?id=307069

Reviewed by Michael Catanzaro and Patrick Griffis.

Add a convenience function to find a WebKitFeature from a given WebKitFeatureList. This will avoid code repetition in different projects.

  • Source/WebKit/UIProcess/API/glib/WebKitFeature.cpp: (webkit_feature_list_find):
  • Source/WebKit/UIProcess/API/glib/WebKitFeature.h.in:
  • Tools/MiniBrowser/gtk/main.c: (parseFeaturesOptionCallback):
    (findFeature): Deleted. Replaced with webkit_feature_list_find().
  • Tools/MiniBrowser/wpe/main.cpp: (activate):
    (findFeature): Deleted. Replaced with webkit_feature_list_find().
  • Tools/TestWebKitAPI/Tests/WebKitGLib/TestWebKitSettings.cpp: (testWebKitFeatures): Add a few test cases for the new function.

Canonical link: https://commits.webkit.org/307348@main
85fd7ef

Build-Tests Layout-Tests
✅ 🛠 wpe-246-amd64-build ✅ 🧪 wpe-246-amd64-layout
✅ 🛠 wpe-246-arm32-build ✅ 🧪 wpe-246-arm32-layout

https://bugs.webkit.org/show_bug.cgi?id=307069

Reviewed by Michael Catanzaro and Patrick Griffis.

Add a convenience function to find a WebKitFeature from a given
WebKitFeatureList. This will avoid code repetition in different
projects.

* Source/WebKit/UIProcess/API/glib/WebKitFeature.cpp:
(webkit_feature_list_find):
* Source/WebKit/UIProcess/API/glib/WebKitFeature.h.in:
* Tools/MiniBrowser/gtk/main.c:
(parseFeaturesOptionCallback):
(findFeature): Deleted. Replaced with webkit_feature_list_find().
* Tools/MiniBrowser/wpe/main.cpp:
(activate):
(findFeature): Deleted. Replaced with webkit_feature_list_find().
* Tools/TestWebKitAPI/Tests/WebKitGLib/TestWebKitSettings.cpp:
(testWebKitFeatures): Add a few test cases for the new function.

Canonical link: https://commits.webkit.org/307348@main
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants