Bug 1451169 - Use `nsStaticAtom* const` instead of `nsStaticAtom**` in DocAccessible.cpp. r=davidb
MozReview-Commit-ID: 6brOOd7hFqV
--- a/accessible/generic/DocAccessible.cpp
+++ b/accessible/generic/DocAccessible.cpp
@@ -49,27 +49,27 @@
#include "mozilla/dom/MutationEventBinding.h"
using namespace mozilla;
using namespace mozilla::a11y;
////////////////////////////////////////////////////////////////////////////////
// Static member initialization
-static nsStaticAtom** kRelationAttrs[] =
+static nsStaticAtom* const kRelationAttrs[] =
{
- &nsGkAtoms::aria_labelledby,
- &nsGkAtoms::aria_describedby,
- &nsGkAtoms::aria_details,
- &nsGkAtoms::aria_owns,
- &nsGkAtoms::aria_controls,
- &nsGkAtoms::aria_flowto,
- &nsGkAtoms::aria_errormessage,
- &nsGkAtoms::_for,
- &nsGkAtoms::control
+ nsGkAtoms::aria_labelledby,
+ nsGkAtoms::aria_describedby,
+ nsGkAtoms::aria_details,
+ nsGkAtoms::aria_owns,
+ nsGkAtoms::aria_controls,
+ nsGkAtoms::aria_flowto,
+ nsGkAtoms::aria_errormessage,
+ nsGkAtoms::_for,
+ nsGkAtoms::control
};
static const uint32_t kRelationAttrsLen = ArrayLength(kRelationAttrs);
////////////////////////////////////////////////////////////////////////////////
// Constructor/desctructor
DocAccessible::
@@ -1552,17 +1552,17 @@ DocAccessible::ProcessLoad()
void
DocAccessible::AddDependentIDsFor(Accessible* aRelProvider, nsAtom* aRelAttr)
{
dom::Element* relProviderEl = aRelProvider->Elm();
if (!relProviderEl)
return;
for (uint32_t idx = 0; idx < kRelationAttrsLen; idx++) {
- nsAtom* relAttr = *kRelationAttrs[idx];
+ nsStaticAtom* relAttr = kRelationAttrs[idx];
if (aRelAttr && aRelAttr != relAttr)
continue;
if (relAttr == nsGkAtoms::_for) {
if (!relProviderEl->IsAnyOfHTMLElements(nsGkAtoms::label,
nsGkAtoms::output))
continue;
@@ -1624,18 +1624,18 @@ void
DocAccessible::RemoveDependentIDsFor(Accessible* aRelProvider,
nsAtom* aRelAttr)
{
dom::Element* relProviderElm = aRelProvider->Elm();
if (!relProviderElm)
return;
for (uint32_t idx = 0; idx < kRelationAttrsLen; idx++) {
- nsAtom* relAttr = *kRelationAttrs[idx];
- if (aRelAttr && aRelAttr != *kRelationAttrs[idx])
+ nsStaticAtom* relAttr = kRelationAttrs[idx];
+ if (aRelAttr && aRelAttr != kRelationAttrs[idx])
continue;
IDRefsIterator iter(this, relProviderElm, relAttr);
while (true) {
const nsDependentSubstring id = iter.NextID();
if (id.IsEmpty())
break;