NEW 180269
[iOS] editing/execCommand tests using 'createLink' or 'unlink' fail 
https://bugs.webkit.org/show_bug.cgi?id=180269
Summary [iOS] editing/execCommand tests using 'createLink' or 'unlink' fail
Ryan Haddad
Reported 2017-12-01 12:38:19 PST
The following tests fail when run on iOS. The diffs show that spaces end up on the wrong side of the link tags. editing/execCommand/createLink.html editing/execCommand/toggle-link-mac.html editing/execCommand/toggle-unlink-mac.html editing/execCommand/unlink.html --- /Volumes/Data/OpenSource/WebKitBuild/Release-iphonesimulator/layout-test-results/editing/execCommand/unlink-expected.txt +++ /Volumes/Data/OpenSource/WebKitBuild/Release-iphonesimulator/layout-test-results/editing/execCommand/unlink-actual.txt @@ -61,6 +61,6 @@ The innerHTML of editable regions after the test: This paragraph should should end up unlinked. -<a href="http://www.apple.com/">The</a> second<a href="http://www.apple.com/"> word in this paragraph should end up being unlinked, everything else should be a link.</a> +<a href="http://www.apple.com/">The </a>second <a href="http://www.apple.com/">word in this paragraph should end up being unlinked, everything else should be a link.</a> This paragraph starts with <a href="http://www.google.com"><i>a</i></a><span id="test3start"> link</span> in the middle. Only the 'a' in the previous sentence should be linked after the test. <p>This <i>editable region</i> contains lists, tables, styled text, and images. Everything in this region that is not selected should be a link, nothing that is selected should be a link.</p> <ul> <li>Item 1</li> <li>Item 2</li> </ul> <table border="1"><tbody><tr><td>1</td><td>2</td><td><span id="test4end"><a href="http://www.google.com/">3</a></span></td></tr></tbody></table> <a href="http://www.google.com/"><br> This <b>line</b> contains <img src="../resources/abe.png"> an image. </a>
Attachments
Archive of test results (292.50 KB, application/x-gzip)
2017-12-01 12:38 PST, Ryan Haddad 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Ryan Haddad
Comment 1 2017-12-01 12:38:31 PST
Created attachment 328134 [details] Archive of test results
Note You need to log in before you can comment on or make changes to this bug.