Bug 66431 - [Qt] editing/selection/collapse-selection-in-bidi.html fails
Summary: [Qt] editing/selection/collapse-selection-in-bidi.html fails
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: Nobody
URL:
Keywords: LayoutTestFailure, Qt, QtTriaged
Depends on: 76821
Blocks: 64626
  Show dependency treegraph
 
Reported: 2011-08-17 16:46 PDT by Balazs Kelemen
Modified: 2014-02-03 03:18 PST (History)
10 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Balazs Kelemen 2011-08-17 16:46:03 PDT
... since added
Comment 1 Csaba Osztrogonác 2011-08-18 02:45:45 PDT
Introduced in https://trac.webkit.org/changeset/93221 and skipped by https://trac.webkit.org/changeset/93283

--- /ramdisk/qt-linux-release/build/layout-test-results/editing/selection/collapse-selection-in-bidi-expected.txt 
+++ /ramdisk/qt-linux-release/build/layout-test-results/editing/selection/collapse-selection-in-bidi-actual.txt 
@@ -1,9 +1,9 @@
 This tests collapsing directionless selection on text in a block with the opposite text direction. To manually test, on Mac, select text below by mouse and use arrow keys (without pressing shift) to collapse the selection. The selection should collapse to the left if you pressed the left arrow key and to the right if you pressed the right arrow key.
 
-PASS Selection is [anchorNode: [object Text](×××××) anchorOffset: 3 focusNode: [object Text](×××××) focusOffset: 3 isCollapsed: true]
-PASS Selection is [anchorNode: [object Text](×××××) anchorOffset: 1 focusNode: [object Text](×××××) focusOffset: 1 isCollapsed: true]
-PASS Selection is [anchorNode: [object Text](hello) anchorOffset: 1 focusNode: [object Text](hello) focusOffset: 1 isCollapsed: true]
-PASS Selection is [anchorNode: [object Text](hello) anchorOffset: 4 focusNode: [object Text](hello) focusOffset: 4 isCollapsed: true]
+FAIL Selection is [anchorNode: [object Text](×××××) anchorOffset: 0 focusNode: [object Text](×××××) focusOffset: 0 isCollapsed: true] should be at anchorNode: [object Text] anchorOffset: 4 focusNode: [object Text] focusOffset: 4
+FAIL Selection is [anchorNode: [object Text](×××××) anchorOffset: 3 focusNode: [object Text](×××××) focusOffset: 3 isCollapsed: true] should be at anchorNode: [object Text] anchorOffset: 4 focusNode: [object Text] focusOffset: 4
+FAIL Selection is [anchorNode: [object Text](hello) anchorOffset: 1 focusNode: [object Text](hello) focusOffset: 1 isCollapsed: true] should be at anchorNode: [object Text] anchorOffset: 2 focusNode: [object Text] focusOffset: 2
+FAIL Selection is [anchorNode: [object Text](hello) anchorOffset: 3 focusNode: [object Text](hello) focusOffset: 3 isCollapsed: true] should be at anchorNode: [object Text] anchorOffset: 2 focusNode: [object Text] focusOffset: 2
 PASS successfullyParsed is true
Comment 2 Ryosuke Niwa 2011-08-18 08:35:11 PDT
Sorry about the failure, I didn't notice it.  But this is odd, I'm sure Qt supports editing behavior so this must be either that eventSender isn't working properly or Hebrew characters and directionality aren't treated properly.
Comment 3 Rafael Brandao 2012-01-24 14:30:38 PST
Jesus patch for bug #76821 seems to fix this one as well.
Comment 4 Jesus Sanchez-Palencia 2012-01-26 09:55:37 PST
(In reply to comment #3)
> Jesus patch for bug #76821 seems to fix this one as well.

This is now unskipped for Qt5 + WK2 and it works fine. It is still skipped for Qt 4 + WK1, though.

Should we keep it open?
Comment 5 Ádám Kallai 2012-10-03 07:09:18 PDT
Is Anybody working this bug? I tested it. It seems this test still fails on Qt. Do you have any idea?

Diff is here:

--- /home/kadam/webkit/WebKit/WebKitBuild/Release/layout-test-results/editing/selection/collapse-selection-in-bidi-expected.txt
+++ /home/kadam/webkit/WebKit/WebKitBuild/Release/layout-test-results/editing/selection/collapse-selection-in-bidi-actual.txt
@@ -1,9 +1,9 @@
 This tests collapsing directionless selection on text in a block with the opposite text direction. To manually test, on Mac, select text below by mouse and use arrow keys (without pressing shift) to collapse the selection. The selection should collapse to the left if you pressed the left arrow key and to the right if you pressed the right arrow key.
 
-PASS Selection is [anchorNode: [object Text](×בגדה) anchorOffset: 3 focusNode: [object Text](×בגדה) focusOffset: 3 isCollapsed: true]
-PASS Selection is [anchorNode: [object Text](×בגדה) anchorOffset: 1 focusNode: [object Text](×בגדה) focusOffset: 1 isCollapsed: true]
-PASS Selection is [anchorNode: [object Text](hello) anchorOffset: 1 focusNode: [object Text](hello) focusOffset: 1 isCollapsed: true]
-PASS Selection is [anchorNode: [object Text](hello) anchorOffset: 4 focusNode: [object Text](hello) focusOffset: 4 isCollapsed: true]
+FAIL Selection is [anchorNode: [object Text](×בגדה) anchorOffset: 1 focusNode: [object Text](×בגדה) focusOffset: 1 isCollapsed: true] should be at anchorNode: [object Text] anchorOffset: 5 focusNode: [object Text] focusOffset: 5
+FAIL Selection is [anchorNode: [object Text](hello) anchorOffset: 5 focusNode: [object Text](hello) focusOffset: 5 isCollapsed: true] should be at anchorNode: [object Text] anchorOffset: 0 focusNode: [object Text] focusOffset: 0
+FAIL Selection is [anchorNode: [object Text](hello) anchorOffset: 1 focusNode: [object Text](hello) focusOffset: 1 isCollapsed: true] should be at anchorNode: [object Text] anchorOffset: 2 focusNode: [object Text] focusOffset: 2
+PASS Selection is [anchorNode: [object Text](hello) anchorOffset: 5 focusNode: [object Text](hello) focusOffset: 5 isCollapsed: true]
 PASS successfullyParsed is true
 
 TEST COMPLETE
Comment 6 Ádám Kallai 2013-02-17 14:26:32 PST
The test is still failing on Qt.
Comment 7 Jocelyn Turcotte 2014-02-03 03:18:37 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.