<?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>245025</bug_id>
          
          <creation_ts>2022-09-10 06:21:48 -0700</creation_ts>
          <short_desc>[css-grid] span i should not serialize to span 1 i</short_desc>
          <delta_ts>2022-09-23 23:32:09 -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>CSS</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          <see_also>https://github.com/web-platform-tests/wpt/pull/36010</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>
          
          <blocked>204611</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Tim Nguyen (:ntim)">ntim</reporter>
          <assigned_to name="Matt Woodrow">mattwoodrow</assigned_to>
          <cc>mattwoodrow</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1897514</commentid>
    <comment_count>0</comment_count>
    <who name="Tim Nguyen (:ntim)">ntim</who>
    <bug_when>2022-09-10 06:21:48 -0700</bug_when>
    <thetext>See https://wpt.fyi/results/css/css-grid/parsing/grid-area-computed.html?label=master&amp;label=experimental&amp;product=chrome&amp;product=firefox&amp;product=safari&amp;aligned&amp;view=interop&amp;q=label%3Ainterop-2021-grid</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1897515</commentid>
    <comment_count>1</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2022-09-10 06:22:11 -0700</bug_when>
    <thetext>&lt;rdar://problem/99778041&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1900275</commentid>
    <comment_count>2</comment_count>
    <who name="Matt Woodrow">mattwoodrow</who>
    <bug_when>2022-09-21 19:06:49 -0700</bug_when>
    <thetext>This test is invalid :(

It&apos;s checking that `grid-column-start: span 1 i` round-trips exactly, as does `grid-row-start: span i`.

The only way to make both of those tests is to preserve whether the &apos;1&apos; was explicitly passed in, and use that to decide how to serialise the computed value, rather than just using the shortest serialisation principle.

Gecko is passing this because they parse an implicit integer as 0, and omit serializing the integer if it&apos;s 0. They then covert 0 to 1 at used value time.

The spec for parsing &apos;span&apos;: If the &lt;integer&gt; is omitted, it defaults to 1. Negative integers or zero are invalid.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1900282</commentid>
    <comment_count>3</comment_count>
    <who name="Matt Woodrow">mattwoodrow</who>
    <bug_when>2022-09-21 19:23:06 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/4587</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1900829</commentid>
    <comment_count>4</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2022-09-23 23:32:08 -0700</bug_when>
    <thetext>Committed 254821@main (fc4a525dc150): &lt;https://commits.webkit.org/254821@main&gt;

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

    </bug>

</bugzilla>