<?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>266199</bug_id>
          
          <creation_ts>2023-12-10 17:20:34 -0800</creation_ts>
          <short_desc>[WTF] Distinguish SerialFunctionDispatcher that are refcounted</short_desc>
          <delta_ts>2023-12-11 01:20:56 -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>Web Template Framework</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <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>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Jean-Yves Avenard [:jya]">jean-yves.avenard</reporter>
          <assigned_to name="Jean-Yves Avenard [:jya]">jean-yves.avenard</assigned_to>
          <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1998528</commentid>
    <comment_count>0</comment_count>
    <who name="Jean-Yves Avenard [:jya]">jean-yves.avenard</who>
    <bug_when>2023-12-10 17:20:34 -0800</bug_when>
    <thetext>Right now, SerialFunctionDispatcher is designed to have an explicit lifetime.

All SerialFunctionDispatcher but one are currently ref counted.

We need to be able to easily determine if a SerialFunctionDispatcher is refounted by having a RefCountedSerialFunctionDispatcher

This change isn&apos;t aimed at resolving the SerialFunctionDispatcher API contact, which is there&apos;s no guarantee currently that the dispatched runnable will actually be run and that the SerialFunctionDispatcher got shutdown.

RefCountedSerialFunctionDispatcher is just about enforcing the lifetime of the object, not the lifetime of the API contract.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1998542</commentid>
    <comment_count>1</comment_count>
    <who name="Jean-Yves Avenard [:jya]">jean-yves.avenard</who>
    <bug_when>2023-12-10 21:14:54 -0800</bug_when>
    <thetext>Correction: 
RefCountedSerialFunctionDispatcher will enforce that the function dispatched is guaranteed to run.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1998543</commentid>
    <comment_count>2</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2023-12-10 21:17:38 -0800</bug_when>
    <thetext>&lt;rdar://problem/119476929&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1998544</commentid>
    <comment_count>3</comment_count>
    <who name="Jean-Yves Avenard [:jya]">jean-yves.avenard</who>
    <bug_when>2023-12-10 21:17:45 -0800</bug_when>
    <thetext>&lt;rdar://problem/119476929&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1998545</commentid>
    <comment_count>4</comment_count>
    <who name="Jean-Yves Avenard [:jya]">jean-yves.avenard</who>
    <bug_when>2023-12-10 21:22:43 -0800</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/21617</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1998558</commentid>
    <comment_count>5</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2023-12-11 01:20:54 -0800</bug_when>
    <thetext>Committed 271860@main (cfe5a3b9ba53): &lt;https://commits.webkit.org/271860@main&gt;

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

    </bug>

</bugzilla>