426546 - Investigate why the mochitest for bug 392746 is failing on linux (and try to fix it)

Status: RESOLVED FIXED

(Core :: DOM: Selection, defect)

Description

[Martijn Wargers (dead)]

The mochitest for bug 392746 was disabled for Linux, because it failed on the Linux tinderbox.

I guess it's failing there, because the wrong text is being selected or something.
Anyhow, this needs to be investigated and fixed, so that this test also works on Linux.

Comment 1

[Serge Gautherie (:sgautherie)]

In the meantime,
{
*** 43863 INFO Running /tests/layout/generic/test/test_bug392746.html...
*** 43864 INFO TEST-KNOWN-FAIL | /tests/layout/generic/test/test_bug392746.html | [SimpleTest.report()] No checks actually run.
}

Comment 2

[Serge Gautherie (:sgautherie)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox3.5/1242134886.1242140795.18418.gz&fulltext=1
Linux mozilla-1.9.1 unit test on 2009/05/12 06:28:06

Comment 3

[Ginn Chen]

The test case failed on Solaris if I build Firefox 3.5 with system cairo library.
But it doesn't fail if I use mozilla internal cairo.
Weird.
After the failure, the mouse cursor looks like some text is being dragged.
And then hundreds of cases are timeout because Firefox couldn't grab the focus.

Comment 4

[Ginn Chen]

Attachment: patch

I think the root cause of this bug is the text somehow wrapped in the mochitest window.
i.e.
text ccc text text text eee text text text text text bbb
text

I removed some texts and hope it will not wrap.

I will send it to try server to confirm.

Comment 5

[Martijn Wargers (dead)]

Comment on attachment 442354 [details] [diff] [review]
patch

> I think the root cause of this bug is the text somehow wrapped in the mochitest
window.
But why does this happen only on linux? I guess the font is a little bit bigger there or something?

-  wu.sendMouseEvent('mouseup',  aY2, aY2, 0, 1, modifyers);
+  wu.sendMouseEvent('mouseup',  aX2, aY2, 0, 1, modifyers);
What is changed here? Nothing it seems.

Anyway, if this fixes the test for linux, then that's great!
Thanks for fixing!

Comment 6

[Ginn Chen]

From try server results, it fixed the problem.

I changed (aY2, aY2) to (aX2, aY2).
It is meaningless for mouseup event, I think.
But (aY2, aY2) doesn't make sense to me.

Comment 7

[Ginn Chen]

http://hg.mozilla.org/mozilla-central/rev/0ada8755f63c