Bug 302733. Expose more important structural elements in HTML. r+sr=roc, a=bsmedberg

accessible/src/base/nsAccessibilityAtomList.h

@@ -73,6 +73,9 @@ ACCESSIBILITY_ATOM(blockquote, "blockquote")
 ACCESSIBILITY_ATOM(br, "br")
 ACCESSIBILITY_ATOM(body, "body")
 ACCESSIBILITY_ATOM(description, "description")    // XUL
+ACCESSIBILITY_ATOM(dd, "dd")
+ACCESSIBILITY_ATOM(dl, "dl")
+ACCESSIBILITY_ATOM(dt, "dt")
 ACCESSIBILITY_ATOM(form, "form")
 ACCESSIBILITY_ATOM(h1, "h1")
 ACCESSIBILITY_ATOM(h2, "h2")
@@ -83,6 +86,7 @@ ACCESSIBILITY_ATOM(h6, "h6")
 ACCESSIBILITY_ATOM(img, "img")
 ACCESSIBILITY_ATOM(input, "input")
 ACCESSIBILITY_ATOM(label, "label")
+ACCESSIBILITY_ATOM(li, "li")
 ACCESSIBILITY_ATOM(link, "link")
 ACCESSIBILITY_ATOM(object, "object")
 ACCESSIBILITY_ATOM(ol, "ol")

accessible/src/base/nsAccessibilityService.cpp

@@ -507,9 +507,17 @@ nsAccessibilityService::CreateHTMLAccessibleByMarkup(nsISupports *aFrame,
   else if (tag == nsAccessibilityAtoms::a) {
     *aAccessible = new nsHTMLLinkAccessible(aNode, aWeakShell, NS_STATIC_CAST(nsIFrame*, aFrame));
   }
+  else if (tag == nsAccessibilityAtoms::li) {
+    // Normally this is created by the list item frame which knows about the bullet frame
+    // However, in this case the list item must have been styled using display: foo
+    *aAccessible = new nsHTMLLIAccessible(aNode, aWeakShell, nsnull, EmptyString());
+  }
   else if (tag == nsAccessibilityAtoms::abbr ||
            tag == nsAccessibilityAtoms::acronym ||
            tag == nsAccessibilityAtoms::blockquote ||
+           tag == nsAccessibilityAtoms::dd ||
+           tag == nsAccessibilityAtoms::dl ||
+           tag == nsAccessibilityAtoms::dt ||
            tag == nsAccessibilityAtoms::form ||
            tag == nsAccessibilityAtoms::h1 ||
            tag == nsAccessibilityAtoms::h2 ||