Bug 61618 - Convert smart-paste-004.html through smart-paste-006.html to dumpAsText tests
Summary: Convert smart-paste-004.html through smart-paste-006.html to dumpAsText tests
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ryosuke Niwa
URL:
Keywords:
Depends on:
Blocks: 34564
  Show dependency treegraph
 
Reported: 2011-05-27 01:44 PDT by Ryosuke Niwa
Modified: 2011-05-27 04:44 PDT (History)
6 users (show)

See Also:


Attachments
converts the tests (74.73 KB, patch)
2011-05-27 01:46 PDT, Ryosuke Niwa
no flags Details | Formatted Diff | Diff
fixed this this (75.08 KB, patch)
2011-05-27 02:01 PDT, Ryosuke Niwa
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ryosuke Niwa 2011-05-27 01:44:14 PDT
The following tests should be converted to dumpAsText tests to that the outputs can easily be understood.
editing/pasteboard/smart-paste-004.html
editing/pasteboard/smart-paste-005.html
editing/pasteboard/smart-paste-006.html
Comment 1 Ryosuke Niwa 2011-05-27 01:46:16 PDT
Created attachment 95142 [details]
converts the tests
Comment 2 Kent Tamura 2011-05-27 01:49:24 PDT
Comment on attachment 95142 [details]
converts the tests

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

> LayoutTests/editing/pasteboard/smart-paste-004-expected.txt:18
> +A space should be added after the pasted word before the other preexisting word. It should like this this: 

Extra 'this'?
Comment 3 Ryosuke Niwa 2011-05-27 02:01:28 PDT
Created attachment 95148 [details]
fixed this this
Comment 4 Ryosuke Niwa 2011-05-27 02:07:56 PDT
Thanks for the reviews!
Comment 5 WebKit Commit Bot 2011-05-27 04:44:21 PDT
Comment on attachment 95148 [details]
fixed this this

Clearing flags on attachment: 95148

Committed r87496: <http://trac.webkit.org/changeset/87496>
Comment 6 WebKit Commit Bot 2011-05-27 04:44:26 PDT
All reviewed patches have been landed.  Closing bug.