<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>236668</bug_id>
          
          <creation_ts>2022-02-15 13:56:01 -0800</creation_ts>
          <short_desc>Introduce postMessage mechanism for PDF.js viewer and use it for context menu items</short_desc>
          <delta_ts>2023-02-16 12:37:40 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKit Misc.</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=251887</see_also>
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>235969</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="pascoe@apple.com">pascoe</reporter>
          <assigned_to name="Tim Nguyen (:ntim)">ntim</assigned_to>
          <cc>mcatanzaro</cc>
    
    <cc>ntim</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1841929</commentid>
    <comment_count>0</comment_count>
    <who name="pascoe@apple.com">pascoe</who>
    <bug_when>2022-02-15 13:56:01 -0800</bug_when>
    <thetext>In https://bugs.webkit.org/show_bug.cgi?id=236525, it would be more efficient to directly get the PDFJSContentScript instead of using executeScriptIgnoringException.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1841931</commentid>
    <comment_count>1</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2022-02-15 13:56:12 -0800</bug_when>
    <thetext>&lt;rdar://problem/88985691&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1842057</commentid>
    <comment_count>2</comment_count>
    <who name="Tim Nguyen (:ntim)">ntim</who>
    <bug_when>2022-02-15 17:14:45 -0800</bug_when>
    <thetext>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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1889489</commentid>
    <comment_count>3</comment_count>
    <who name="Tim Nguyen (:ntim)">ntim</who>
    <bug_when>2022-08-07 00:14:10 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/3083</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1889724</commentid>
    <comment_count>4</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2022-08-08 11:28:04 -0700</bug_when>
    <thetext>Committed 253223@main (5b3fe7dc3092): &lt;https://commits.webkit.org/253223@main&gt;

Reviewed commits have been landed. Closing PR #3083 and removing active labels.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>