475503 - mochitest for hide event when DOM node containing accessibles is removed from DOM

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[alexander :surkov (:asurkov)]

no hide event when DOM node containing accessibles is removed from DOM

Comment 1

[alexander :surkov (:asurkov)]

it works actually, the problem was in mochitest

Comment 2

[alexander :surkov (:asurkov)]

Attachment: patch

Comment 3

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

Comment on attachment 378858 [details] [diff] [review]
patch

>+      /**
>+       * Change default target registered for the given event type for first
>+       * target from the array and inserts another targets from the array
>+       * after default target.
>+       */

I don't understand this.

Comment 4

[alexander :surkov (:asurkov)]

Attachment: patch2

is it better?

Comment 5

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

Comment on attachment 378862 [details] [diff] [review]
patch2

>+      /**
>+       * Change default target registered for the given event type on the first
>+       * node from the targets array. Inserts remaining nodes from the targets
>+       * array right after default target into event sequence array.
>+       */

Maybe something like:

Replace the default target currently registered for a given event type with the nodes in the targets array. The first node in the targets array specifies the default target to be replaced, the remaining nodes are what replaces it.

Comment 6

[alexander :surkov (:asurkov)]

Ok I changed locally on the following:

> Replace the default target currently registered for a given event type with the
> nodes in the targets array.

Comment 7

[Marco Zehe (:MarcoZ)]

Comment on attachment 378862 [details] [diff] [review]
patch2

r=me.

Comment 8

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

(In reply to comment #6)
> Ok I changed locally on the following:
> 
> > Replace the default target currently registered for a given event type with the
> > nodes in the targets array.

Sounds good.

Comment 9

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

Comment on attachment 378862 [details] [diff] [review]
patch2

r=me with the comment change.

Comment 10

[alexander :surkov (:asurkov)]

checked in on mozilla 1.9.2, http://hg.mozilla.org/mozilla-central/rev/aeca3394b861