Last few steps of text alternative computation algorithm

Source/modules/accessibility/AXNodeObject.cpp

 /*
-* Copyright (C) 2012, Google Inc. All rights reserved.
-*
-* Redistribution and use in source and binary forms, with or without
-* modification, are permitted provided that the following conditions
-* are met:
-*
-* 1.  Redistributions of source code must retain the above copyright
-*     notice, this list of conditions and the following disclaimer.
-* 2.  Redistributions in binary form must reproduce the above copyright
-*     notice, this list of conditions and the following disclaimer in the
-*     documentation and/or other materials provided with the distribution.
-* 3.  Neither the name of Apple Computer, Inc. ("Apple") nor the names of
-*     its contributors may be used to endorse or promote products derived
-*     from this software without specific prior written permission.
-*
-* THIS SOFTWARE IS PROVIDED BY APPLE AND ITS CONTRIBUTORS "AS IS" AND ANY
-* EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
-* WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
-* DISCLAIMED. IN NO EVENT SHALL APPLE OR ITS CONTRIBUTORS BE LIABLE FOR ANY
-* DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
-* (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
-* LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND
-* ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
-* (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
-* THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-*/
+ * Copyright (C) 2012, Google Inc. All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ *
+ * 1.  Redistributions of source code must retain the above copyright
+ *     notice, this list of conditions and the following disclaimer.
+ * 2.  Redistributions in binary form must reproduce the above copyright
+ *     notice, this list of conditions and the following disclaimer in the
+ *     documentation and/or other materials provided with the distribution.
+ * 3.  Neither the name of Apple Computer, Inc. ("Apple") nor the names of
+ *     its contributors may be used to endorse or promote products derived
+ *     from this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY APPLE AND ITS CONTRIBUTORS "AS IS" AND ANY
+ * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
+ * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
+ * DISCLAIMED. IN NO EVENT SHALL APPLE OR ITS CONTRIBUTORS BE LIABLE FOR ANY
+ * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
+ * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
+ * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND
+ * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
+ * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
 
 #include "config.h"
 #include "modules/accessibility/AXNodeObject.h"
 
 #include "core/InputTypeNames.h"
 #include "core/dom/NodeTraversal.h"
 #include "core/dom/Text.h"
+#include "core/dom/shadow/ComposedTreeTraversal.h"
 #include "core/html/HTMLDListElement.h"
 #include "core/html/HTMLFieldSetElement.h"
 #include "core/html/HTMLFrameElementBase.h"
 #include "core/html/HTMLImageElement.h"
 #include "core/html/HTMLInputElement.h"
 #include "core/html/HTMLLabelElement.h"
 #include "core/html/HTMLLegendElement.h"
 #include "core/html/HTMLMediaElement.h"
 #include "core/html/HTMLMeterElement.h"
 #include "core/html/HTMLPlugInElement.h"
 #include "core/html/HTMLSelectElement.h"
 #include "core/html/HTMLTableCaptionElement.h"
 #include "core/html/HTMLTableCellElement.h"
 #include "core/html/HTMLTableElement.h"
 #include "core/html/HTMLTableRowElement.h"
 #include "core/html/HTMLTableSectionElement.h"
 #include "core/html/HTMLTextAreaElement.h"
 #include "core/html/parser/HTMLParserIdioms.h"
 #include "core/html/shadow/MediaControlElements.h"
 #include "core/layout/LayoutObject.h"
 #include "modules/accessibility/AXObjectCacheImpl.h"
 #include "platform/UserGestureIndicator.h"
