RESOLVED FIXED 52883
[Qt] editing/selection/rtl-move-selection-right-left.html fails 
https://bugs.webkit.org/show_bug.cgi?id=52883
Summary [Qt] editing/selection/rtl-move-selection-right-left.html fails
Csaba Osztrogonác
Reported 2011-01-21 04:11:38 PST
editing/selection/rtl-move-selection-right-left.html introduced in http://trac.webkit.org/changeset/76312 and fails: --- /home/webkitbuildbot/slaves/release32bit/buildslave/qt-linux-release/build/layout-test-results/editing/selection/rtl-move-selection-right-left-expected.txt 2011-01-21 03:43:00.092591397 -0800 +++ /home/webkitbuildbot/slaves/release32bit/buildslave/qt-linux-release/build/layout-test-results/editing/selection/rtl-move-selection-right-left-actual.txt 2011-01-21 03:43:00.092591397 -0800 @@ -5,11 +5,11 @@ PASS Move selection right PASS Move selection left -PASS Extend dragged selection right (Mac) +FAIL Extend dragged selection right (Mac): Selection start is 9 but should be 10 PASS Extend dragged selection left (Mac) -PASS Extend dragged selection right (Win) +FAIL Extend dragged selection right (Win): Selection start is 9 but should be 10 PASS Extend dragged selection left (Win) -PASS Extend dragged selection right (Unix) +FAIL Extend dragged selection right (Unix): Selection start is 9 but should be 10 PASS Extend dragged selection left (Unix) PASS successfullyParsed is true The runtime of this test is 25 seconds, it seems as if a timeout expired.
Attachments
Add attachment proposed patch, testcase, etc.
Csaba Osztrogonác
Comment 1 2011-01-21 04:17:06 PST
Added to the skipped list: http://trac.webkit.org/changeset/76343
Yael
Comment 2 2011-01-25 15:30:06 PST
Looks like QFontMetrics::width() is not returning the right size for Hebrew characters. It returns 4 as the size of each glyph, which is obviously wrong.
Yael
Comment 3 2011-01-25 17:22:28 PST
(In reply to comment #2) > Looks like QFontMetrics::width() is not returning the right size for Hebrew characters. It returns 4 as the size of each glyph, which is obviously wrong. And this is happening only with DumpRenderTree. In QtTestBrowser, the width is correct. Both of them use font family "Times" for the Hebrew text.
Yael
Comment 4 2011-01-26 08:31:41 PST
I removed the function DumpRenderTree::initializeFonts() from DumpRenderTreeQt.cpp, and this test passes. it is now clear to me that this is a font issue.
Ádám Kallai
Comment 5 2012-10-03 02:08:04 PDT
Now this test is working on Qt. I'm going to removed from the Qt Test Expectations.
Ádám Kallai
Comment 6 2012-10-04 09:00:15 PDT
I skipped it: http://trac.webkit.org/changeset/130385 Closing bug.
Note You need to log in before you can comment on or make changes to this bug.