Bug 62823 - convert editing/deleting/delete-3608430-fix.html to DumpAsTextEditingTest
Summary: convert editing/deleting/delete-3608430-fix.html to DumpAsTextEditingTest
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Enhancement
Assignee: Nobody
URL: http://trac.webkit.org/browser/trunk/...
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-16 14:35 PDT by Wyatt Carss
Modified: 2011-06-21 14:18 PDT (History)
6 users (show)

See Also:


Attachments
possible patch (87.15 KB, patch)
2011-06-16 14:55 PDT, Wyatt Carss
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Wyatt Carss 2011-06-16 14:35:33 PDT
If possible, I'd like to find out what this test is actually testing. I'm reviewing possible candidates for conversion to text, and this one's very opaque, but doesn't appear to need to be rendered.

It also happens to have a broken image reference and a superfluous quote floating in it - after fixing them, the test still passed. My hypothesis is that it's testing that deleting an image will remove the empty <i></i> before it, so with a certain number of deletion operations we'll end up with a certain success text. In that case, the progression of operations and the success text matter more than the pixels, so DumpAsTextEditingTest seems like a great choice. But I'm concerned that there's something I'm missing here -- also, I'd love to add a self-contained description to the test for the future.
Comment 1 Wyatt Carss 2011-06-16 14:55:02 PDT
Created attachment 97508 [details]
possible patch
Comment 2 Wyatt Carss 2011-06-16 14:56:54 PDT
Comment on attachment 97508 [details]
possible patch

If this description is correct, and there aren't objections to conversion, I submit this patch. Otherwise, I'll make corrections.
Comment 3 Ryosuke Niwa 2011-06-21 13:58:53 PDT
Comment on attachment 97508 [details]
possible patch

View in context: https://bugs.webkit.org/attachment.cgi?id=97508&action=review

> LayoutTests/editing/deleting/delete-3608430-fix.html:37
> -runEditingTest();
> +runDumpAsTextEditingTest();

We might need to wait until page load event because the image may not have been loaded at this point.
Comment 4 WebKit Review Bot 2011-06-21 14:18:37 PDT
Comment on attachment 97508 [details]
possible patch

Clearing flags on attachment: 97508

Committed r89382: <http://trac.webkit.org/changeset/89382>
Comment 5 WebKit Review Bot 2011-06-21 14:18:42 PDT
All reviewed patches have been landed.  Closing bug.