<div>Hi all,</div><div><br></div><div>Since this would require a lot of rebaselines anyways, can we also add some enhancements to DRT?</div><div><br></div><div>I propose to an option to dump as text with images.  In many editing tests, we don&#39;t need render tree dumps because we care more about how DOM looks like before and after editing operations.  However, we also need to verify that selection and caret are rendered properly on multiple occasions, and the only way to do this right now is to use a pixel test.  But even then, dumping render tree is not helpful and almost harmful because it doesn&#39;t tell us how selection / caret are rendered (this can never be tested by comparing text) and hides some important information about DOM and makes us rebaseline tests whenever there&#39;s slight change (that we don&#39;t care) in the render tree.</div>

<div><br></div><div>James (jamesr) and I talked about this on IRC, and he said this feature will also be useful for repaint tests and canvas tests.  We can implement this feature by adding dumpAsTextWithImage to layoutTestController, which forces DRT to dump as text but also outputs the png image.  For repainting tests, we can also add dumpRepaintRects to output more information about painting.</div>

<div><br></div><div>- Ryosuke</div><div><br><div class="gmail_quote">On Mon, Dec 6, 2010 at 12:30 PM, David Hyatt <span dir="ltr">&lt;<a href="mailto:hyatt@apple.com">hyatt@apple.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

RenderTreeAsTetxt has a large number of hacks in it that have been put in over time to keep the dumps from changing too dramatically.  Some of these include:<br>
<br>
(1) Table cells dump incorrect dimensions and positions.<br>
(2) Text nodes dump an incorrect bounding box position.<br>
(3) RenderInlines don&#39;t dump their position at all.<br>
(4) The root layer incorporates overflow when it shouldn&#39;t.<br>
(5) The root element has a RenderLayer when there has been no need for it to have a RenderLayer for years.  It only has one in order to not change all the layout tests.<br>
<br>
In addition there is information not being captured that would be useful to include in the dump.  Examples of this include:<br>
<br>
(1) Layout and visual overflow for elements.  Right now we&#39;re completely dependent on repaint tests to catch changes in visual overflow.<br>
(2) scrollOrigin for the ScrollView and for overflow sections.<br>
(3) intrinsic padding of table cells.<br>
(4) Transforms and relative positioning offsets<br>
<br>
I&#39;m sure people may have other ideas about things to include in the geometry dumps that aren&#39;t there right now, so send me your suggestions.<br>
<br>
What I&#39;d like to do is have a rebaselining day (probably after the holidays in January) where we just shut the tree down and all the ports rebaseline to the new format (with the hacks removed and any changes we want to make added).<br>


<br>
What do people think of this idea?  How can we make sure that a rebaselining like this goes smoothly?<br>
<br>
dave<br>
(<a href="mailto:hyatt@apple.com">hyatt@apple.com</a>)<br>
<br>
<br>
<br>
_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br>
<a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
</blockquote></div><br></div>