Math tag is not exposed in Accessibility.

Math tag is not exposed in Accessibility.

This CL will return MathRole from blink to chromium for math tag.

BUG=429476
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=185264

[shreeramk, 2014-11-12 15:53:10]

shreeram.k@samsung.com changed reviewers:
+ dmazzoni@chromium.org

[shreeramk, 2014-11-12 15:53:11]

Please review.

Thanks :)

[dmazzoni, 2014-11-12 19:42:34]

The CQ bit was checked by dmazzoni@chromium.org

[dmazzoni, 2014-11-12 19:42:35]

lgtm

[commit-bot: I haz the power, 2014-11-12 19:43:23]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/722653002/1

[commit-bot: I haz the power, 2014-11-12 19:44:04]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-11-12 19:44:04]

Failed to apply patch for Source/core/accessibility/AXRenderObject.cpp:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  can't find file to patch at input line 6
  Perhaps you used the wrong -p or --strip option?
  The text leading up to this was:
  --------------------------
  |Index: Source/core/accessibility/AXRenderObject.cpp
  |diff --git a/Source/core/accessibility/AXRenderObject.cpp
b/Source/core/accessibility/AXRenderObject.cpp
  |index
d048fddaba7d127b13ceeb98bf717576fdb6aac4..6c36e961cd3384057c026a0396cb95d1121f726f
100644
  |--- a/Source/core/accessibility/AXRenderObject.cpp
  |+++ b/Source/core/accessibility/AXRenderObject.cpp
  --------------------------
  No file to patch.  Skipping patch.
  2 out of 2 hunks ignored

Patch:       Source/core/accessibility/AXRenderObject.cpp
Index: Source/core/accessibility/AXRenderObject.cpp
diff --git a/Source/core/accessibility/AXRenderObject.cpp
b/Source/core/accessibility/AXRenderObject.cpp
index
d048fddaba7d127b13ceeb98bf717576fdb6aac4..6c36e961cd3384057c026a0396cb95d1121f726f
100644
--- a/Source/core/accessibility/AXRenderObject.cpp
+++ b/Source/core/accessibility/AXRenderObject.cpp
@@ -368,6 +368,9 @@ AccessibilityRole
AXRenderObject::determineAccessibilityRole()
     if (node && node->hasTagName(dtTag))
         return DescriptionListTermRole;
 
+    if (node && (node->nodeName() == "math"))
+        return MathRole;
+
     if (node && (node->hasTagName(rpTag) || node->hasTagName(rtTag)))
         return AnnotationRole;
 
@@ -736,6 +739,9 @@ bool AXRenderObject::computeAccessibilityIsIgnored() const
     if (roleValue() == DetailsRole)
         return false;
 
+    if (roleValue() == MathRole)
+        return false;
+
     if (roleValue() == MeterRole)
         return false;

[shreeramk, 2014-11-13 03:13:55]

The CQ bit was checked by shreeram.k@samsung.com

[commit-bot: I haz the power, 2014-11-13 03:14:32]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/722653002/20001

[commit-bot: I haz the power, 2014-11-13 04:17:32]

Committed patchset #2 (id:20001) as 185264