| Summary: | LayoutTests\editing\deleting\4875189.html should be a dumpAsText test instead of a pixel test | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> | ||||
| Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Normal | ||||||
| Priority: | P2 | ||||||
| Version: | 528+ (Nightly build) | ||||||
| Hardware: | PC | ||||||
| OS: | OS X 10.5 | ||||||
| Attachments: |
|
||||||
|
Description
Ryosuke Niwa
2009-06-09 17:32:50 PDT
Created attachment 31169 [details]
This patch converts LayoutTests\editing\deleting\4875189.html to a dumpAsText test.
Comment on attachment 31169 [details] This patch converts LayoutTests\editing\deleting\4875189.html to a dumpAsText test. > <p>This test deletes the contents of a list, then it performs another delete, which should remove the empty list item. There should be two paragraphs below.</p> > <div id="div" contentEditable="true">Paragraph One.<ul><li id="li">bar</li></ul>Paragraph Two.</div> > +<ol id="console"></ol> Why is the console a list? Comment on attachment 31169 [details]
This patch converts LayoutTests\editing\deleting\4875189.html to a dumpAsText test.
Looks fine.
>Why is the console a list?
I'm not sure the original file had console as a list. Maybe it's to do with reproducing the error. It's maybe that making it div would affect the result. e.g. it only occurs when there is exactly one div, or when it's the last div item in the page.
Committed as http://trac.webkit.org/changeset/44621 |