Bug 272676
Summary: | Rename screenDPI() to fontDPI() where used for font scaling | ||
---|---|---|---|
Product: | WebKit | Reporter: | Glen Whitney <gwhitneycom5> |
Component: | WebKitGTK | Assignee: | Diego Pino <dpino> |
Status: | RESOLVED FIXED | ||
Severity: | Minor | CC: | bugs-noreply, dpino |
Priority: | P2 | ||
Version: | Other | ||
Hardware: | Unspecified | ||
OS: | Unspecified |
Glen Whitney
As reported in https://bugs.webkit.org/show_bug.cgi?id=247980, WebKitGTK has display and font scaling issues on certain combinations of display pixel density and GTK settings. Based on the code and the proposed fix in https://github.com/WebKit/WebKit/pull/26938, it appears that these problems have arisen because in the GTK context, "screenDPI" in some places in the code refers to the pixel density of the underlying display, whereas in others it refers to a nominal DPI used for the purposes of scaling fonts. The latter uses should be renamed as fontDPI. This bug is being filed at @mcatanzaro's suggestion to provide an associated bug for the renaming PR https://github.com/WebKit/WebKit/pull/26954.
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Glen Whitney
Pull request: https://github.com/WebKit/WebKit/pull/26954
EWS
Committed 277537@main (51f27458d0d4): <https://commits.webkit.org/277537@main>
Reviewed commits have been landed. Closing PR #26954 and removing active labels.
Diego Pino
The WPE-Linux-ARM32-bit-Release-Debian-Stable-Build bot is failing after this change with the following error:
```
/usr/bin/ld: Source/WebCore/CMakeFiles/WebCore.dir/./__/__/WebCore/DerivedSources/unified-sources/UnifiedSource-3c72abbe-81.cpp.o: in function `WebCore::fontDPI()':
UnifiedSource-3c72abbe-81.cpp:(.text._ZN7WebCore7fontDPIEv+0x2): undefined reference to `WebCore::primaryScreenDisplayID()'
collect2: error: ld returned 1 exit status
[1953/2173] Building CXX object Source/WebKit/CMakeFiles/WebKit.dir/__/__/DerivedSources/WebKit/unified-sources/UnifiedSource-54928a2b-30.cpp.o
```
This bot builds with the flag `--cmakeargs=-DENABLE_WPE_PLATFORM_DRM=OFF`.
Diego Pino
Re-opening for pull request https://github.com/webkit/webkit/pull/27378
EWS
Committed 277609@main (8965f9b7d1be): <https://commits.webkit.org/277609@main>
Reviewed commits have been landed. Closing PR #27378 and removing active labels.