Bug 42608 – dumpAsMarkup test conversion: create-list-from-range-selection.html and insert-list-empty-div.html

• Status: RESOLVED
• Resolution: FIXED
• Component: HTML Editing

[Ryosuke Niwa]

create-list-from-range-selection.html, create-list-with-hr.html, and insert-list-empty-div.html currently uses render test but they should all be using dumpAsMarkup test since what we need to know is the final DOM tree and the selection.

[Ryosuke Niwa]

Created attachment 62019 [details]
converts the tests

[Ryosuke Niwa]

Will not convert create-list-with-hr.html since it demonstrates a rendering bug.

[Ojan Vafai]

Comment on attachment 62019 [details]
converts the tests

> Index: LayoutTests/resources/dump-as-markup.js
> ===================================================================
> --- LayoutTests/resources/dump-as-markup.js	(revision 63331)
> +++ LayoutTests/resources/dump-as-markup.js	(working copy)
> @@ -17,7 +17,7 @@ Markup.dump = function(opt_node)

> +Markup.setNode = function(node)

Can we call this setNodeToDump so it's more clear what it does? 

> +{
> +  if (typeof node == "string")
> +    node = document.getElementById(node);
> +  if (node instanceof Node)

I don't think you need this if-check. If node is not a Node here, we should error out.

> +    Markup.node = node

This should be private, i.e., Markup._node.

[Ryosuke Niwa]

Created attachment 62027 [details]
removed create-list-with-hr.html and fixed per ojan's comments

[Ojan Vafai]

Comment on attachment 62027 [details]
removed create-list-with-hr.html and fixed per ojan's comments

Wow. It's so much easier to read these expected results!

[WebKit Commit Bot]

Comment on attachment 62027 [details]
removed create-list-with-hr.html and fixed per ojan's comments

Rejecting patch 62027 from commit-queue.

Failed to run "[u'/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', u'--reviewer', u'Ojan Vafai', u'--force']" exit_code: 1
Last 500 characters of output:
ommand/create-list-from-range-selection-expected.txt'
patching file LayoutTests/platform/qt/editing/execCommand/insert-list-empty-div-expected.txt
rm 'LayoutTests/platform/qt/editing/execCommand/insert-list-empty-div-expected.txt'
patching file LayoutTests/resources/dump-as-markup.js
Hunk #2 FAILED at 31.
Hunk #3 succeeded at 65 with fuzz 1 (offset 10 lines).
Hunk #4 succeeded at 237 (offset 12 lines).
1 out of 4 hunks FAILED -- saving rejects to file LayoutTests/resources/dump-as-markup.js.rej

Full output: http://queues.webkit.org/results/3395665

[Ryosuke Niwa]

Landed as http://trac.webkit.org/changeset/63873.