<?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>158060</bug_id>
          
          <creation_ts>2016-05-25 00:18:28 -0700</creation_ts>
          <short_desc>[css-grid] Turn on ENABLE_CSS_GRID_LAYOUT by default</short_desc>
          <delta_ts>2016-05-25 12:38:41 -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>New Bugs</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></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>60731</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Manuel Rego Casasnovas">rego</reporter>
          <assigned_to name="Manuel Rego Casasnovas">rego</assigned_to>
          <cc>cgarcia</cc>
    
    <cc>commit-queue</cc>
    
    <cc>gyuyoung.kim</cc>
    
    <cc>jfernandez</cc>
    
    <cc>jh718.park</cc>
    
    <cc>mcatanzaro</cc>
    
    <cc>ossy</cc>
    
    <cc>rego</cc>
    
    <cc>svillar</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1196233</commentid>
    <comment_count>0</comment_count>
    <who name="Manuel Rego Casasnovas">rego</who>
    <bug_when>2016-05-25 00:18:28 -0700</bug_when>
    <thetext>[css-grid] Turn on ENABLE_CSS_GRID_LAYOUT by default</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1196234</commentid>
    <comment_count>1</comment_count>
      <attachid>279746</attachid>
    <who name="Manuel Rego Casasnovas">rego</who>
    <bug_when>2016-05-25 00:21:08 -0700</bug_when>
    <thetext>Created attachment 279746
Patch

As @mcatanzaro suggested on bug #157391, I&apos;m enabling CSS Grid Layout build flag by default.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1196264</commentid>
    <comment_count>2</comment_count>
      <attachid>279746</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2016-05-25 03:32:15 -0700</bug_when>
    <thetext>Comment on attachment 279746
Patch

