<div dir="ltr"><div class="gmail_extra"><div><div>Since I don&#39;t understand table layout well, I&#39;m sorry if I said wrong.</div></div><div><br></div><div>RenderTable::layout() calls RenderTableSection::layout() at [1].</div>

<div>RenderTableSection::layout() calls RenderTableRow::layout() at [2].</div><div>RenderTableRow::layout() clears its setNeedsLayout flag at [3].</div><div>Therefore, I think the flag is cleared.</div><div><br></div><div>

[1]: <a href="http://trac.webkit.org/browser/trunk/Source/WebCore/rendering/RenderTable.cpp#L465">http://trac.webkit.org/browser/trunk/Source/WebCore/rendering/RenderTable.cpp#L465</a><br></div>
[2]: <a href="http://trac.webkit.org/browser/trunk/Source/WebCore/rendering/RenderTableSection.cpp#L420">http://trac.webkit.org/browser/trunk/Source/WebCore/rendering/RenderTableSection.cpp#L420</a></div><div class="gmail_extra">

[3]: <a href="http://trac.webkit.org/browser/trunk/Source/WebCore/rendering/RenderTableRow.cpp#L192">http://trac.webkit.org/browser/trunk/Source/WebCore/rendering/RenderTableRow.cpp#L192</a><br><br><div class="gmail_quote">

On Thu, Mar 13, 2014 at 11:39 PM, Mayur K <span dir="ltr">&lt;<a href="mailto:eminemence@gmail.com" target="_blank">eminemence@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<div dir="ltr">Hi,<div><br></div><div>I am checking the layout and rendering of the table element in webkit.</div><div>In the process I figured out that the link of renderers is as follows:</div><div>RenderTable &lt;has&gt; RenderTableSection &lt;has&gt; RenderTableRow &lt;has&gt; RenderTableCell</div>


<div>The layout of the table rows is done in RenderTableSection::layoutRows.</div><div>I did not see any code to specifically set RenderTableRow::setNeedsLayout(false) at the end of the function.</div><div>While there is a call to RenderTableCell::layout , which will then clear the layout flag for RenderTableCell.</div>


<div>So is a RenderTableRow::setNeedsLayout(false) needed at the end of the layout of each row in RenderTableSection::layoutRows?</div><div>If not then where is the RenderTableRow layout flag cleared?</div><div><br></div>


<div>Regards,<br></div><div>Mayur Kankanwadi.<span class=""><font color="#888888"><br clear="all"><div><br></div>-- <br>Symbiangeek,Codekata &amp; Webkitwiki all in one - <a href="http://flaminghorns.com" target="_blank">http://flaminghorns.com</a><br>


</font></span></div></div>
<br>_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br>
<a href="https://lists.webkit.org/mailman/listinfo/webkit-dev" target="_blank">https://lists.webkit.org/mailman/listinfo/webkit-dev</a><br>
<br></blockquote></div><br></div></div>

<br>
.
<pre>________________________________________
The contents of this e-mail message and any attachments are confidential and are intended solely for the addressee. The information may also be legally privileged. 
This transmission is sent in trust, and the sole purpose of delivery to the intended recipient. If you have received this transmission in error, any use, reproduction or dissemination of this transmission is strictly prohibited. 
If you are not the intended recipient, please immediately notify the sender by reply e-mailer and delete this message and its attachments, if any.
Thank you for your cooperation.
________________________________________