+#include "platform/text/PlatformLocale.h"
 #include "wtf/text/StringBuilder.h"
 
 
 namespace blink {
 
 using namespace HTMLNames;
 
 AXNodeObject::AXNodeObject(Node* node, AXObjectCacheImpl& axObjectCache)
     : AXObject(axObjectCache)
     , m_ariaRole(UnknownRole)
@@ skipping 176 matching lines @@
 static bool isRequiredOwnedElement(AXObject* parent, AccessibilityRole currentRole, HTMLElement* currentElement)
 {
     Node* parentNode = parent->node();
     if (!parentNode || !parentNode->isHTMLElement())
         return false;
 
     if (currentRole == ListItemRole)
         return isListElement(parentNode);
     if (currentRole == ListMarkerRole)
         return isHTMLLIElement(*parentNode);
-    if (currentRole == MenuItemCheckBoxRole || currentRole ==  MenuItemRole || currentRole ==  MenuItemRadioRole)
+    if (currentRole == MenuItemCheckBoxRole || currentRole == MenuItemRole || currentRole == MenuItemRadioRole)
         return isHTMLMenuElement(*parentNode);
 
     if (!currentElement)
         return false;
     if (isHTMLTableCellElement(*currentElement))
         return isHTMLTableRowElement(*parentNode);
     if (isHTMLTableRowElement(*currentElement))
         return isHTMLTableSectionElement(toHTMLElement(*parentNode));
 
     // In case of ListboxRole and it's child, ListBoxOptionRole,
     // Inheritance of presentation role is handled in AXListBoxOption
     // Because ListBoxOption Role doesn't have any child.
     // If it's just ignored because of presentation, we can't see any AX tree related to ListBoxOption.
     return false;
 }
 
-
 const AXObject* AXNodeObject::inheritsPresentationalRoleFrom() const
 {
     // ARIA states if an item can get focus, it should not be presentational.
     if (canSetFocusAttribute())
         return 0;
 
     if (isPresentational())
         return this;
 
     // http://www.w3.org/TR/wai-aria/complete#presentation
@@ skipping 48 matching lines @@
     if (node()->isLink() || (isHTMLAnchorElement(*node()) && isClickable()))
         return LinkRole;
 
     if (isHTMLButtonElement(*node()))
         return buttonRoleType();
 
     if (isHTMLDetailsElement(*node()))
         return DetailsRole;
 
     if (isHTMLSummaryElement(*node())) {
-        if (node()->parentNode() && isHTMLDetailsElement(node()->parentNode()))
+        if (ComposedTreeTraversal::parent(*node()) && isHTMLDetailsElement(ComposedTreeTraversal::parent(*node())))
             return DisclosureTriangleRole;
         return UnknownRole;
     }
 
     if (isHTMLInputElement(*node())) {
         HTMLInputElement& input = toHTMLInputElement(*node());
         const AtomicString& type = input.type();
         if (input.dataList())
             return ComboBoxRole;
         if (type == InputTypeNames::button) {
@@ skipping 1421 matching lines @@
             return placeholder;
     }
 
     return String();
 }
 
 //
 // New AX name calculation.
 //
 
-String AXNodeObject::textAlternative(bool recursive, bool inAriaLabelledByTraversal, AXObjectSet& visited, AXNameFrom& nameFrom, AXObjectVector& nameObjects, NameSources* nameSources) const
+String AXNodeObject::textAlternative(bool recursive, bool inAriaLabelledByTraversal, AXObjectSet& visited, AXNameFrom& nameFrom, AXObjectVector* nameObjects, NameSources* nameSources) const
 {
+    // If nameSources is non-null, nameObjects is used in filling it in, so it must be non-null as well.
+    if (nameSources)
+        assert(nameObjects);
+
     bool alreadyVisited = visited.contains(this);
     bool foundTextAlternative = false;
     visited.add(this);
     String textAlternative;
 
     if (!node() && !layoutObject())
         return String();
 
     // Step 2A from: http://www.w3.org/TR/accname-aam-1.1
     if (!recursive && layoutObject()
@@ skipping 14 matching lines @@
         if (hasAttribute(attr)) {
             if (nameSources)
                 nameSources->last().attributeValue = getAttribute(attr);
 
             textAlternative = textFromAriaLabelledby(visited, nameObjects);
 
             if (!textAlternative.isNull()) {
                 if (nameSources) {
                     NameSource& source = nameSources->last();
                     source.type = nameFrom;
-                    source.nameObjects = nameObjects;
+                    source.nameObjects = *nameObjects;
                     source.text = textAlternative;
                     foundTextAlternative = true;
                 } else {
                     return textAlternative;
                 }
             } else if (nameSources) {
                 nameSources->last().invalid = true;
             }
         }
     }
@@ skipping 19 matching lines @@
             }
         }
     }
 
     // Step 2D from: http://www.w3.org/TR/accname-aam-1.1
     textAlternative = nativeTextAlternative(visited, nameFrom, nameObjects, nameSources, &foundTextAlternative);
     if (!textAlternative.isNull() && !nameSources)
         return textAlternative;
 
     // Step 2E from: http://www.w3.org/TR/accname-aam-1.1
