337993 - Clicking on a button doesn't always give a pressed look anymore

Status: RESOLVED WORKSFORME

(Core :: XPCOM, defect, P3)

Description

[Martijn Wargers (dead)]

Steps to reproduce:
- Visit url testcase
- While the page is loading and laying out the page, press the stop button

Actual result:
Sometimes, the stop button doesn't get the pressed look when it is clicked upon.

Expected result:
The stop button should always get a pressed look when clicked upon.

If you don't see the bug directly, then try again with doing a shift->reload.

I'm not seeing this with a 2006-05-10 build, but I'm seeing this with a 2006-05-11 build, so I suspect this is a regression from bug 326273.

Note that I've also seen this with reload buttons and submit buttons. The steps to reproduce are just a way to get this bug to occur semi-reproducable.
Also note that this may be particular noticeable on a slower computer, I have a 800MHz Duron.

Comment 1

[Darin Fisher]

I bet this bug is related to bug 337711.  I know that there is special Windows widget code that attempts to flush all pending gecko events (PLEvents in the old world) before painting during mouse moves.  That code is critical for making menus render properly under heavy event queue load.  It seems like there might have been some other places in the code that need something like that as well that were previously benefiting from PL_ProcessPendingEvents processing all pending events.  Now, since gecko events are intermixed with UI events, we can end up with UI events running earlier than they previously would have run.

-> me

Comment 2

[Darin Fisher]

-> reassign to default owner

Comment 3

[Martijn Wargers (dead)]

I can still quite easily reproduce this in current trunk build, when I have http://www.movenetworks.com/ opened in 2 tabs.
It seems like the Flash clip on that site is filling up the message queue or something like that, making this bug easier visible.

Comment 4

[Samuel Sidler (old account; do not CC)]

Doesn't meet the "wanted" criteria (security, stability, regression from maintenance release) for 1.9.0.x.