<?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>286274</bug_id>
          
          <creation_ts>2025-01-20 13:46:47 -0800</creation_ts>
          <short_desc>[WebDriver] Properly use smart pointers instead of raw pointers for the HTTPServer, WebSocketServer, and WebDriverService</short_desc>
          <delta_ts>2025-04-24 18:50:02 -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>WebDriver</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>NEW</bug_status>
          <resolution></resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=291925</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>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Lauro Moura">lmoura</reporter>
          <assigned_to name="Lauro Moura">lmoura</assigned_to>
          <cc>bburg</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2088567</commentid>
    <comment_count>0</comment_count>
    <who name="Lauro Moura">lmoura</who>
    <bug_when>2025-01-20 13:46:47 -0800</bug_when>
    <thetext>In WebDriverBiDi&apos;s a number of classes (WebDriverService, HTTPServer, WebSocketServer) held each other through plain references, and, in the case of WebSocketServer, mixed with WeakPtr usage.

We should make them use smart pointers correctly, per our safer cpp guidelines: https://github.com/WebKit/WebKit/wiki/Safer-CPP-Guidelines#use-smart-pointers-instead-of-raw-pointers-for-all-data-members

Patch incoming.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2088616</commentid>
    <comment_count>1</comment_count>
    <who name="Lauro Moura">lmoura</who>
    <bug_when>2025-01-20 19:05:30 -0800</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/39310</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2090030</commentid>
    <comment_count>2</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2025-01-27 13:47:11 -0800</bug_when>
    <thetext>&lt;rdar://problem/143715487&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2113019</commentid>
    <comment_count>3</comment_count>
    <who name="Lauro Moura">lmoura</who>
    <bug_when>2025-04-24 18:50:02 -0700</bug_when>
    <thetext>(In reply to Lauro Moura from comment #1)
&gt; Pull request: https://github.com/WebKit/WebKit/pull/39310

This PR was closed in favor of a simpler fix in bug291925. We can revisit it later</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>