Summary: | Convert editing/selection/5136696.html to dumpAsText | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Martin Robinson <mrobinson> | ||||||
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | ||||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | PC | ||||||||
OS: | Linux | ||||||||
Attachments: |
|
Description
Martin Robinson
2010-01-28 21:31:48 PST
Created attachment 47673 [details]
Convert the test
I've attached a patch for this conversion.
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.
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. 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.
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-.
Committed r55852: <http://trac.webkit.org/changeset/55852> |