<?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>247745</bug_id>
          
          <creation_ts>2022-11-10 09:51:06 -0800</creation_ts>
          <short_desc>Setting -webkit-column-count to auto is the same as not setting it.</short_desc>
          <delta_ts>2022-11-12 11:35:18 -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>Layout and Rendering</component>
          <version>Safari Technology Preview</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=247826</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="Ahmad Saleem">ahmad.saleem792</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>bfulgham</cc>
    
    <cc>simon.fraser</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>zalan</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1911570</commentid>
    <comment_count>0</comment_count>
    <who name="Ahmad Saleem">ahmad.saleem792</who>
    <bug_when>2022-11-10 09:51:06 -0800</bug_when>
    <thetext>Hi Team,

I tried to look for similar bug but couldn&apos;t find and it is again based of another Blink commit:

Blink Commit - https://chromium.googlesource.com/chromium/blink/+/da85d1a90375fe50a92cf46f8a49571850618294

Webkit GitHub Source - https://github.com/WebKit/WebKit/blob/fa2a14f12031baf24d5bc6d6727c3e25c5052589/Source/WebCore/rendering/style/RenderStyle.h#L1282

Test Case (Can&apos;t test) - https://jsfiddle.net/L35709ro/

Because it gives error - &quot;There&apos;s no web exposed API usable for this test. The bug doesn&apos;t affect layout in a reliable way. Please rerun inside the layout test framework.&quot;

I am happy to do PR and fix it but would appreciate input of others. Thanks!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1911780</commentid>
    <comment_count>1</comment_count>
    <who name="Ahmad Saleem">ahmad.saleem792</who>
    <bug_when>2022-11-11 03:47:33 -0800</bug_when>
    <thetext>I think the problem with this merge is that the test cases use internal APIs which are not present in Webkit.

I tried it here: https://github.com/WebKit/WebKit/pull/6368

 FAIL internals.updateStyleAndReturnAffectedElementCount() should be 1. Threw exception TypeError: internals.updateStyleAndReturnAffectedElementCount is not a function. (In &apos;internals.updateStyleAndReturnAffectedElementCount()&apos;, &apos;internals.updateStyleAndReturnAffectedElementCount&apos; is undefined)
 
FAIL internals.needsLayoutCount() should be 0. Threw exception TypeError: internals.needsLayoutCount is not a function. (In &apos;internals.needsLayoutCount()&apos;, &apos;internals.needsLayoutCount&apos; is undefined)

____

Any solution would be appreciated. Thanks!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1911909</commentid>
    <comment_count>2</comment_count>
    <who name="Ahmad Saleem">ahmad.saleem792</who>
    <bug_when>2022-11-11 15:06:51 -0800</bug_when>
    <thetext>Close my PR with following comment:

&quot;I have to close this bug because Webkit does not support &quot;-webkit-columns: auto &quot; properly and also don&apos;t have related Internals Webkit Testing Framework API to land this as it is. I have added first parsing bug as &quot;See Also&quot; while also explained the internal testing framework thing on the bug for future reference. Closing this now.&quot;

If we can land this without test case then it is good to go. Thanks!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1912014</commentid>
    <comment_count>3</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2022-11-12 11:34:13 -0800</bug_when>
    <thetext>Committed 256607@main (bc1d0765c50f): &lt;https://commits.webkit.org/256607@main&gt;

Reviewed commits have been landed. Closing PR #6368 and removing active labels.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1912015</commentid>
    <comment_count>4</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2022-11-12 11:35:18 -0800</bug_when>
    <thetext>&lt;rdar://problem/102278097&gt;</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>