Comment 1 alexander :surkov 2012-06-04 08:14:54 PDT

Created attachment 629784 [details] [diff] [review]
patch

ignore aria-busy stuffs, they complicate the logic and no evidence the web really needs it

Comment 2 David Bolter [:davidb] 2012-06-04 08:23:14 PDT

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

Review of attachment 629784 [details] [diff] [review]:
-----------------------------------------------------------------

Fine with me. I agree with comment 1.

Comment 3 Trevor Saunders (:tbsaunde) 2012-06-06 22:42:07 PDT

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

>+
>+  // Fire document load complete on ARIA documents.
>+  // XXX: we should delay an event if the ARIA document has aria-busy.
>+  if (aRoot->HasARIARole() && !aRoot->IsDoc()) {
>+    a11y::role role = aRoot->ARIARole();
>+    if (role == roles::DIALOG || role == roles::DOCUMENT)

kind of funny we do it for dialogs too, but I guss its sort of reasonable

>+      FireDelayedAccessibleEvent(nsIAccessibleEvent::EVENT_DOCUMENT_LOAD_COMPLETE,
>+                                 aRoot->GetContent());

it'd be nice if you didn't add to the event firing based on nodes.  I'd be sort of fine with a temporary inline function that got the accessibles node and fired the event.

>+    {
>+      gQueue = new eventQueue();
>+
>+      gQueue.push(new showARIADialog("dialog"));

it'd be nice to have one for ria documents in case logic changes somehow one day.

Comment 4 alexander :surkov 2012-06-06 23:19:40 PDT

(In reply to Trevor Saunders (:tbsaunde) from comment #3)

> kind of funny we do it for dialogs too, but I guss its sort of reasonable

yep but we were asked for this and after all we do that anyway if you apply proper ARIA role to document.

> >+      FireDelayedAccessibleEvent(nsIAccessibleEvent::EVENT_DOCUMENT_LOAD_COMPLETE,
> >+                                 aRoot->GetContent());
> 
> it'd be nice if you didn't add to the event firing based on nodes.  I'd be
> sort of fine with a temporary inline function that got the accessibles node
> and fired the event.

I wouldn't do this time since coalescence happens by DOM tree. I need to reorg all this code one day.

> >+      gQueue.push(new showARIADialog("dialog"));
> 
> it'd be nice to have one for ria documents in case logic changes somehow one
> day.

ok

Comment 5 alexander :surkov 2012-06-08 06:34:45 PDT

https://hg.mozilla.org/integration/mozilla-inbound/rev/0f29f00df2ff

Comment 6 Ed Morley [:emorley] 2012-06-08 13:54:54 PDT

https://hg.mozilla.org/mozilla-central/rev/0f29f00df2ff