WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
236668
Introduce postMessage mechanism for PDF.js viewer and use it for context menu items
https://bugs.webkit.org/show_bug.cgi?id=236668
Summary
Introduce postMessage mechanism for PDF.js viewer and use it for context menu...
pascoe@apple.com
Reported
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.
Attachments
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2022-02-15 13:56:12 PST
<
rdar://problem/88985691
>
Tim Nguyen (:ntim)
Comment 2
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.
Tim Nguyen (:ntim)
Comment 3
2022-08-07 00:14:10 PDT
Pull request:
https://github.com/WebKit/WebKit/pull/3083
EWS
Comment 4
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.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug