RESOLVED FIXED 73510
New tests introduced in r101575 fail on all platform 
https://bugs.webkit.org/show_bug.cgi?id=73510
Summary New tests introduced in r101575 fail on all platform
Hayato Ito
Reported 2011-11-30 18:49:27 PST
The following layout test is failing on chromium port. editing/deleting/delete-and-cleanup.html editing/pasteboard/paste-without-nesting.html Probable cause: https://trac.webkit.org/changeset/101575
Attachments
Add attachment proposed patch, testcase, etc.
Csaba Osztrogonác
Comment 1 2011-11-30 23:25:17 PST
These new tests introduced in r101575 and fail on all platform: SL, GTK, Qt and Chromium. --- /Volumes/Data/slave/snowleopard-intel-release-tests/build/layout-test-results/editing/pasteboard/paste-without-nesting-expected.txt +++ /Volumes/Data/slave/snowleopard-intel-release-tests/build/layout-test-results/editing/pasteboard/paste-without-nesting-actual.txt @@ -1,11 +1,11 @@ This test checks that copy a selection and paste over the same selection does not create nested divs. -PASS confimedMarkup is 'Hello' -PASS confimedMarkup is 'Hello<br>world' -PASS confimedMarkup is '<div>Hello</div><div>world</div>' -PASS confimedMarkup is '<div>Hello</div><div>world</div>' -PASS confimedMarkup is '<div><b><i>Hello</i></b></div><div><b><i>world</i></b></div>' -PASS confimedMarkup is '<div style="text-align: center;"><div><font color="#ff0000"><b><i>hello</i></b></font></div><div><font color="#ff0000"><b><i>world</i></b></font></div></div>' +FAIL confimedMarkup should be Hello. Threw exception ReferenceError: Can't find variable: confimedMarkup +FAIL confimedMarkup should be Hello<br>world. Threw exception ReferenceError: Can't find variable: confimedMarkup +FAIL confimedMarkup should be <div>Hello</div><div>world</div>. Threw exception ReferenceError: Can't find variable: confimedMarkup +FAIL confimedMarkup should be <div>Hello</div><div>world</div>. Threw exception ReferenceError: Can't find variable: confimedMarkup +FAIL confimedMarkup should be <div><b><i>Hello</i></b></div><div><b><i>world</i></b></div>. Threw exception ReferenceError: Can't find variable: confimedMarkup +FAIL confimedMarkup should be <div style="text-align: center;"><div><font color="#ff0000"><b><i>hello</i></b></font></div><div><font color="#ff0000"><b><i>world</i></b></font></div></div>. Threw exception ReferenceError: Can't find variable: confimedMarkup PASS successfullyParsed is true TEST COMPLETE --- /Volumes/Data/slave/snowleopard-intel-release-tests/build/layout-test-results/editing/deleting/delete-and-cleanup-expected.txt +++ /Volumes/Data/slave/snowleopard-intel-release-tests/build/layout-test-results/editing/deleting/delete-and-cleanup-actual.txt @@ -1,14 +1,14 @@ This test checks that deletion does not leave unnecessary nested divs. -PASS confimedMarkup is '<br>' -PASS confimedMarkup is '<br>' -PASS confimedMarkup is '<div id="mydiv"><br></div>' -PASS confimedMarkup is '<br>' -PASS confimedMarkup is '<b><i><br></i></b>' -PASS confimedMarkup is '<b><div style="border: solid red"><i><br></i></div></b>' -PASS confimedMarkup is '<br>' -PASS confimedMarkup is '<br>' -PASS confimedMarkup is '<p><br></p>' +FAIL confimedMarkup should be <br>. Threw exception ReferenceError: Can't find variable: confimedMarkup +FAIL confimedMarkup should be <br>. Threw exception ReferenceError: Can't find variable: confimedMarkup +FAIL confimedMarkup should be <div id="mydiv"><br></div>. Threw exception ReferenceError: Can't find variable: confimedMarkup +FAIL confimedMarkup should be <br>. Threw exception ReferenceError: Can't find variable: confimedMarkup +FAIL confimedMarkup should be <b><i><br></i></b>. Threw exception ReferenceError: Can't find variable: confimedMarkup +FAIL confimedMarkup should be <b><div style="border: solid red"><i><br></i></div></b>. Threw exception ReferenceError: Can't find variable: confimedMarkup +FAIL confimedMarkup should be <br>. Threw exception ReferenceError: Can't find variable: confimedMarkup +FAIL confimedMarkup should be <br>. Threw exception ReferenceError: Can't find variable: confimedMarkup +FAIL confimedMarkup should be <p><br></p>. Threw exception ReferenceError: Can't find variable: confimedMarkup PASS successfullyParsed is true TEST COMPLETE
Ryosuke Niwa
Comment 2 2011-11-30 23:39:17 PST
Fixed the typo in http://trac.webkit.org/changeset/101622. Hopefully this would make the bots happy.
Ryosuke Niwa
Comment 3 2011-12-01 02:32:39 PST
Expected result fixed in http://trac.webkit.org/changeset/101629.
Enrica Casucci
Comment 4 2011-12-01 09:31:51 PST
(In reply to comment #3) > Expected result fixed in http://trac.webkit.org/changeset/101629. My apologies for the mistake.
Enrica Casucci
Comment 5 2011-12-01 09:32:21 PST
(In reply to comment #2) > Fixed the typo in http://trac.webkit.org/changeset/101622. Hopefully this would make the bots happy. Thanks for fixing it!
Vsevolod Vlasov
Comment 6 2011-12-02 07:14:03 PST
(In reply to comment #5) > (In reply to comment #2) > > Fixed the typo in http://trac.webkit.org/changeset/101622. Hopefully this would make the bots happy. > > Thanks for fixing it! Shouldn't this text be unfixed in test_expectations? Looks like the bots are happy now!
Vsevolod Vlasov
Comment 7 2011-12-02 07:14:32 PST
I meant "these tests"
Note You need to log in before you can comment on or make changes to this bug.