<?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>293190</bug_id>
          
          <creation_ts>2025-05-17 14:42:05 -0700</creation_ts>
          <short_desc>HashTable doesn&apos;t need `ShouldValidateKey` as template parameter for the whole class</short_desc>
          <delta_ts>2025-05-18 11:07:28 -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>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="Chris Dumez">cdumez</reporter>
          <assigned_to name="Chris Dumez">cdumez</assigned_to>
          <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2117926</commentid>
    <comment_count>0</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2025-05-17 14:42:05 -0700</bug_when>
    <thetext>HashTable doesn&apos;t need `ShouldValidateKey` as template parameter for the whole class. Instead, only add this template parameter for HashTable functions that actually need it. This reduces unnecessary code duplication, reduces binary size in WebCore by 1MB and avoids redundant key validations since we can now be more fine-grained.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2117927</commentid>
    <comment_count>1</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2025-05-17 14:44:09 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/45555</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2117991</commentid>
    <comment_count>2</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2025-05-18 11:06:10 -0700</bug_when>
    <thetext>Committed 295078@main (a8416d2c6a3b): &lt;https://commits.webkit.org/295078@main&gt;

Reviewed commits have been landed. Closing PR #45555 and removing active labels.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2117992</commentid>
    <comment_count>3</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2025-05-18 11:07:28 -0700</bug_when>
    <thetext>&lt;rdar://problem/151562991&gt;</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>