Bug 89436 – [Qt] New fast/text/line-breaks-after-ideographic-comma-or-full-stop-2.html introduced in r120624 fails on ICU platforms

• Component: Tools / Tests
• Status: RESOLVED INVALID

[Csaba Osztrogonác]

fast/text/line-breaks-after-ideographic-comma-or-full-stop-2.html 
introduced in http://trac.webkit.org/changeset/120624 and fails
on Qt5 which is ICU platform. (But pass on non-ICU Qt4)


--- /home/webkitbuildbot/slaves/release32bit-qt5/buildslave/qt-linux-32-release-qt5/build/layout-test-results/fast/text/line-breaks-after-ideographic-comma-or-full-stop-2-expected.txt 
+++ /home/webkitbuildbot/slaves/release32bit-qt5/buildslave/qt-linux-32-release-qt5/build/layout-test-results/fast/text/line-breaks-after-ideographic-comma-or-full-stop-2-actual.txt 
@@ -1,4 +1,4 @@
 This tests that a line break is prohibited between an ideographic comma and a right corner bracket.
 
 ăăăă
-PASS
+FAIL

[Csaba Osztrogonác]

I skipped it on Qt5 which is ICU platform - http://trac.webkit.org/changeset/120682/trunk/LayoutTests/platform/qt-5.0/Skipped

[Marcelo Lira]

Created attachment 165145 [details]
Patch

[Alexey Proskuryakov]

Comment on attachment 165145 [details]
Patch

EWS says that the patch doesn't apply, please update your checkout, and make a new one.

[Csaba Osztrogonác]

I tested it and it still fails on WK1 and WK2 too.
I don't know how can you make it pass. :)

[Marcelo Lira]

(In reply to comment #4)
> I tested it and it still fails on WK1 and WK2 too.
> I don't know how can you make it pass. :)

Sorry, it was a mistake. ><

[Jocelyn Turcotte]

=== Bulk closing of Qt bugs ===

If you believe that this bug report is still relevant for a non-Qt port of webkit.org, please re-open it and remove [Qt] from the summary.

If you believe that this is still an important QtWebKit bug, please fill a new report at https://bugreports.qt-project.org and add a link to this issue. See http://qt-project.org/wiki/ReportingBugsInQt for additional guidelines.