Bug 89436

Summary: [Qt] New fast/text/line-breaks-after-ideographic-comma-or-full-stop-2.html introduced in r120624 fails on ICU platforms
Product: WebKit Reporter: Csaba Osztrogonác <ossy>
Component: Tools / TestsAssignee: Marcelo Lira <marcelo.lira>
Status: RESOLVED INVALID    
Severity: Normal CC: ap, marcelo.lira, mitz, ossy
Priority: P2 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 87008, 87041    
Attachments:
Description Flags
Patch ap: review-, ap: commit-queue-

Description Csaba Osztrogonác 2012-06-18 23:48:10 PDT
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
Comment 1 Csaba Osztrogonác 2012-06-18 23:54:49 PDT
I skipped it on Qt5 which is ICU platform - http://trac.webkit.org/changeset/120682/trunk/LayoutTests/platform/qt-5.0/Skipped
Comment 2 Marcelo Lira 2012-09-21 09:30:05 PDT
Created attachment 165145 [details]
Patch
Comment 3 Alexey Proskuryakov 2012-09-21 09:52:13 PDT
Comment on attachment 165145 [details]
Patch

EWS says that the patch doesn't apply, please update your checkout, and make a new one.
Comment 4 Csaba Osztrogonác 2012-09-22 00:31:21 PDT
I tested it and it still fails on WK1 and WK2 too.
I don't know how can you make it pass. :)
Comment 5 Marcelo Lira 2012-09-25 14:32:54 PDT
(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. ><
Comment 6 Jocelyn Turcotte 2014-02-03 03:21:18 PST
=== 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.