[webkit-changes] [WebKit/WebKit] dc6a01: Enable UnifiedPDF compile time flag on Cocoa platf...
Tim Horton
noreply at github.com
Fri Feb 23 23:08:19 PST 2024
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: dc6a01c9fd207e1b1394a1cb8261b410a1909cf6
https://github.com/WebKit/WebKit/commit/dc6a01c9fd207e1b1394a1cb8261b410a1909cf6
Author: Tim Horton <timothy_horton at apple.com>
Date: 2024-02-23 (Fri, 23 Feb 2024)
Changed paths:
M Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml
M Source/WTF/wtf/PlatformEnableCocoa.h
M Source/WTF/wtf/PlatformHave.h
M Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.h
M Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.mm
M Source/WebKit/WebProcess/Plugins/WebPluginInfoProvider.cpp
M Tools/WebKitTestRunner/TestOptions.cpp
Log Message:
-----------
Enable UnifiedPDF compile time flag on Cocoa platforms
https://bugs.webkit.org/show_bug.cgi?id=269440
Reviewed by Simon Fraser.
* Source/WTF/wtf/PlatformEnableCocoa.h:
Turn it on.
* Source/WTF/wtf/PlatformHave.h:
Add a define for some CG API added recently.
* Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.mm:
(WebKit::UnifiedPDFPlugin::pdfElementTypesForPluginPoint const):
(WebKit::UnifiedPDFPlugin::handlePDFActionForAnnotation):
Adopt defines for not-yet-everywhere API.
(WebKit::UnifiedPDFPlugin::shouldShowDebugIndicators const):
(WebKit::UnifiedPDFPlugin::didChangeSettings):
(WebKit::UnifiedPDFPlugin::paintPDFContent):
Fix the build on old compilers.
* Tools/WebKitTestRunner/TestOptions.cpp:
(WTR::TestOptions::defaults):
Disable by default in test runner for now, so we have to explicitly opt tests into unified PDF.
We'll swap this at some point.
* Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml:
* Source/WebKit/WebProcess/Plugins/WebPluginInfoProvider.cpp:
(WebKit::pluginInfoVector):
(WebKit::WebPluginInfoProvider::pluginInfo):
(WebKit::WebPluginInfoProvider::webVisiblePluginInfo):
Fix a longstanding mis-assumption that if ENABLE(PDF_PLUGIN) is true, we
will always have PDF plugins (the macOS version was never runtime-disableable).
This mistake previously caused us to pretend to have a PDF plugin on iOS
when Unified PDF was enabled at build time, even if it was disabled at runtime,
breaking our fallback to PDFDocumentImage-based presentation.
Canonical link: https://commits.webkit.org/275274@main
To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications
More information about the webkit-changes
mailing list