This only affects platforms using CMake. Is this already correct on Mac and iOS?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1196301</commentid>
    <comment_count>3</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2016-05-25 05:36:58 -0700</bug_when>
    <thetext>(In reply to comment #2)
&gt; Comment on attachment 279746 [details]
&gt; Patch
&gt; 
&gt; This only affects platforms using CMake. Is this already correct on Mac and
&gt; iOS?

It is already enabled in FeatureDefines.xcconfig files.

What about the EFL port? Don&apos;t you guys want to enable this feature?
It seems EFL port is the last one disabling CSS Grid Layout.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1196302</commentid>
    <comment_count>4</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2016-05-25 05:38:22 -0700</bug_when>
    <thetext>Ah, I was wrong, it is already enabled on EFL too because of FeatureList.pm.
But in this case there is no reason to disable it in cmake.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1196357</commentid>
    <comment_count>5</comment_count>
      <attachid>279746</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2016-05-25 09:22:14 -0700</bug_when>
    <thetext>Comment on attachment 279746
Patch

OK.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1196414</commentid>
    <comment_count>6</comment_count>
      <attachid>279746</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2016-05-25 12:38:37 -0700</bug_when>
    <thetext>Comment on attachment 279746
Patch

Clearing flags on attachment: 279746

Committed r201396: &lt;http://trac.webkit.org/changeset/201396&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1196415</commentid>
    <comment_count>7</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2016-05-25 12:38:41 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>279746</attachid>
            <date>2016-05-25 00:21:08 -0700</date>
            <delta_ts>2016-05-25 12:38:37 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-158060-20160525092214.patch</filename>
            <type>text/plain</type>
            <size>1845</size>
            <attacher name="Manuel Rego Casasnovas">rego</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjAxMzI1CmRpZmYgLS1naXQgYS9Tb3VyY2UvY21ha2UvV2Vi
S2l0RmVhdHVyZXMuY21ha2UgYi9Tb3VyY2UvY21ha2UvV2ViS2l0RmVhdHVyZXMuY21ha2UKaW5k
ZXggOWMxY2EwNTExZTkzZWY4MDhmZWVkMmIzODRhYzMyZjc3NDFmOTQ3My4uMTY2MzFmMjRkOWQx
ODIwMTM0NjRhMWNkMGIxYTIwMjc3OTI4MTc2MiAxMDA2NDQKLS0tIGEvU291cmNlL2NtYWtlL1dl
YktpdEZlYXR1cmVzLmNtYWtlCisrKyBiL1NvdXJjZS9jbWFrZS9XZWJLaXRGZWF0dXJlcy5jbWFr
ZQpAQCAtODAsNyArODAsNyBAQCBtYWNybyhXRUJLSVRfT1BUSU9OX0JFR0lOKQogICAgIFdFQktJ
VF9PUFRJT05fREVGSU5FKEVOQUJMRV9DU1NfQk9YX0RFQ09SQVRJT05fQlJFQUsgIlRvZ2dsZSBC
b3ggRGVjb3JhdGlvbiBicmVhayAoQ1NTIEJhY2tncm91bmRzIGFuZCBCb3JkZXJzKSBzdXBwb3J0
IiBQUklWQVRFIE9OKQogICAgIFdFQktJVF9PUFRJT05fREVGSU5FKEVOQUJMRV9DU1NfQ09NUE9T
SVRJTkcgIlRvZ2dsZSBDU1MgQ09NUE9TSVRJTkcgc3VwcG9ydCIgUFJJVkFURSBPRkYpCiAgICAg
V0VCS0lUX09QVElPTl9ERUZJTkUoRU5BQkxFX0NTU19ERVZJQ0VfQURBUFRBVElPTiAiVG9nZ2xl
IENTUyBEZXZpY2UgQWRhcHRhdGlvbiBzdXBwb3J0IiBQUklWQVRFIE9GRikKLSAgICBXRUJLSVRf
T1BUSU9OX0RFRklORShFTkFCTEVfQ1NTX0dSSURfTEFZT1VUICJUb2dnbGUgQ1NTIEdyaWQgTGF5
b3V0IHN1cHBvcnQiIFBSSVZBVEUgT0ZGKQorICAgIFdFQktJVF9PUFRJT05fREVGSU5FKEVOQUJM
RV9DU1NfR1JJRF9MQVlPVVQgIlRvZ2dsZSBDU1MgR3JpZCBMYXlvdXQgc3VwcG9ydCIgUFJJVkFU
RSBPTikKICAgICBXRUJLSVRfT1BUSU9OX0RFRklORShFTkFCTEVfQ1NTX0lNQUdFX09SSUVOVEFU
SU9OICJUb2dnbGUgQ1NTIGltYWdlLW9yaWVudGF0aW9uIHN1cHBvcnQiIFBSSVZBVEUgT0ZGKQog
ICAgIFdFQktJVF9PUFRJT05fREVGSU5FKEVOQUJMRV9DU1NfSU1BR0VfUkVTT0xVVElPTiAiVG9n
Z2xlIENTUyBpbWFnZS1yZXNvbHV0aW9uIHN1cHBvcnQiIFBSSVZBVEUgT0ZGKQogICAgIFdFQktJ
VF9PUFRJT05fREVGSU5FKEVOQUJMRV9DU1NfSU1BR0VfU0VUICJUb2dnbGUgQ1NTIGltYWdlLXNl
dCBzdXBwb3J0IiBQUklWQVRFIE9GRikKZGlmZiAtLWdpdCBhL0NoYW5nZUxvZyBiL0NoYW5nZUxv
ZwppbmRleCAzMjhmOTg2NWQ2OWMwMmIzOGFhMDE3YWQ5OTI1NzFiOGM2YzdiZWQyLi5kM2U0ZDc1
MTM0NDk1MzQ1M2E0N2I3YjY1MWJmZWE2ZjZkMjAxODUzIDEwMDY0NAotLS0gYS9DaGFuZ2VMb2cK
KysrIGIvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTYgQEAKKzIwMTYtMDUtMjUgIE1hbnVlbCBSZWdv
IENhc2Fzbm92YXMgIDxyZWdvQGlnYWxpYS5jb20+CisKKyAgICAgICAgW2Nzcy1ncmlkXSBUdXJu
IG9uIEVOQUJMRV9DU1NfR1JJRF9MQVlPVVQgYnkgZGVmYXVsdAorICAgICAgICBodHRwczovL2J1
Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTU4MDYwCisKKyAgICAgICAgUmV2aWV3ZWQg
YnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgVGhlIHJ1bnRpbWUgZmxhZyBpcyBkaXNhYmxl
ZCBieSBkZWZhdWx0LAorICAgICAgICBidXQgd2Ugd2FudCB0byBidWlsZCBDU1MgR3JpZCBMYXlv
dXQgYnkgZGVmYXVsdC4KKyAgICAgICAgT3RoZXJ3aXNlIHRoZSBydW50aW1lIGZsYWcgd291bGQg
YmUgdXNlbGVzcy4KKworICAgICAgICAqIFNvdXJjZS9jbWFrZS9XZWJLaXRGZWF0dXJlcy5jbWFr
ZToKKwogMjAxNi0wNS0yMiAgQnJhZHkgRWlkc29uICA8YmVpZHNvbkBhcHBsZS5jb20+CiAKICAg
ICAgICAgTW92ZSB0byBDKysxNC4K
</data>

          </attachment>
      

    </bug>

</bugzilla>