open the attached test file and observe the rendering result. related chromium bug: http://code.google.com/p/chromium/issues/detail?id=70081
Created attachment 79531 [details] test case
Created attachment 79671 [details] patch w/ layout test
Comment on attachment 79671 [details] patch w/ layout test The code change seems good, but BidiResolver is not DOM-related, so it is strange that the test goes into fast/dom. Is there a way to test this without going through complicated editing operations. If the bug affects ordering of characters, it should be testable using Range.getClientRects() and comparing results. This would also allow the test to dump as text.
Committed r76319: <http://trac.webkit.org/changeset/76319>
(In reply to comment #4) > Committed r76319: <http://trac.webkit.org/changeset/76319> Looks like this test has been failing on the Windows 7 Release bot since it was introduced: http://build.webkit.org/results/Windows%207%20Release%20(Tests)/r76319%20(8469)/fast/dom/52776-pretty-diff.html I will be landing these results as expected results for Windows 7. Are these results correct, or should I also file a bug?
Landed the Windows expected results in http://trac.webkit.org/changeset/76349
(In reply to comment #6) > Landed the Windows expected results in http://trac.webkit.org/changeset/76349 Thanks for rebaseline win7 results! The results looks correct. The most important part is the Success message printed at the end of the file. Others I guess are platform difference.
(In reply to comment #7) > (In reply to comment #6) > > Landed the Windows expected results in http://trac.webkit.org/changeset/76349 > > Thanks for rebaseline win7 results! > The results looks correct. The most important part is the Success message printed at the end of the file. > Others I guess are platform difference. Looks like Windows XP also needs platform-specific results: http://build.webkit.org/results/Windows%20XP%20Debug%20(Tests)/r78339%20(25164)/fast/dom/52776-pretty-diff.html I am going to land these results to get the bots green. Do they look correct? (it appears that the Success message is at the end of the file since that does not show up in the diff).
Added the Windows-XP results in http://trac.webkit.org/changeset/78344
(In reply to comment #9) > Added the Windows-XP results in http://trac.webkit.org/changeset/78344 Thanks again for rebaseline! Sorry that I missed it.