462314 - Refactor test_aria_activedescendant.html

Status: RESOLVED FIXED

Description

[Marco Zehe (:MarcoZ)]

Refactor the file so it uses common files and functions.

Comment 1

[Marco Zehe (:MarcoZ)]

Created attachment 345464 [details] [diff] [review]
Patch

Comment 2

[alexander :surkov]

Comment on attachment 345464 [details] [diff] [review]
Patch


>     var gContainerFocused = false;
> 
>     function doTest()
>     {
>       var focusHandler = {
>         handleEvent: function handleEvent(aEvent) {
>           var target = aEvent.target;
>           if (target.nodeType ==  ELEMENT_NODE &&
>               target.getAttribute("id") == "container")
>             gContainerFocused = true;
>         }
>       };
> 
>       var container = document.getElementById("container");
>       container.addEventListener("focus", focusHandler, false);
> 
>-      gAccRetrieval = Components.classes["@mozilla.org/accessibleRetrieval;1"].
>-                      getService(nsIAccessibleRetrieval);
> 
>       var item2 = document.getElementById("item2");

remove this line and

>+      var item2Acc = getAccessible(item2);

pass "item2"

>+      if (!item2Acc) {
>+        SimpleTest.finish();
>+        return;
>+      }

nit: I would prefer
if  (item2Acc) {
}

once you are here please remove event listener at the end.

>       SimpleTest.finish();
>     }

Comment 3

[Marco Zehe (:MarcoZ)]

Pushed with Surkov's comments addressed in changeset:
http://hg.mozilla.org/mozilla-central/rev/1ac20b494c0f