Bug 35145
Summary: | [Qt] editing tests fails after r54980 | ||
---|---|---|---|
Product: | WebKit | Reporter: | Csaba Osztrogonác <ossy> |
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED INVALID | ||
Severity: | Normal | CC: | abarth, ojan, rniwa, tonikitoo |
Priority: | P3 | Keywords: | LayoutTestFailure, Qt, QtTriaged |
Version: | 528+ (Nightly build) | ||
Hardware: | PC | ||
OS: | All |
Csaba Osztrogonác
1.) editing/selection/extend-selection.html
diff: http://build.webkit.org/results/Qt%20Linux%20Release/r55007%20%287579%29/editing/selection/extend-selection-pretty-diff.html
2.) editing/selection/extend-after-mouse-selection.html (new test introduced in r54980)
resut: http://build.webkit.org/results/Qt%20Linux%20Release/r55007%20%287579%29/editing/selection/extend-after-mouse-selection-actual.txt
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Csaba Osztrogonác
original changeset: http://trac.webkit.org/changeset/54980
skipped them by: http://trac.webkit.org/changeset/55010
Ryosuke Niwa
editing/selection/extend-selection.html has been fixed since the bug 49849 has been resolved.
http://trac.webkit.org/browser/trunk/LayoutTests/editing/selection/extend-after-mouse-selection.html
This test is missing leapForward between mouseDown & mouseUp. Could you try adding eventSender.leapForward(100) between each and see if it passes? Also, we probably need to add 1 or 2 pixels to startTarget.offsetLeftwhen moving mouse to.
Jocelyn Turcotte
=== 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.