[webkit-changes] [WebKit/WebKit] 9e49e1: [UnifiedPDF] [iOS] Plugin missing when loading QLP...

Abrar Rahman Protyasha noreply at github.com
Fri Feb 28 13:56:05 PST 2025


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 9e49e100a02cc5b87fce078376434733885e97b4
      https://github.com/WebKit/WebKit/commit/9e49e100a02cc5b87fce078376434733885e97b4
  Author: Abrar Rahman Protyasha <a_protyasha at apple.com>
  Date:   2025-02-28 (Fri, 28 Feb 2025)

  Changed paths:
    M Source/WebCore/dom/Document.cpp

  Log Message:
  -----------
  [UnifiedPDF] [iOS] Plugin missing when loading QLPreviewConverter-generated substituted data for supported MIME types
https://bugs.webkit.org/show_bug.cgi?id=288869
rdar://145863043

Reviewed by Tim Horton.

For certain MIME types, we generate PDF representations on-the-fly using
QLPreviewConverter, and then use that as the substituted data when
writing the plugin document. Unfortunately, in this process, we end up
applying sandbox changes that restrict plugins, which disallows the
UnifiedPDFPlugin from being instantiated. This is driven through
Document::applyQuickLookSandbox().

To fix this, we loosen the plugin restriction on the QL sandbox in the
case the QL sandbox is being applied to a PluginDocument. This can only
happen when we want to present QL converted data in the PDF plugin.

Tested manually. API test will follow in a future patch.

* Source/WebCore/dom/Document.cpp:
(WebCore::Document::applyQuickLookSandbox):

Canonical link: https://commits.webkit.org/291372@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