Fall back to title element text for SVG root text alternatives

third_party/WebKit/Source/modules/accessibility/AXNodeObject.cpp

Index: third_party/WebKit/Source/modules/accessibility/AXNodeObject.cpp
diff --git a/third_party/WebKit/Source/modules/accessibility/AXNodeObject.cpp b/third_party/WebKit/Source/modules/accessibility/AXNodeObject.cpp
index 96bdd2579a833fc65c58ac72478832d680bde08a..ad039772217754bf79bbeb494460b4faa95a20f4 100644
--- a/third_party/WebKit/Source/modules/accessibility/AXNodeObject.cpp
+++ b/third_party/WebKit/Source/modules/accessibility/AXNodeObject.cpp
@@ -55,6 +55,7 @@
 #include "core/html/shadow/MediaControlElements.h"
 #include "core/layout/LayoutBlockFlow.h"
 #include "core/layout/LayoutObject.h"
+#include "core/svg/SVGElement.h"
 #include "modules/accessibility/AXObjectCacheImpl.h"
 #include "platform/UserGestureIndicator.h"
 #include "platform/text/PlatformLocale.h"
@@ -2369,6 +2370,34 @@ String AXNodeObject::nativeTextAlternative(AXObjectSet& visited, AXNameFrom& nam
         return textAlternative;
     }
 
+    // Per SVG AAM 1.0's modifications to 2D of this algorithm.
+    if (node()->isSVGElement()) {
+        Element* title = nullptr;

[aboxhall, 2015/12/15 19:56:03]

Before this line, add:

nameFrom = AXNameFromRelatedElement;
if (nameSources) {
    nameSources->append(NameSource(*foundTextAlternative));
    nameSources->last().type = nameFrom;
    nameSources->last().nativeSource = AXTextFromNativeHTMLTitleElement;   //
will need to update test expectation as well
}

Then you don't need line 2388 or 2391 - and you don't need line 2389 either
because it's not an attribute, but a related element (details/summary and
fieldset/legend use a similar pattern, see below).

The reason for initializing the nameSource first up is that nameSources tracks
_potential_ as well as actual name sources, so we add in the nameSource
"optimistically" and then fill it in if we actually find something. And we set
nameFrom at the same time because nameSources may be null (and the code will
potentially early-out much sooner if it is), but nameFrom is used outside it.

+        for (Element& element : ElementTraversal::descendantsOf(*(node()))) {
+            if (element.hasTagName(SVGNames::titleTag)) {
+                title = &element;
+                break;
+            }
+        }
+
+        if (title) {
+            String text = title->innerText();
+            textAlternative = text;
+            if (nameSources) {
+                NameSource& source = nameSources->last();
+                source.type = AXNameFromTitle;
+                source.attributeValue = AtomicString(text);
+                source.text = textAlternative;
+                source.nativeSource = AXTextFromNativeHTMLTitleElement;
+                *foundTextAlternative = true;
+            } else {
+                return textAlternative;
+            }
+        }
+
+        return textAlternative;
+    }
+
     // Fieldset / legend.
     if (isHTMLFieldSetElement(node())) {
         nameFrom = AXNameFromRelatedElement;