Bug 89436 - [Qt] New fast/text/line-breaks-after-ideographic-comma-or-full-stop-2.html introduced in r120624 fails on ICU platforms
Summary: [Qt] New fast/text/line-breaks-after-ideographic-comma-or-full-stop-2.html in...
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Marcelo Lira
URL:
Keywords: Qt, QtTriaged
Depends on:
Blocks: 87008 87041
  Show dependency tree / graph
 
Reported: 2012-06-18 23:48 PDT by Csaba Osztrogonác
Modified: 2014-02-03 03:21 PST (History)
4 users (show)

See Also:

Attachments
Patch (1.63 KB, patch)
2012-09-21 09:30 PDT, Marcelo Lira 		ap: review-
ap: commit-queue-
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.