[webkit-dev] Introducing dumpAsMarkup

[Ojan Vafai]

I like the idea of standardizing the dump output and the dumping code for
that matter. If we can resolve the below, then we can make sure all the
dumping logic in dom2string gets moved into dump-as-markup and standardize
on that. As best I can tell, there's only 2 uses of dom2string.

There is another DOM dump format which the html5lib tests use:
>
> http://trac.webkit.org/browser/trunk/LayoutTests/html5lib/runner-expected-html5.txt
> Note, it's a DOM Dump, not a Markup dump, but it serves a similar
> purpose.
>

I'm not really sure what distinction you're drawing here. They both just
dump the DOM. When this came up before, people preferred the name
dump-as-markup to dum-as-dom.

I I think I like the html5lib format better for a few reasons:
> 2. The DOM tree indent is nice.
>

What does the indenting do with linebreaks? For example, "foo\nbar\nbaz". In
dump-as-markup, that gets printed as:
<#text>foo
bar
baz</text>

I guess I'd be OK with indenting, but having anything after a newline not
indent.

3. <#text> is takes less space.
>

Meh. I find the quotes confusing, e.g. if there are quotes in the text node:
"And then I said, "wot?""

But I guess that's not too bad and it is a bit easier on the eyes overall.
I'm ok with changing this if that's what others think is better.

I suggest adding # signs to the selection pseudo-elements,
> just to avoid the tiny risk of conflicting with an actual element in a
> test. Also, perhaps it would be better to show tag names in lowercase -
> that can be achieved by using localName instead of nodeName or tagName.
>

Both of these make sense.

Filed https://bugs.webkit.org/show_bug.cgi?id=43015.

Ojan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20100726/e3bf3e90/attachment.html>