<?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>283507</bug_id>
          
          <creation_ts>2024-11-21 11:32:50 -0800</creation_ts>
          <short_desc>Pack CSS::PrimitiveNumeric</short_desc>
          <delta_ts>2024-11-22 12:50:15 -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>New Bugs</component>
          <version>Safari 18</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="Sam Weinig">sam</reporter>
          <assigned_to name="Sam Weinig">sam</assigned_to>
          <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2076761</commentid>
    <comment_count>0</comment_count>
    <who name="Sam Weinig">sam</who>
    <bug_when>2024-11-21 11:32:50 -0800</bug_when>
    <thetext>Pack CSS::PrimitiveNumeric by replacing std::variant with a union, and using the CSSUnitType to determine the correct alternative.

An alternative we may want to explore in the future when we no longer have the old CSSValue system around is creating a type per unit (e.g. Px { double value; }; Em { double value; }; ...) and using a std::variant with them, but since we still need to utilize CSSUnitTypes for the moment, that would be less efficient.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2076770</commentid>
    <comment_count>1</comment_count>
    <who name="Sam Weinig">sam</who>
    <bug_when>2024-11-21 11:56:21 -0800</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/36966</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2077111</commentid>
    <comment_count>2</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2024-11-22 12:49:49 -0800</bug_when>
    <thetext>Committed 286978@main (2ba07c2f90a7): &lt;https://commits.webkit.org/286978@main&gt;

Reviewed commits have been landed. Closing PR #36966 and removing active labels.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2077112</commentid>
    <comment_count>3</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2024-11-22 12:50:15 -0800</bug_when>
    <thetext>&lt;rdar://problem/140435621&gt;</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>