Closed Bug 462316 Opened 11 years ago Closed 11 years ago

Refactor test_aria_role_article.html

Categories

(Core :: Disability Access APIs, defect)

defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: MarcoZ, Assigned: MarcoZ)

References

Details

(Keywords: access)

Attachments

(1 file)

Refactor the file to use common files and functions.
Attached patch PatchSplinter Review
Assignee: nobody → marco.zehe
Status: NEW → ASSIGNED
Attachment #345465 - Flags: review?(surkov.alexander)
Comment on attachment 345465 [details] [diff] [review]
Patch


>       // Test article exposed as document
>       var articleElement = document.getElementById("testArticle");
>-      var articleAcc = null;
>-      try {
>-        articleAcc = accRetrieval.getAccessibleFor(articleElement);
>-      } catch(e) { }
>-      ok(articleAcc, "no accessible for article!");
>-      if (articleAcc) {
>-        is(articleAcc.finalRole,
>-           Components.interfaces.nsIAccessibleRole.ROLE_DOCUMENT,
>-           "Wrong role for article!");
>-        is(articleAcc.name, "Test article", "Wrong name for article!");
>+      var articleAcc = getAccessible(articleElement);

pass ID instead.
Attachment #345465 - Flags: review?(surkov.alexander) → review+
Pushed with Surkov's comments addressed in changeset:
http://hg.mozilla.org/mozilla-central/rev/443f9092eba5
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.