<?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>275485</bug_id>
          
          <creation_ts>2024-06-14 07:56:29 -0700</creation_ts>
          <short_desc>[WPE][GTK] Bubblewrap sandbox should not kill auxiliary process when UI process terminates</short_desc>
          <delta_ts>2024-07-01 10:19:32 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKitGTK</component>
          <version>Other</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Michael Catanzaro">mcatanzaro</reporter>
          <assigned_to name="Michael Catanzaro">mcatanzaro</assigned_to>
          <cc>bugs-noreply</cc>
    
    <cc>mcatanzaro</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2041345</commentid>
    <comment_count>0</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2024-06-14 07:56:29 -0700</bug_when>
    <thetext>The bubblewrap sandbox should not kill auxiliary processes when the UI process terminates. Instead, let the child process terminate cleanly. The idea here is to match the behavior of unsandboxed WebKit and flatpaked WebKit and reduce the odds of bugs going unnoticed. This will allow exit handlers to run; notably, EGL termination has historically been a source of crashes. This will also avoid subverting the purpose of the watchdog thread that crashes the web process 10 seconds after the UI process exits.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2041364</commentid>
    <comment_count>1</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2024-06-14 08:49:55 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/29825</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2041583</commentid>
    <comment_count>2</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2024-06-16 05:43:57 -0700</bug_when>
    <thetext>Committed 280061@main (5d324a1c159c): &lt;https://commits.webkit.org/280061@main&gt;

Reviewed commits have been landed. Closing PR #29825 and removing active labels.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2043971</commentid>
    <comment_count>3</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2024-07-01 07:39:52 -0700</bug_when>
    <thetext>Carlos Garcia discovered this doesn&apos;t actually work. I didn&apos;t actually test to ensure the web process would actually shut down cleanly.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2043993</commentid>
    <comment_count>4</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2024-07-01 10:19:32 -0700</bug_when>
    <thetext>Sorry, I misunderstood. Carlos is saying it works for GTK but not for WPE. That seems extremely strange, though.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>