-
+    if (recursive && !inAriaLabelledByTraversal && isControl()) {
+        // No need to set any name source info in a recursive call.
+        if (roleValue() == TextFieldRole || roleValue() == ComboBoxRole)
+            return text();
+        if (isRange()) {
+            if (hasAttribute(aria_valuetextAttr))
+                return getAttribute(aria_valuetextAttr).string();
+            return String::number(valueForRange());
+        }
+    }
 
     // Step 2F / 2G from: http://www.w3.org/TR/accname-aam-1.1
     if (recursive || nameFromContents()) {
         nameFrom = AXNameFromContents;
         if (nameSources) {
             nameSources->append(NameSource(foundTextAlternative));
             nameSources->last().type = nameFrom;
         }
 
         Node* node = this->node();
         if (node && node->isTextNode())
             textAlternative = toText(node)->wholeText();
         else
             textAlternative = textFromDescendants(visited);
 
-        if (nameSources) {
-            foundTextAlternative = true;
-            nameSources->last().text = textAlternative;
-        } else {
-            return textAlternative;
+        if (!textAlternative.isEmpty()) {
+            if (nameSources) {
+                foundTextAlternative = true;
+                nameSources->last().text = textAlternative;
+            } else {
+                return textAlternative;
+            }
+        }
+    }
+
+    // Step 2H from: http://www.w3.org/TR/accname-aam-1.1
+    nameFrom = AXNameFromAttribute;
+    if (nameSources) {
+        nameSources->append(NameSource(foundTextAlternative, titleAttr));
+        nameSources->last().type = nameFrom;
+    }
+    if (hasAttribute(titleAttr)) {
+        const AtomicString& title = getAttribute(titleAttr);
+        if (!title.isEmpty()) {
+            textAlternative = title;
+            if (nameSources) {
+                foundTextAlternative = true;
+                nameSources->last().text = textAlternative;
+            } else {
+                return textAlternative;
+            }
         }
     }
 
     nameFrom = AXNameFromUninitialized;
 
-    if (nameSources && !nameSources->isEmpty()) {
+    if (foundTextAlternative) {
         for (size_t i = 0; i < nameSources->size(); ++i) {
             if (!(*nameSources)[i].text.isNull() && !(*nameSources)[i].superseded) {
-                nameFrom = (*nameSources)[i].type;
-                nameObjects = (*nameSources)[i].nameObjects;
-                return (*nameSources)[i].text;
+                NameSource& nameSource = (*nameSources)[i];
+                nameFrom = nameSource.type;
+                if (!nameSource.nameObjects.isEmpty())
+                    *nameObjects = nameSource.nameObjects;
+                return nameSource.text;
             }
         }
     }
 
     return String();
 }
 
 String AXNodeObject::textFromDescendants(AXObjectSet& visited) const
 {
     StringBuilder accumulatedText;
     AXObject* previous = nullptr;
     for (AXObject* child = firstChild(); child; child = child->nextSibling()) {
         // If we're going between two layoutObjects that are in separate LayoutBoxes, add
         // whitespace if it wasn't there already. Intuitively if you have
         // <span>Hello</span><span>World</span>, those are part of the same LayoutBox
         // so we should return "HelloWorld", but given <div>Hello</div><div>World</div> the
         // strings are in separate boxes so we should return "Hello World".
         if (previous && accumulatedText.length() && !isHTMLSpace(accumulatedText[accumulatedText.length() - 1])) {
             if (!isSameLayoutBox(child->layoutObject(), previous->layoutObject()))
                 accumulatedText.append(' ');
         }
 
-        AXNameFrom nameFrom;
-        AXObjectVector nameObjects;
-        String result = child->textAlternative(true, false, visited, nameFrom, nameObjects, nullptr);
+        String result = recursiveTextAlternative(*child, false, visited);
         accumulatedText.append(result);
         previous = child;
     }
 
     return accumulatedText.toString();
 }
 
-String AXNodeObject::textFromElements(bool inAriaLabelledbyTraversal, AXObjectSet& visited, WillBeHeapVector<RawPtrWillBeMember<Element>>& elements, AXObjectVector& nameObjects) const
+String AXNodeObject::textFromElements(bool inAriaLabelledbyTraversal, AXObjectSet& visited, WillBeHeapVector<RawPtrWillBeMember<Element>>& elements, AXObjectVector* nameObjects) const
 {
     StringBuilder accumulatedText;
     bool foundValidElement = false;
     AXObjectVector localNameObjects;
 
     for (const auto& element : elements) {
         RefPtrWillBeRawPtr<AXObject> axElement = axObjectCache().getOrCreate(element);
         if (axElement) {
             foundValidElement = true;
             localNameObjects.append(axElement.get());
 
             String result = recursiveTextAlternative(*axElement, inAriaLabelledbyTraversal, visited);
             if (!result.isEmpty()) {
                 if (!accumulatedText.isEmpty())
                     accumulatedText.append(" ");
                 accumulatedText.append(result);
             }
         }
     }
     if (!foundValidElement)
         return String();
-    nameObjects = localNameObjects;
+    if (nameObjects)
+        *nameObjects = localNameObjects;
     return accumulatedText.toString();
 }
 
-String AXNodeObject::textFromAriaLabelledby(AXObjectSet& visited, AXObjectVector& nameObjects) const
+String AXNodeObject::textFromAriaLabelledby(AXObjectSet& visited, AXObjectVector* nameObjects) const
 {
     WillBeHeapVector<RawPtrWillBeMember<Element>> elements;
     ariaLabelledbyElements(elements);
     return textFromElements(true, visited, elements, nameObjects);
 }
 
 LayoutRect AXNodeObject::elementRect() const
 {
     // First check if it has a custom rect, for example if this element is tied to a canvas path.
     if (!m_explicitElementRect.isEmpty())
@@ skipping 182 matching lines @@
 }
 
 Element* AXNodeObject::actionElement() const
 {
     Node* node = this->node();
     if (!node)
         return 0;
 
     if (isHTMLInputElement(*node)) {
         HTMLInputElement& input = toHTMLInputElement(*node);
-        if (!input.isDisabledFormControl() && (isCheckboxOrRadio() || input.isTextButton()
-            || input.type() == InputTypeNames::file))
+        if (!input.isDisabledFormControl() && (isCheckboxOrRadio() || input.isTextButton() || input.type() == InputTypeNames::file))
             return &input;
     } else if (isHTMLButtonElement(*node)) {
         return toElement(node);
     }
 
     if (AXObject::isARIAInput(ariaRoleAttribute()))
         return toElement(node);
 
     if (isImageButton())
         return toElement(node);
@@ skipping 285 matching lines @@
         ariaLabelledbyElements(elements);
 
         for (const auto& element : elements) {
             RefPtrWillBeRawPtr<AXObject> axElement = axObjectCache().getOrCreate(element);
             textOrder.append(AccessibilityText::create(ariaLabelledby, AlternativeText, axElement));
         }
     }
 }
 
 // Based on http://rawgit.com/w3c/aria/master/html-aam/html-aam.html#accessible-name-and-description-calculation
