<?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>104090</bug_id>
          
          <creation_ts>2012-12-04 22:59:29 -0800</creation_ts>
          <short_desc>Layout Test fast/css-grid-layout/grid-columns-rows-get-set-multiple.html, grid-columns-rows-get-set.html is failing</short_desc>
          <delta_ts>2013-01-08 19:22:52 -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>Tools / Tests</component>
          <version>528+ (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>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Takashi Sakamoto">tasak</reporter>
          <assigned_to name="Julien Chaffraix">jchaffraix</assigned_to>
          <cc>dglazkov</cc>
    
    <cc>jchaffraix</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>783415</commentid>
    <comment_count>0</comment_count>
    <who name="Takashi Sakamoto">tasak</who>
    <bug_when>2012-12-04 22:59:29 -0800</bug_when>
    <thetext>The following layout test is failing on Mac(Debug),

fast/css-grid-layout/grid-columns-rows-get-set-multiple.html
fast/css-grid-layout/grid-columns-rows-get-set.html

Probable cause:

crash log for DumpRenderTree (pid 1178):
STDOUT: &lt;empty&gt;
STDERR: ASSERTION FAILED: m_type == LengthTrackSizing
STDERR: ../rendering/style/GridTrackSize.h(54) : const WebCore::Length &amp;WebCore::GridTrackSize::length() const
STDERR: 1   0x45ce48be WebCore::GridTrackSize::length() const
STDERR: 2   0x45ce3e49 WebCore::RenderGrid::computePreferredLogicalWidths()
STDERR: 3   0x45c3dc52 WebCore::RenderBox::minPreferredLogicalWidth() const
STDERR: 4   0x45bbe156 WebCore::RenderBlock::computeBlockPreferredLogicalWidths()
STDERR: 5   0x45bbc471 WebCore::RenderBlock::computePreferredLogicalWidths()
STDERR: 6   0x45c3dc52 WebCore::RenderBox::minPreferredLogicalWidth() const
STDERR: 7   0x45bbe156 WebCore::RenderBlock::computeBlockPreferredLogicalWidths()
STDERR: 8   0x45bbc471 WebCore::RenderBlock::computePreferredLogicalWidths()
STDERR: 9   0x45c3dc52 WebCore::RenderBox::minPreferredLogicalWidth() const
STDERR: 10  0x45bbe156 WebCore::RenderBlock::computeBlockPreferredLogicalWidths()
STDERR: 11  0x45bbc471 WebCore::RenderBlock::computePreferredLogicalWidths()
STDERR: 12  0x45e7cd5b WebCore::RenderView::computePreferredLogicalWidths()
STDERR: 13  0x45c3dc52 WebCore::RenderBox::minPreferredLogicalWidth() const
STDERR: 14  0x435a3876 WebKit::WebFrameImpl::contentsPreferredWidth() const
STDERR: 15  0x42e4a050 WebViewHost::didUpdateLayout()
STDERR: 16  0x4367e90e WebKit::WebViewImpl::layoutUpdated(WebKit::WebFrameImpl*)
STDERR: 17  0x434af33a WebKit::ChromeClientImpl::layoutUpdated(WebCore::Frame*) const
STDERR: 18  0x45985a7a WebCore::FrameView::layout(bool)
STDERR: 19  0x4597fd92 WebCore::FrameView::layoutTimerFired(WebCore::Timer&lt;WebCore::FrameView&gt;*)
STDERR: 20  0x459a7e97 WebCore::Timer&lt;WebCore::FrameView&gt;::fired()
STDERR: 21  0x449fb32b WebCore::ThreadTimers::sharedTimerFiredInternal()
STDERR: 22  0x449fb0af WebCore::ThreadTimers::sharedTimerFired()</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>783427</commentid>
    <comment_count>1</comment_count>
    <who name="Takashi Sakamoto">tasak</who>
    <bug_when>2012-12-04 23:10:33 -0800</bug_when>
    <thetext>Committed r136644: &lt;http://trac.webkit.org/changeset/136644&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>802748</commentid>
    <comment_count>2</comment_count>
    <who name="Julien Chaffraix">jchaffraix</who>
    <bug_when>2013-01-08 15:43:18 -0800</bug_when>
    <thetext>r139025 / bug 104700 removed the bad call to GridTrackSize::length in RenderGrid::computePreferredLogicalWidths() and the tests don&apos;t ASSERT anymore per http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=fast%2Fcss-grid-layout%2Fgrid-columns-rows-get-set-multiple.html

I am going to re-enable them.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>802840</commentid>
    <comment_count>3</comment_count>
      <attachid>181810</attachid>
    <who name="Julien Chaffraix">jchaffraix</who>
    <bug_when>2013-01-08 17:19:48 -0800</bug_when>
    <thetext>Created attachment 181810
Patch for landing</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>802908</commentid>
    <comment_count>4</comment_count>
      <attachid>181810</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2013-01-08 19:22:49 -0800</bug_when>
    <thetext>Comment on attachment 181810
Patch for landing

Clearing flags on attachment: 181810

Committed r139147: &lt;http://trac.webkit.org/changeset/139147&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>802909</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2013-01-08 19:22:52 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>181810</attachid>
            <date>2013-01-08 17:19:48 -0800</date>
            <delta_ts>2013-01-08 19:22:49 -0800</delta_ts>
            <desc>Patch for landing</desc>
            <filename>bug-104090-20130108171652.patch</filename>
            <type>text/plain</type>
            <size>1721</size>
            <attacher name="Julien Chaffraix">jchaffraix</attacher>
            
              <data encoding="base64">SW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9D
aGFuZ2VMb2cJKHJldmlzaW9uIDEzOTEzNikKKysrIExheW91dFRlc3RzL0NoYW5nZUxvZwkod29y
a2luZyBjb3B5KQpAQCAtMSwzICsxLDE0IEBACisyMDEzLTAxLTA4ICBKdWxpZW4gQ2hhZmZyYWl4
ICA8amNoYWZmcmFpeEB3ZWJraXQub3JnPgorCisgICAgICAgIExheW91dCBUZXN0IGZhc3QvY3Nz
LWdyaWQtbGF5b3V0L2dyaWQtY29sdW1ucy1yb3dzLWdldC1zZXQtbXVsdGlwbGUuaHRtbCwgZ3Jp
ZC1jb2x1bW5zLXJvd3MtZ2V0LXNldC5odG1sIGlzIGZhaWxpbmcKKyAgICAgICAgaHR0cHM6Ly9i
dWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEwNDA5MAorCisgICAgICAgIFVucmV2aWV3
ZWQgZ2FyZGVuaW5nLgorCisgICAgICAgICogcGxhdGZvcm0vY2hyb21pdW0vVGVzdEV4cGVjdGF0
aW9uczoKKyAgICAgICAgSW4gcjEzOTAyNSwgdGhlIG5vdC10b3RhbGx5LU9LIGNhbGwgdG8gR3Jp
ZFRyYWNrU2l6ZTo6bGVuZ3RoKCkgd2FzIHJlbW92ZWQgYW5kIHRodXMgdGhlCisgICAgICAgIHRl
c3RzIGRvbid0IHRyaWdnZXIgYW4gQVNTRVJUIGFueW1vcmUuIFRoaXMgY2hhbmdlIG1lcnJpbHkg
cmUtZW5hYmxlIHRoZW0uCisKIDIwMTMtMDEtMDggIEZpbGlwIFBpemxvICA8ZnBpemxvQGFwcGxl
LmNvbT4KIAogICAgICAgICBERkcgc2hvdWxkbid0IHRyZWF0IHRoZSAndGhpcycgYXJndW1lbnQg
YXMgYmVpbmcgY2FwdHVyZWQgaWYgYSBjb2RlIGJsb2NrIHVzZXMgYXJndW1lbnRzCkluZGV4OiBM
YXlvdXRUZXN0cy9wbGF0Zm9ybS9jaHJvbWl1bS9UZXN0RXhwZWN0YXRpb25zCj09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0K
LS0tIExheW91dFRlc3RzL3BsYXRmb3JtL2Nocm9taXVtL1Rlc3RFeHBlY3RhdGlvbnMJKHJldmlz
aW9uIDEzOTEzNikKKysrIExheW91dFRlc3RzL3BsYXRmb3JtL2Nocm9taXVtL1Rlc3RFeHBlY3Rh
dGlvbnMJKHdvcmtpbmcgY29weSkKQEAgLTQyMzIsMTAgKzQyMzIsNiBAQCB3ZWJraXQub3JnL2Iv
OTk4MDAgZmFzdC9oYXJuZXNzL3BlcmZ0ZXN0CiB3ZWJraXQub3JnL2IvMTA0MDM1IGZhc3QvdGV4
dC9vcmllbnRhdGlvbi1zaWRld2F5cy5odG1sIFsgTWlzc2luZyBdCiB3ZWJraXQub3JnL2IvMTA0
MTcyIGZhc3QvdGV4dC9kZWNvcmF0aW9ucy13aXRoLXRleHQtY29tYmluZS5odG1sIFsgTWlzc2lu
ZyBdCiAKLSMgQ3Jhc2ggc2luY2UgcjEzNjU4OC4KLXdlYmtpdC5vcmcvYi8xMDQwOTAgWyBNYWMg
RGVidWcgXSBmYXN0L2Nzcy1ncmlkLWxheW91dC9ncmlkLWNvbHVtbnMtcm93cy1nZXQtc2V0LW11
bHRpcGxlLmh0bWwgWyBDcmFzaCBdCi13ZWJraXQub3JnL2IvMTA0MDkwIFsgTWFjIERlYnVnIF0g
ZmFzdC9jc3MtZ3JpZC1sYXlvdXQvZ3JpZC1jb2x1bW5zLXJvd3MtZ2V0LXNldC5odG1sIFsgQ3Jh
c2ggXQotCiB3ZWJraXQub3JnL2IvMTA0MDkyIGZhc3QvZnJhbWVzL3BhaW50LWlmcmFtZS1iYWNr
Z3JvdW5kLmh0bWwgWyBGYWlsdXJlIFBhc3MgXQogCiB3ZWJraXQub3JnL2IvMTAzNDg3IFsgTGlu
dXggXSBmYXN0L2Zvcm1zL3BsYWNlaG9sZGVyLXBvc2l0aW9uLmh0bWwgWyBJbWFnZU9ubHlGYWls
dXJlIFBhc3MgXQo=
</data>

          </attachment>
      

    </bug>

</bugzilla>