Bug 236668

Summary:	Introduce postMessage mechanism for PDF.js viewer and use it for context menu items
Product:	WebKit	Reporter:	pascoe <pascoe>
Component:	WebKit Misc.	Assignee:	Tim Nguyen (:ntim) <ntim>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	mcatanzaro, ntim, webkit-bug-importer
Priority:	P2	Keywords:	InRadar
Version:	WebKit Nightly Build
Hardware:	Unspecified
OS:	Unspecified
See Also:	https://bugs.webkit.org/show_bug.cgi?id=251887
Bug Depends on:
Bug Blocks:	235969

Description pascoe@apple.com 2022-02-15 13:56:01 PST

In https://bugs.webkit.org/show_bug.cgi?id=236525, it would be more efficient to directly get the PDFJSContentScript instead of using executeScriptIgnoringException.

Comment 1 Radar WebKit Bug Importer 2022-02-15 13:56:12 PST

<rdar://problem/88985691>

Comment 2 Tim Nguyen (:ntim) 2022-02-15 17:14:45 PST

After discussing with Chris, it sounds like using postMessage here is the better option, for security reasons. When process isolation happens, the iframe and PDFDocument will be in different processes (since they are in different origins), so accessing functions from the iframe will not be safe.

Comment 3 Tim Nguyen (:ntim) 2022-08-07 00:14:10 PDT

Pull request: https://github.com/WebKit/WebKit/pull/3083

Comment 4 EWS 2022-08-08 11:28:04 PDT

Committed 253223@main (5b3fe7dc3092): <https://commits.webkit.org/253223@main>

Reviewed commits have been landed. Closing PR #3083 and removing active labels.