508523 - Crash [@ nsDocAccessible::FlushPendingEvents]

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[[e-mail]]

Hit this while browsing, no STR, sorry.

>  NS_RELEASE_THIS(); // Release kung fu death grip
>
>  // After a flood of events, reset so that user input flag is off
>  nsAccEvent::ResetLastInputState();
>
>  mInFlushPendingEvents = PR_FALSE;
Crashed here; "this" might have been destroyed by NS_RELEASE_THIS above?

Comment 1

[[e-mail]]

Attachment: Possible patch (untested)

Comment 2

[alexander :surkov (:asurkov)]

Comment on attachment 392690 [details] [diff] [review]
Possible patch (untested)

r=me

Comment 3

[Martijn Wargers (dead)]

Would this fix bug 488505?

Comment 4

[[e-mail]]

Pushed changeset 42a364a99f95 to mozilla-central.

Comment 5

[David Bolter [:davidb] (NeedInfo me for attention)]

Please see https://bugzilla.mozilla.org/show_bug.cgi?id=488505#c14 for help with a+ing blocking 1.9.2

Comment 6

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

Nominate the patch for approval, not the bug for blocking, please.

Comment 7

[David Bolter [:davidb] (NeedInfo me for attention)]

Comment on attachment 392690 [details] [diff] [review]
Possible patch (untested)

Crash fix.

Comment 8

[David Bolter [:davidb] (NeedInfo me for attention)]

Comment on attachment 392690 [details] [diff] [review]
Possible patch (untested)

I took a look at the 1.9.2 source and this patch is already appplied. Canceling 1.9.2?