398334 - "ASSERTION: previous transaction is not finished!"

Status: RESOLVED FIXED

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

Description

[Jesse Ruderman]

Attachment: stack trace

I hit this assertion a few times today:

###!!! ASSERTION: previous transaction is not finished!: '!sTargetFrame', file /Users/jruderman/trunk/mozilla/content/events/src/nsEventStateManager.cpp, line 299

I think I hit it by clicking a link from one bug report to another and two-finger scrolling before the second page started to load.  I can't reproduce it reliably.

Comment 1

[Mats Palmgren (inactive)]

Also occurs on Linux (100% reproducible):
1. load a Bugzilla bug page (logged in)
2. hover the "Product" combobox
3. mouse-wheel scroll down

Comment 2

[Mats Palmgren (inactive)]

Attachment: Patch rev. 1

If we started a transaction for the combobox (because it returned true from
CanScroll()) we need to end it before continuing the loop.

Not really important to fix for 1.9 but it seems low risk if we want it.

Comment 3

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Comment on attachment 307696 [details] [diff] [review]
Patch rev. 1

oops

Comment 4

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 307696 [details] [diff] [review]
Patch rev. 1

a1.9=beltzner

Comment 5

[Mats Palmgren (inactive)]

mozilla/content/events/src/nsEventStateManager.cpp 	1.734 

-> FIXED