<?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>284976</bug_id>
          
          <creation_ts>2024-12-19 14:30:20 -0800</creation_ts>
          <short_desc>Add a feature flag for remote snapshotting</short_desc>
          <delta_ts>2025-01-06 20:51:30 -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>Layout and Rendering</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=284427</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>282664</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Said Abou-Hallawa">sabouhallawa</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>bfulgham</cc>
    
    <cc>simon.fraser</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>zalan</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2083109</commentid>
    <comment_count>0</comment_count>
    <who name="Said Abou-Hallawa">sabouhallawa</who>
    <bug_when>2024-12-19 14:30:20 -0800</bug_when>
    <thetext>Currently snapshotting a page to an image or a PDFDocument has to go through this IPC workflow: UIProcess -&gt; WebProcess -&gt; UIProcess. After 288113@main and with site isolation enabled, snapshotting will go through the following IPC workflow: UIProcess -&gt; WebProcess -&gt; GPUProcess -&gt; UIProcess. So any WebProcess can&apos;t see the drawing of the other WebProcesses.

To address the review comment https://github.com/WebKit/WebKit/pull/37756#discussion_r1891010340, a new feature flag will be introduced to allow testing the remote snapshotting separate from the site isolation. If site isolation is enabled, the main frame has to compose itself with all the remote iframes to get the correct snapshotting. This will be done in future patches.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2083110</commentid>
    <comment_count>1</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2024-12-19 14:31:36 -0800</bug_when>
    <thetext>&lt;rdar://problem/141783206&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2083112</commentid>
    <comment_count>2</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2024-12-19 14:32:14 -0800</bug_when>
    <thetext>&lt;rdar://problem/141783279&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2083126</commentid>
    <comment_count>3</comment_count>
    <who name="Said Abou-Hallawa">sabouhallawa</who>
    <bug_when>2024-12-19 15:11:33 -0800</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/38228</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2085301</commentid>
    <comment_count>4</comment_count>
    <who name="Said Abou-Hallawa">sabouhallawa</who>
    <bug_when>2025-01-06 20:51:30 -0800</bug_when>
    <thetext>Committed 288504@main (d9f35fa): https://commits.webkit.org/288504@main

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

    </bug>

</bugzilla>