[webkit-changes] [WebKit/WebKit] 9ccc5c: [GLIB] Unify code for accessing default `cairo_fon...
Vitaly Dyachkov
noreply at github.com
Wed Feb 15 01:02:32 PST 2023
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 9ccc5caacd23526def9cf7111283b5233be038e1
https://github.com/WebKit/WebKit/commit/9ccc5caacd23526def9cf7111283b5233be038e1
Author: Vitaly Dyachkov <vitaly at igalia.com>
Date: 2023-02-15 (Wed, 15 Feb 2023)
Changed paths:
M Source/WebCore/platform/Cairo.cmake
M Source/WebCore/platform/graphics/cairo/CairoUtilities.cpp
M Source/WebCore/platform/graphics/cairo/CairoUtilities.h
M Source/WebCore/platform/graphics/gtk/GdkCairoUtilities.cpp
M Source/WebKit/WebProcess/glib/WebProcessGLib.cpp
M Source/WebKit/WebProcess/gtk/GtkSettingsManagerProxy.cpp
M Source/WebKit/WebProcess/gtk/GtkSettingsManagerProxy.h
M Tools/TestWebKitAPI/Tests/WebCore/ComplexTextController.cpp
Log Message:
-----------
[GLIB] Unify code for accessing default `cairo_font_options`
https://bugs.webkit.org/show_bug.cgi?id=252057
Reviewed by Carlos Garcia Campos.
Currently, there are two different implementations of
`getDefaultCairoFontOptions()` on WPE and GTK.
WPE version simply returns a static value created by
`cairo_font_options_create()`, whereas GTK one is creating a singleton
class that stores and updates an instance variable to keep it in sync
with `GtkSettings`.
This patch makes `getDefaultCairoFontOptions()` simply return a static
value and adds `setDefaultCairoFontOptions()` function. GTK port use it
to set a new value when `GtkSettingsManagerProxy` applies new settings.
* Source/WebCore/platform/graphics/cairo/CairoUtilities.cpp:
(WebCore::getDefaultCairoFontOptions):
(WebCore::setDefaultCairoFontOptions):
* Source/WebCore/platform/graphics/cairo/CairoUtilities.h:
* Source/WebCore/platform/graphics/gtk/GdkCairoUtilities.cpp:
(WebCore::SystemFontOptions::singleton): Deleted.
(WebCore::SystemFontOptions::SystemFontOptions): Deleted.
(WebCore::SystemFontOptions::fontOptions const): Deleted.
(WebCore::SystemFontOptions::updateFontOptions): Deleted.
(WebCore::getDefaultCairoFontOptions): Deleted.
* Source/WebKit/WebProcess/gtk/GtkSettingsManagerProxy.cpp:
(WebKit::GtkSettingsManagerProxy::applySettings):
(WebKit::GtkSettingsManagerProxy::applyFontOptions):
* Source/WebKit/WebProcess/gtk/GtkSettingsManagerProxy.h:
Canonical link: https://commits.webkit.org/260301@main
More information about the webkit-changes
mailing list