426195 - Colorpicker test can go wrong after ctrl-select test

Status: NEW

(Core :: DOM: UI Events & Focus Handling, defect)

Description

[Martijn Wargers (dead)]

Attachment: zipped up testcase (unminimized)

This is a follow-up from bug 392746 where it was noticed that the checked in mochitest there caused failures in completely unrelated mochitests.

In this bug, I'm attaching an attempt at minimizing that problem, see attached zipped up testcase (unminimized).

To reproduce:
- Extract the attached file
- Open parentframe.htm, let the test run

Actual result:
Failed: 2 for the Colorpicker tests

Expected result:
Passed: 2 for the Colorpicker tests

Apparently, parentframe.htm needs a scrollbar in the page (the page needs to overflow), in order for the failures to occur. So it seems like it has something to do with the arrow key down scrolling or something.

I'm filing this for now in Core->Testing, mainly because I don't really know in which component it has to be in, for now.

Comment 1

[Martijn Wargers (dead)]

Attachment: zipped up testcase, as minimized as possible

To reproduce:
- Unzip the testcase
- Press the arrow down key, notice that the selected color cell is the red one.
- wait 1 second (some redirecting and loading the colorpicker again is happening)
- Press the arrow down key, notice that the selected color cell is now the white one (the first one).

Comment 2

[Martijn Wargers (dead)]

Attachment: some added dump commands in colorpicker.xml

When running the testcase, I get:
Before test_bug392746.html:
initialize
iscolorcell#FFFFFF
onpopupshowing
cells.item(0)
iscolorcell#FF0000

After test_bug392746.html:
initialize
iscolorcell#FFFFFF
onpopupshowing
cells.item(0)
iscolorcell#FF0000
mouseover
iscolorcell#CCCCCC

So apparently, somehow a mouseover event is fired after test_bug392746.html has been run. I have no idea why that is happening.

Comment 3

[Martijn Wargers (dead)]

Attachment: testcase

The testcase uses enhanced privileges, so you need to download it to your computer.

With this testcase, I'm getting mouseover and mouseout alerts, which I would not expect to happen.

Comment 4

[Martijn Wargers (dead)]

Attachment: testcase2

Comment 5

[Martijn Wargers (dead)]

Moving this to Event Handling.
Neil, perhaps you have anything meaningful to say? The alert should not be appearing with the testcases, right?

Comment 6

[Martijn Wargers (dead)]

One day, this issue might cause mochitest failures again.