Bug 281751
| Summary: | [WPE] Rename WPEMonitor to WPEScreen | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Jani Hautakangas <jani> |
| Component: | WPE WebKit | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | bugs-noreply |
| Priority: | P2 | ||
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Jani Hautakangas
The current naming of "WPEMonitor" in the WPE Platform doesn’t fit well across all device form factors. "WPEMonitor" suggests an individual display screen, but this naming feels out of place for smaller devices such as mobile phones.
This naming convention is inherited from Gtk/Gdk, where GdkMonitor was introduced in GTK4 to replace the older GdkScreen. However, GdkMonitor was a compromise due to accumulated technical debt in GdkScreen.
Additionally, in WebKit, there are many "*Monitor" classes used to monitor events, progress, and other functionalities. "WPEMonitor" could easily be confused with these, leading to potential misunderstandings about its purpose. Therefore, renaming WPEMonitor to WPEScreen would provide a clearer and more appropriate representation of its function.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Jani Hautakangas
Pull request: https://github.com/WebKit/WebKit/pull/35433
EWS
Committed 285586@main (ac17704df9a5): <https://commits.webkit.org/285586@main>
Reviewed commits have been landed. Closing PR #35433 and removing active labels.