RESOLVED INVALID 89426
[Qt] REGRESSION(r120624): It made fast/text/line-breaks-after-ideographic-comma-or-full-stop.html fail on non-ICU platforms 
https://bugs.webkit.org/show_bug.cgi?id=89426
Summary [Qt] REGRESSION(r120624): It made fast/text/line-breaks-after-ideographic-com...
Csaba Osztrogonác
Reported 2012-06-18 23:05:34 PDT
http://trac.webkit.org/changeset/120624 made fast/text/line-breaks-after-ideographic-comma-or-full-stop.html fail on Qt: --- /ramdisk/qt-linux-64-release/build/layout-test-results/fast/text/line-breaks-after-ideographic-comma-or-full-stop-expected.txt +++ /ramdisk/qt-linux-64-release/build/layout-test-results/fast/text/line-breaks-after-ideographic-comma-or-full-stop-actual.txt @@ -15,4 +15,4 @@ ăătexttexttexttext -PASS +FAIL Could you check it, please?
Attachments
Add attachment proposed patch, testcase, etc.
Csaba Osztrogonác
Comment 1 2012-06-18 23:10:20 PDT
I skipped it to paint the bot green - http://trac.webkit.org/changeset/120677/trunk/LayoutTests/platform/qt/Skipped Please unskip it with the proper fix. Thanks.
mitz
Comment 2 2012-06-18 23:13:31 PDT
(In reply to comment #0) > http://trac.webkit.org/changeset/120624 made fast/text/line-breaks-after-ideographic-comma-or-full-stop.html fail on Qt: > > > --- /ramdisk/qt-linux-64-release/build/layout-test-results/fast/text/line-breaks-after-ideographic-comma-or-full-stop-expected.txt > +++ /ramdisk/qt-linux-64-release/build/layout-test-results/fast/text/line-breaks-after-ideographic-comma-or-full-stop-actual.txt > @@ -15,4 +15,4 @@ > > ăătexttexttexttext > > -PASS > +FAIL > > > Could you check it, please? I guess the configuration failing this new test isn’t using ICU at all.
Csaba Osztrogonác
Comment 3 2012-06-18 23:21:59 PDT
Oh it is a new test, and not regression.
Csaba Osztrogonác
Comment 4 2012-06-18 23:44:55 PDT
You confused me a little bit. :) There are two different bugs here: - r120624 made fast/text/line-breaks-after-ideographic-comma-or-full-stop.html fail (it is an old test) on Qt 4.8 which is non-ICU platform. - The new fast/text/line-breaks-after-ideographic-comma-or-full-stop-2.html fails on Qt 5 whihc is ICU platform. (I'm going to file a new bug for it.)
Csaba Osztrogonác
Comment 5 2012-06-18 23:54:11 PDT
I skipped it on the non-ICU Qt4 platform - http://trac.webkit.org/changeset/120682/trunk/LayoutTests/platform/qt-4.8/Skipped The new bug on the other fail is https://bugs.webkit.org/show_bug.cgi?id=89436
Note You need to log in before you can comment on or make changes to this bug.