Bug 1665962: Introduce roles::LISTITEM_MARKER to rolemap, add mac mapping to AXListMarker r=eeejay,Jamie

Differential Revision: https://phabricator.services.mozilla.com/D90745
2020-09-23 23:33:49 +00:00 · 2020-09-23 23:33:49 +00:00 · bfb99727ec
--- a/accessible/base/Role.h
+++ b/accessible/base/Role.h
@ -1071,7 +1071,13 @@ enum Role {
   */
  TIME_EDITOR = 183,

-  LAST_ROLE = TIME_EDITOR
+  /**
+   * Represents the marker associated with a list item. In unordered lists,
+   * this is a bullet, while in ordered lists this is a number.
+   */
+  LISTITEM_MARKER = 184,
+
+  LAST_ROLE = LISTITEM_MARKER
 };

 }  // namespace roles
--- a/accessible/base/RoleMap.h
+++ b/accessible/base/RoleMap.h
@ -1866,4 +1866,14 @@ ROLE(TIME_EDITOR,
     ROLE_SYSTEM_GROUPING,
     java::SessionAccessibility::CLASSNAME_VIEW,
     eNameFromSubtreeIfReqRule)
+
+ROLE(LISTITEM_MARKER,
+     "list item marker",
+     ATK_ROLE_UNKNOWN,
+     @"AXListMarker",
+     NSAccessibilityUnknownSubrole,
+     ROLE_SYSTEM_STATICTEXT,
+     ROLE_SYSTEM_STATICTEXT,
+     java::SessionAccessibility::CLASSNAME_VIEW,
+     eNoNameRule)
 // clang-format on
