356720 - Event processing stops during certain UI interactions

Status: RESOLVED FIXED

(Core :: Widget: Cocoa, defect)

Description

[[e-mail]]

Page loading pause while you hold down mouse button on the scroll-bars.
The same goes for animations on a fully loaded website. All animations, GIF, Flash, etc. pause when you hold down the mouse button.

I reproduced this on the latest Camino trunk and CocoaFox (on www.vg.no ).
...and found the regression to happend all the way back in May!
- Works: Gecko/20060510 Camino/1.2+
- Fails: Gecko/20060511 Camino/1.2+

Checkins between 2006-05-10 01:00 and 2006-05-11 01:00: http://tinyurl.com/ymjp9k


By the way, I can't reproduce this with yesterdays Camino 1.8 branch build.

Comment 1

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

This is almost surely a regression from the thread manager landing (bug 326273), which is smack in the middle of that range.

Comment 2

[Håkan Waara]

I've noticed this as well. 

If anyone can find a regression range for when the check-in happened, that would be good. I would suspect Mark's event changes (some, but not all of them, were part of the thread manager landing).

Comment 3

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

This should probably just be duped to bug 338225.

Comment 4

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Invoking a CM (in Camino) or opening a menu also stop all loading/animations.

Comment 5

[Mark Mentovai]

Yeah, this is definitely caused by the new events system, from when we had to back out the stuff we did to get Gecko events firing inside native event loops.  338225 sounds right.

Comment 7

[Colin Barrett [:cbarrett]]

I'm in favor of duping this to bug 338225.

Comment 8

[Steven Michaud [:smichaud] (Retired)]

This bug was perhaps the only bug fixed by a patch I wrote a few
months ago (not yet posted) that removed one level of nesting of
native event loops (what I've been calling my "appshell patch" or "run
loop patch", and which I emailed to you (Colin) shortly after I wrote
it).

I'm not sure it's possible (or even desirable) to remove all levels of
nesting of native event loops.  But it _does_ make sense to stop this
nesting from causing problems with Gecko (which on one interpretation
is the subject of bug 338225).

So I've DUPed as Colin suggests ... but this is (of course) only
tentative.

Comment 9

[Håkan Waara]

Will you also carry over the blocking1.9+ flag? I.e., will this appshell patch make it to Firefox 3?

Comment 10

[Steven Michaud [:smichaud] (Retired)]

My appshell patch would be superceded by a fix for bug 338225.

But I'm not sure it'll be possible to fix bug 338225 before Firefox 3
is released.  And my appshell patch is pretty simple and exists now.
I'll try to open a bug on it next week.  When I do so I'll request
blocking1.9 on it.

Comment 11

[Steven Michaud [:smichaud] (Retired)]

Come to think of it, it makes more sense to make this bug depend on
bug 338225.

Comment 12

[Steven Michaud [:smichaud] (Retired)]

(Following up comment #10)

It took me longer than expected, but I've now posted my appshell patch
at bug 395397.

It doesn't fix the problem reported in comment #0 (which in any case
is no longer reproducible).  But it does fix the following problems,
all of which are related to the original report:

Gecko event processing is suspended (on the trunk in both Minefield
and Camino) while you do the following:

1) Resize a browser window using its "resize region" (in the lower
   right corner).
2) Keep one of the "main" menus open.
3) (Camino only) Keep a context menu open.  (This happens only in
   Camino because only Camino uses "native" context menus.)

Comment 13

[Josh Aas]

Changed the title of this bug to reflect the fact that holding the mouse down on a scrollbar is only one way of causing the same problem and stopping pageloads is only one symptom. Resizing the window, using the native menu bar, and holding the mouse down on certain UI elements are all ways of causing this bug. Symptoms include paused page loading and paused video/sound playback from plugins (think YouTube).

Comment 14

[Steven Michaud [:smichaud] (Retired)]

I just landed my appshell patch (bug 395397), which fixes this bug.

Comment 15

[Serge Gautherie (:sgautherie)]

FWIW, in "Depends on", you may want to choose between bug 389931 and bug 395397.