AX: Remove dead code in AccessibilityRenderObject::textUnderElement
authorfred.wang@free.fr <fred.wang@free.fr@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 28 Jun 2016 14:16:36 +0000 (14:16 +0000)
committerfred.wang@free.fr <fred.wang@free.fr@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 28 Jun 2016 14:16:36 +0000 (14:16 +0000)
https://bugs.webkit.org/show_bug.cgi?id=159205

Patch by Frederic Wang <fwang@igalia.com> on 2016-06-28
Reviewed by Joanmarie Diggs.

RenderMathMLOperator used to destroy its descendants and to replace them with an anonymous
text node wrapped inside anonymous blocks. After r202420, it just behaves as any other token
elements. Hence we remove the code in AccessibilityRenderObject::textUnderElement that was
used to handle this specific render tree structure.

No new tests, already covered by accessibility/math-text.html.

* accessibility/AccessibilityRenderObject.cpp:
(WebCore::AccessibilityRenderObject::textUnderElement): Remove dead code for RenderText and RenderMathMLOperator.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@202562 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/accessibility/AccessibilityRenderObject.cpp

index fb3ccd9..0a3bc6e 100644 (file)
@@ -1,3 +1,20 @@
+2016-06-28  Frederic Wang  <fwang@igalia.com>
+
+        AX: Remove dead code in AccessibilityRenderObject::textUnderElement
+        https://bugs.webkit.org/show_bug.cgi?id=159205
+
+        Reviewed by Joanmarie Diggs.
+
+        RenderMathMLOperator used to destroy its descendants and to replace them with an anonymous
+        text node wrapped inside anonymous blocks. After r202420, it just behaves as any other token
+        elements. Hence we remove the code in AccessibilityRenderObject::textUnderElement that was
+        used to handle this specific render tree structure.
+
+        No new tests, already covered by accessibility/math-text.html.
+
+        * accessibility/AccessibilityRenderObject.cpp:
+        (WebCore::AccessibilityRenderObject::textUnderElement): Remove dead code for RenderText and RenderMathMLOperator.
+
 2016-06-28  Per Arne Vollan  <pvollan@apple.com>
 
         [Win] Custom elements tests are failing.
index 01e9ea7..6e16ae2 100644 (file)
@@ -632,16 +632,10 @@ String AccessibilityRenderObject::textUnderElement(AccessibilityTextUnderElement
     bool isRenderText = is<RenderText>(*m_renderer);
 
 #if ENABLE(MATHML)
-    // Math operators create RenderText nodes on the fly that are not tied into the DOM in a reasonable way,
-    // so rangeOfContents does not work for them (nor does regular text selection).
-    if (isRenderText && m_renderer->isAnonymous() && ancestorsOfType<RenderMathMLOperator>(*m_renderer).first())
-        return downcast<RenderText>(*m_renderer).text();
     if (isAnonymousMathOperator()) {
         UChar operatorChar = downcast<RenderMathMLOperator>(*m_renderer).textContent();
         return operatorChar ? String(&operatorChar, 1) : String();
     }
-    if (is<RenderMathMLOperator>(*m_renderer) && !m_renderer->isAnonymous())
-        return downcast<RenderMathMLOperator>(*m_renderer).element().textContent();
 #endif
 
     if (shouldGetTextFromNode(mode))