Summary: | REGRESSION: caret-ltr-right.html, caret-rtl-2-left.html, and caret-rtl-right.html fail on Windows bots | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> | ||||||||
Component: | HTML Editing | Assignee: | Ryosuke Niwa <rniwa> | ||||||||
Status: | RESOLVED LATER | ||||||||||
Severity: | Normal | CC: | aroben, bweinstein, Hironori.Fujii, mitz, xji | ||||||||
Priority: | P1 | Keywords: | InRadar, LayoutTestFailure, PlatformOnly | ||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | PC | ||||||||||
OS: | Windows 7 | ||||||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=262460 | ||||||||||
Attachments: |
|
Description
Ryosuke Niwa
2010-12-08 19:54:03 PST
(In reply to comment #0) > The main logic of these tests are implemented in http://trac.webkit.org/browser/trunk/LayoutTests/editing/selection/resources/caret-edge-shared.js > 15 eventSender.mouseMoveTo(div.offsetLeft + x, div.offsetTop + div.offsetHeight / 2); > 16 eventSender.leapForward(200); > 17 eventSender.mouseDown(); > 18 eventSender.leapForward(200); > 19 eventSender.mouseUp(); > > I'm emulating to click on the left or the right of LTR/RTL text but maybe I leapForward too much? > Or maybe we're setting focus to content editable area automatically and it's messing with selection. leapForward doesn't work on Windows. If the test relies on it then it is not surprising that it is failing. (In reply to comment #1) > (In reply to comment #0) > > The main logic of these tests are implemented in http://trac.webkit.org/browser/trunk/LayoutTests/editing/selection/resources/caret-edge-shared.js > > 15 eventSender.mouseMoveTo(div.offsetLeft + x, div.offsetTop + div.offsetHeight / 2); > > 16 eventSender.leapForward(200); > > 17 eventSender.mouseDown(); > > 18 eventSender.leapForward(200); > > 19 eventSender.mouseUp(); > > > > I'm emulating to click on the left or the right of LTR/RTL text but maybe I leapForward too much? > > Or maybe we're setting focus to content editable area automatically and it's messing with selection. > > leapForward doesn't work on Windows. If the test relies on it then it is not surprising that it is failing. Should we check in failing results and reference this bug? Created attachment 76288 [details]
fixed the tests
Comment on attachment 76288 [details]
fixed the tests
I lied this doesn't work :(
(In reply to comment #6) > (From update of attachment 76288 [details]) > I lied this doesn't work :( Apparently, Windows DRT is flaky. The test passes when I insert console.log but doesn't pass when there's no log. Also, changing the style of div seems to help sometimes. Adam & Brian, could you look into this? There's definitely something weird going on but I can't tell whether it's a bug in DRT or in WebCore that happens to only reproduce in Windows. Created attachment 76290 [details]
adds a magic console.log
Created attachment 76292 [details]
Added sad comments
Comment on attachment 76292 [details]
Added sad comments
Please file a bug about remove the console.log, and link to the bug (in a comment) next to the log (or in the log itself).
|