Bug 34306 – Convert editing/selection/5136696.html to dumpAsText

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Martin Robinson]

The test is currently skipped for GTK+ because there is no baseline, so I'll just convert it to a dumpAsText() test.

[Martin Robinson]

Created attachment 47673 [details]
Convert the test

I've attached a patch for this conversion.

[David Levin]

Comment on attachment 47673 [details]
Convert the test

I'm not too familiar with this test, but I tried to ensure that the final result would test the same thing that the test was verifying before.

To me it looks like converting this test to dumpAsText in this manner destroys what it was trying to do. It looks like with the patch, this test no longer verifies that text is selected anymore nor does it verify what has focus.

[Martin Robinson]

It verifies that the selection text is equal to "This test should be selected, but this frame shouldn't be focused." by appending the current selection to the body.

I'm not sure exactly how the original test verifies that the text field has focus (I don't see it in the DRT output), but I can make sure that this conversion does test that.

[Martin Robinson]

Created attachment 47757 [details]
Patch with fixes

Looks like I had ordered the test statements incorrectly. I've uploaded a patch that verifies that the text field has focus and also that the iframe's selection has not been collapsed.

[Adam Barth]

Comment on attachment 47757 [details]
Patch with fixes

Looks like you've addressed David's concerns.  David, if you still have concerns, please feel free to convert my r+ into an r-.

[Martin Robinson]

Committed r55852: <http://trac.webkit.org/changeset/55852>