-String AXNodeObject::nativeTextAlternative(AXObjectSet& visited, AXNameFrom& nameFrom, AXObjectVector& nameObjects, NameSources* nameSources, bool* foundTextAlternative) const
+String AXNodeObject::nativeTextAlternative(AXObjectSet& visited, AXNameFrom& nameFrom, AXObjectVector* nameObjects, NameSources* nameSources, bool* foundTextAlternative) const
 {
     if (!node())
         return String();
 
+    // If nameSources is non-null, nameObjects is used in filling it in, so it must be non-null as well.
+    if (nameSources)
+        assert(nameObjects);
+
     String textAlternative;
     AXObjectVector localNameObjects;
 
     const HTMLInputElement* inputElement = nullptr;
     if (isHTMLInputElement(node()))
         inputElement = toHTMLInputElement(node());
 
+    // 5.1/5.5 Text inputs, Other labelable Elements
+    HTMLElement* htmlElement = nullptr;
+    if (node()->isHTMLElement())
+        htmlElement = toHTMLElement(node());
+    if (htmlElement && htmlElement->isLabelable()) {
+        // label
+        nameFrom = AXNameFromRelatedElement;
+        if (nameSources) {
+            nameSources->append(NameSource(*foundTextAlternative));
+            nameSources->last().type = nameFrom;
+            nameSources->last().nativeSource = AXTextFromNativeHTMLLabel;
+        }
+        HTMLLabelElement* label = labelForElement(htmlElement);
+        if (label) {
+            RefPtrWillBeRawPtr<AXObject> labelAXObject = axObjectCache().getOrCreate(label);
+            if (labelAXObject) {
+                if (nameObjects) {
+                    localNameObjects.append(labelAXObject.get());
+                    *nameObjects = localNameObjects;
+                    localNameObjects.clear();
+                }
+                textAlternative = recursiveTextAlternative(*labelAXObject, false, visited);
+
+                if (nameSources) {
+                    NameSource& source = nameSources->last();
+                    source.nameObjects = *nameObjects;
+                    source.text = textAlternative;
+                    if (label->getAttribute(forAttr).isNull())
+                        source.nativeSource = AXTextFromNativeHTMLLabelWrapped;
+                    else
+                        source.nativeSource = AXTextFromNativeHTMLLabelFor;
+                    *foundTextAlternative = true;
+                } else {
+                    return textAlternative;
+                }
+            }
+        }
+    }
+
     // 5.2 input type="button", input type="submit" and input type="reset"
     if (inputElement && inputElement->isTextButton()) {
         // value attribue
         nameFrom = AXNameFromAttribute;
         if (nameSources) {
             nameSources->append(NameSource(*foundTextAlternative, valueAttr));
             nameSources->last().type = nameFrom;
         }
         String value = inputElement->value();
         if (!value.isNull()) {
             textAlternative = value;
             if (nameSources) {
                 NameSource& source = nameSources->last();
                 source.text = textAlternative;
                 *foundTextAlternative = true;
             } else {
                 return textAlternative;
             }
         }
-
-        // localised default value ("Submit" or "Reset")
-        String defaultValue = inputElement->defaultValue();
-        if (!defaultValue.isNull()) {
-            // defaultValue is always set for submit and reset buttons, and never set for anything else.
-            nameFrom = AXNameFromAttribute;
-            textAlternative = defaultValue;
-            if (nameSources) {
-                nameSources->append(NameSource(*foundTextAlternative, typeAttr));
-                NameSource& source = nameSources->last();
-                source.attributeValue = inputElement->getAttribute(typeAttr);
-                source.text = textAlternative;
-                *foundTextAlternative = true;
-            } else {
-                return textAlternative;
-            }
-        }
         return textAlternative;
     }
 
     // 5.3 input type="image"
     if (inputElement && inputElement->getAttribute(typeAttr) == InputTypeNames::image) {
         // alt attr
         nameFrom = AXNameFromAttribute;
         if (nameSources) {
             nameSources->append(NameSource(*foundTextAlternative, altAttr));
             nameSources->last().type = nameFrom;
@@ skipping 22 matching lines @@
             textAlternative = value;
             if (nameSources) {
                 NameSource& source = nameSources->last();
                 source.text = textAlternative;
                 *foundTextAlternative = true;
             } else {
                 return textAlternative;
             }
         }
 
+        // localised default value ("Submit")

[dmazzoni, 2015/08/24 18:02:03]

Oh, got it - <input type=image> implies a Submit button by default.

+        nameFrom = AXNameFromAttribute;
+        textAlternative = inputElement->locale().queryString(WebLocalizedString::SubmitButtonDefaultLabel);
+        if (nameSources) {
+            nameSources->append(NameSource(*foundTextAlternative, typeAttr));
+            NameSource& source = nameSources->last();
+            source.attributeValue = inputElement->getAttribute(typeAttr);
+            source.type = nameFrom;
+            source.text = textAlternative;
+            *foundTextAlternative = true;
+        } else {
+            return textAlternative;
+        }
         return textAlternative;
     }
 
-    // 5.5 Other labelable Elements
-    if (node()->isHTMLElement() && toHTMLElement(node())->isLabelable()) {
-        // label
-        nameFrom = AXNameFromRelatedElement;
+    // 5.1 Text inputs - step 3 (placeholder attribute)
+    if (htmlElement && htmlElement->isTextFormControl()) {
+        nameFrom = AXNameFromPlaceholder;
         if (nameSources) {
-            nameSources->append(NameSource(*foundTextAlternative));
-            nameSources->last().type = nameFrom;
-            nameSources->last().nativeSource = AXTextFromNativeHTMLLabel;
+            nameSources->append(NameSource(*foundTextAlternative, placeholderAttr));
+            NameSource& source = nameSources->last();
+            source.type = nameFrom;
         }
-        HTMLLabelElement* label = labelForElement(toHTMLElement(node()));
-        if (label) {
-            RefPtrWillBeRawPtr<AXObject> labelAXObject = axObjectCache().getOrCreate(label);
-            if (labelAXObject) {
-                localNameObjects.append(labelAXObject.get());
-                nameObjects = localNameObjects;
-                localNameObjects.clear();
-
-                textAlternative = recursiveTextAlternative(*labelAXObject, false, visited);
-
+        HTMLElement* element = toHTMLElement(node());
+        if (element->fastHasAttribute(placeholderAttr)) {
+            const AtomicString& placeholder = element->fastGetAttribute(placeholderAttr);
+            if (!placeholder.isEmpty()) {
+                textAlternative = placeholder;
                 if (nameSources) {
                     NameSource& source = nameSources->last();
-                    source.nameObjects = nameObjects;
                     source.text = textAlternative;
-                    if (label->getAttribute(forAttr).isNull())
-                        source.nativeSource = AXTextFromNativeHTMLLabelWrapped;
-                    else
-                        source.nativeSource = AXTextFromNativeHTMLLabelFor;
-                    *foundTextAlternative = true;
+                    source.attributeValue = placeholder;
                 } else {
                     return textAlternative;
                 }
             }
         }
         return textAlternative;
     }
 
     // 5.7 figure and figcaption Elements
     if (node()->hasTagName(figureTag)) {
         // figcaption
         nameFrom = AXNameFromRelatedElement;
         if (nameSources) {
             nameSources->append(NameSource(*foundTextAlternative));
             nameSources->last().type = nameFrom;
             nameSources->last().nativeSource = AXTextFromNativeHTMLFigcaption;
         }
         Element* figcaption = nullptr;
         for (Element& element : ElementTraversal::descendantsOf(*(node()))) {
             if (element.hasTagName(figcaptionTag)) {
                 figcaption = &element;
                 break;
             }
         }
         if (figcaption) {
             RefPtrWillBeRawPtr<AXObject> figcaptionAXObject = axObjectCache().getOrCreate(figcaption);
             if (figcaptionAXObject) {
-                localNameObjects.append(figcaptionAXObject.get());
-                nameObjects = localNameObjects;
-                localNameObjects.clear();
+                if (nameObjects) {
+                    localNameObjects.append(figcaptionAXObject.get());
+                    *nameObjects = localNameObjects;
+                    localNameObjects.clear();
+                }
 
                 textAlternative = recursiveTextAlternative(*figcaptionAXObject, false, visited);
 
                 if (nameSources) {
                     NameSource& source = nameSources->last();
-                    source.nameObjects = nameObjects;
+                    source.nameObjects = *nameObjects;
                     source.text = textAlternative;
                 } else {
                     return textAlternative;
                 }
             }
         }
         return textAlternative;
     }
 
     // 5.8 img Element
     if (isHTMLImageElement(node())) {
-        HTMLImageElement* imgElement = toHTMLImageElement(node());
-
         // alt
         nameFrom = AXNameFromAttribute;
         if (nameSources) {
             nameSources->append(NameSource(*foundTextAlternative, altAttr));
             nameSources->last().type = nameFrom;
         }
-        if (imgElement->hasAttribute(altAttr)) {
-            AtomicString alt = imgElement->getAttribute(altAttr);
+        if (hasAttribute(altAttr)) {
+            AtomicString alt = getAttribute(altAttr);
             textAlternative = alt;
             if (nameSources) {
                 NameSource& source = nameSources->last();
                 source.attributeValue = alt;
                 source.text = textAlternative;
                 *foundTextAlternative = true;
             } else {
                 return textAlternative;
             }
         }
         return textAlternative;
     }
 
     // 5.9 table Element
     if (isHTMLTableElement(node())) {
         HTMLTableElement* tableElement = toHTMLTableElement(node());
 
         // caption
         nameFrom = AXNameFromRelatedElement;
         if (nameSources) {
             nameSources->append(NameSource(*foundTextAlternative));
             nameSources->last().type = nameFrom;
             nameSources->last().nativeSource = AXTextFromNativeHTMLTableCaption;
         }
         HTMLTableCaptionElement* caption = tableElement->caption();
         if (caption) {
             RefPtrWillBeRawPtr<AXObject> captionAXObject = axObjectCache().getOrCreate(caption);
             if (captionAXObject) {
-                localNameObjects.append(captionAXObject.get());
-                nameObjects = localNameObjects;
-                localNameObjects.clear();
+                if (nameObjects) {
+                    localNameObjects.append(captionAXObject.get());
+                    *nameObjects = localNameObjects;
+                    localNameObjects.clear();
+                }
 
                 textAlternative = recursiveTextAlternative(*captionAXObject, false, visited);
                 if (nameSources) {
                     NameSource& source = nameSources->last();
-                    source.nameObjects = nameObjects;
+                    source.nameObjects = *nameObjects;
                     source.text = textAlternative;
                 } else {
                     return textAlternative;
                 }
             }
         }
+
+        // summary
+        nameFrom = AXNameFromAttribute;
+        if (nameSources) {
+            nameSources->append(NameSource(*foundTextAlternative));
+            nameSources->last().type = nameFrom;
+        }
+        if (hasAttribute(summaryAttr)) {
+            AtomicString summary = getAttribute(summaryAttr);
+            textAlternative = summary;
+            if (nameSources) {
+                NameSource& source = nameSources->last();
+                source.attributeValue = summary;
+                source.text = textAlternative;
+                *foundTextAlternative = true;
+            } else {
+                return textAlternative;
+            }
+        }
+
         return textAlternative;
     }
 
     return textAlternative;
 }
 
 DEFINE_TRACE(AXNodeObject)
 {
     visitor->trace(m_node);
     AXObject::trace(visitor);
 }
 
 } // namespace blink