--- a/accessible/base/TextAttrs.cpp
+++ b/accessible/base/TextAttrs.cpp
@ -578,7 +578,8 @@ TextAttrsMgr::AutoGeneratedTextAttr::AutoGeneratedTextAttr(

  if (aAccessible)
    mIsDefined = mNativeValue =
-        (aAccessible->NativeRole() == roles::STATICTEXT);
+        ((aAccessible->NativeRole() == roles::STATICTEXT) ||
+         (aAccessible->NativeRole() == roles::LISTITEM_MARKER));
 }

 bool TextAttrsMgr::AutoGeneratedTextAttr::GetValueFor(Accessible* aAccessible,
--- a/accessible/html/HTMLListAccessible.cpp
+++ b/accessible/html/HTMLListAccessible.cpp
@ -154,7 +154,9 @@ ENameValueFlag HTMLListBulletAccessible::Name(nsString& aName) const {
  return eNameOK;
 }

-role HTMLListBulletAccessible::NativeRole() const { return roles::STATICTEXT; }
+role HTMLListBulletAccessible::NativeRole() const {
+  return roles::LISTITEM_MARKER;
+}

 uint64_t HTMLListBulletAccessible::NativeState() const {
  return LeafAccessible::NativeState() | states::READONLY;
--- a/accessible/interfaces/nsIAccessibleRole.idl
+++ b/accessible/interfaces/nsIAccessibleRole.idl
@ -1065,4 +1065,10 @@ interface nsIAccessibleRole : nsISupports
   */
  const unsigned long ROLE_TIME_EDITOR = 183;

+  /**
+   * Represents the marker associated with a list item. In unordered lists,
+   * this is a bullet, while in ordered lists this is a number.
+   */
+  const unsigned long ROLE_LISTITEM_MARKER = 184;
+
 };
--- a/accessible/tests/mochitest/elm/test_HTMLSpec.html
+++ b/accessible/tests/mochitest/elm/test_HTMLSpec.html
@ -1030,21 +1030,21 @@
        children: [
          { role: ROLE_LISTITEM,
            children: [ // home
-              { role: ROLE_STATICTEXT },
+              { role: ROLE_LISTITEM_MARKER },
              { role: ROLE_TEXT_LEAF }
            ]
          },
          {
            role: ROLE_LISTITEM,
            children: [
-              { role: ROLE_STATICTEXT },
+              { role: ROLE_LISTITEM_MARKER },
              { role: ROLE_TEXT_LEAF }, // about
              {
                role: ROLE_LIST, // menu
                children: [
                  { role: ROLE_LISTITEM,
                    children: [
-                    { role: ROLE_STATICTEXT },
+                    { role: ROLE_LISTITEM_MARKER },
                    { role: ROLE_TEXT_LEAF } // our story
                    ]
                  },
@ -1062,7 +1062,7 @@
        {
          role: ROLE_LISTITEM,
          children: [
-          { role: ROLE_STATICTEXT },
+          { role: ROLE_LISTITEM_MARKER },
          {
            role: ROLE_PUSHBUTTON,
            children: [
@ -1075,7 +1075,7 @@
            {
              role: ROLE_LISTITEM,
              children: [
-                { role: ROLE_STATICTEXT },
+                { role: ROLE_LISTITEM_MARKER },
                {
                  role: ROLE_PUSHBUTTON,
                  children: [
--- a/accessible/tests/mochitest/pivot.js
+++ b/accessible/tests/mochitest/pivot.js
@ -62,7 +62,7 @@ var ObjectTraversalRule = {
      rv = FILTER_IGNORE_SUBTREE | FILTER_MATCH;
    } else if (
      aAccessible.childCount == 0 &&
-      role != ROLE_STATICTEXT &&
+      role != ROLE_LISTITEM_MARKER &&
      aAccessible.name.trim()
    ) {
      rv = FILTER_MATCH;
--- a/accessible/tests/mochitest/role.js
+++ b/accessible/tests/mochitest/role.js
@ -55,6 +55,7 @@ const ROLE_LINK = nsIAccessibleRole.ROLE_LINK;
 const ROLE_LIST = nsIAccessibleRole.ROLE_LIST;
 const ROLE_LISTBOX = nsIAccessibleRole.ROLE_LISTBOX;
 const ROLE_LISTITEM = nsIAccessibleRole.ROLE_LISTITEM;
+const ROLE_LISTITEM_MARKER = nsIAccessibleRole.ROLE_LISTITEM_MARKER;
 const ROLE_MARK = nsIAccessibleRole.ROLE_MARK;
 const ROLE_MATHML_MATH = nsIAccessibleRole.ROLE_MATHML_MATH;
 const ROLE_MATHML_IDENTIFIER = nsIAccessibleRole.ROLE_MATHML_IDENTIFIER;
--- a/accessible/tests/mochitest/tree/test_aria_menu.html
+++ b/accessible/tests/mochitest/tree/test_aria_menu.html
@ -19,7 +19,7 @@
      { SECTION: [ // container
        { MENUPOPUP: [ // menu
          { MENUITEM: [
-            { STATICTEXT: [] }, // bullet
+            { LISTITEM_MARKER: [] }, // bullet
            { TEXT_LEAF: [] },
          ] },
        ] },
@ -31,7 +31,7 @@
      { SECTION: [ // container
        { MENUPOPUP: [ // menu
          { MENUITEM: [
-            { STATICTEXT: [] }, // bullet
+            { LISTITEM_MARKER: [] }, // bullet
            { TEXT_LEAF: [] },
          ] },
        ] },
@ -43,7 +43,7 @@
      { SECTION: [ // container
        { MENUPOPUP: [ // menu
          { PARENT_MENUITEM: [ // menuitem with aria-haspopup="true"
-            { STATICTEXT: [] }, // bullet
+            { LISTITEM_MARKER: [] }, // bullet
            { TEXT_LEAF: [] },
          ] },
        ] },
--- a/accessible/tests/mochitest/tree/test_brokencontext.html
+++ b/accessible/tests/mochitest/tree/test_brokencontext.html
@ -77,11 +77,11 @@
    tree =
      { PUSHBUTTON: [ // ul
        { TEXT_CONTAINER: [ // li
-          { STATICTEXT: [ ] },
+          { LISTITEM_MARKER: [ ] },
          { TEXT_LEAF: [ ] },
        ] },
        { TEXT_CONTAINER: [ // span styled as a list
-          { STATICTEXT: [ ] },
+          { LISTITEM_MARKER: [ ] },
          { TEXT_LEAF: [ ] },
        ] },
      ] };
@ -90,7 +90,7 @@
    tree =
      { PUSHBUTTON: [ // ol
        { TEXT_CONTAINER: [ // li
-          { STATICTEXT: [ ] },
+          { LISTITEM_MARKER: [ ] },
          { TEXT_LEAF: [ ] },
        ] },
      ] };
--- a/accessible/tests/mochitest/tree/test_display_contents.html
+++ b/accessible/tests/mochitest/tree/test_display_contents.html
@ -18,11 +18,11 @@ function doTest() {
  let tree =
    { LIST: [
      { LISTITEM: [
-        { STATICTEXT: [] },
+        { LISTITEM_MARKER: [] },
        { TEXT_LEAF: [] },
      ]},
      { LISTITEM: [
-        { STATICTEXT: [] },
+        { LISTITEM_MARKER: [] },
        { TEXT_LEAF: [] },
      ]},
    ] };
--- a/accessible/tests/mochitest/tree/test_list.html
+++ b/accessible/tests/mochitest/tree/test_list.html
@ -18,7 +18,7 @@
        role: ROLE_LISTITEM,
        children: [
          {
-            role: ROLE_STATICTEXT,
+            role: ROLE_LISTITEM_MARKER,
            name: aBulletText,
          },
          {
@ -106,7 +106,7 @@
      tree =
        { LIST: [ // ol
          { LISTITEM: [ // li
-            { STATICTEXT: [ ] },
+            { LISTITEM_MARKER: [ ] },
            { DEFINITION_LIST: [ // dl
              { TERM: [ // dt
                { TEXT_LEAF: [] },
--- a/accessible/tests/mochitest/treeupdate/test_delayed_removal.html
+++ b/accessible/tests/mochitest/treeupdate/test_delayed_removal.html
@ -197,7 +197,7 @@

    function listItemReframe() {
      testAccessibleTree("li",{ LISTITEM: [
-        { STATICTEXT: [] },
+        { LISTITEM_MARKER: [] },
        { TEXT_LEAF: [] },
      ] });

@ -206,7 +206,7 @@
      window.windowUtils.advanceTimeAndRefresh(100);

      testAccessibleTree("li",{ LISTITEM: [
-        { STATICTEXT: [] },
+        { LISTITEM_MARKER: [] },
        { TEXT_LEAF: [] },
      ] });

--- a/accessible/tests/mochitest/treeupdate/test_general.html
+++ b/accessible/tests/mochitest/treeupdate/test_general.html
@ -120,7 +120,7 @@
          { SECTION: [ // container
            { LIST: [
              { LISTITEM: [
-                { STATICTEXT: [] },
+                { LISTITEM_MARKER: [] },
                { TEXT_LEAF: [] },
              ] },
            ] },
--- a/accessible/tests/mochitest/treeupdate/test_list.html
+++ b/accessible/tests/mochitest/treeupdate/test_list.html
@ -26,7 +26,7 @@
        role: ROLE_LISTITEM,
        children: [
          {
-            role: ROLE_STATICTEXT,
+            role: ROLE_LISTITEM_MARKER,
            children: [],
          },
          {
@ -87,7 +87,7 @@
      this.onProcessed = function textReplaceProcessor_onProcessed() {
        var tree = {
          LISTITEM: [
-            { STATICTEXT: [] },
+            { LISTITEM_MARKER: [] },
            { TEXT_LEAF: [] },
          ],
        };
--- a/accessible/tests/mochitest/treeupdate/test_list_editabledoc.html
+++ b/accessible/tests/mochitest/treeupdate/test_list_editabledoc.html
@ -42,7 +42,7 @@
              role: ROLE_LISTITEM,
              children: [
                {
-                  role: ROLE_STATICTEXT,
+                  role: ROLE_LISTITEM_MARKER,
                  name: "1. ",
                  children: [],
                },