img tag should have xml-role as img.

According to specs img tag should have xml-role as img. BUG=417230 Review URL: https://codereview.chromium.org/592353004 Cr-Commit-Position: refs/heads/master@{#296910}

img tag should have xml-role as img.
According to specs img tag should have xml-role as img. BUG=417230 Review URL: https://codereview.chromium.org/592353004 Cr-Commit-Position: refs/heads/master@{#296910}
3f09a820 · shreeram.k · Commit bot · 1e4f66db · 3f09a820 · 3f09a820
Commit 3f09a820 authored Sep 26, 2014 by shreeram.k Committed by Commit bot Sep 26, 2014
6 changed files
--- a/content/browser/accessibility/dump_accessibility_tree_browsertest.cc
+++ b/content/browser/accessibility/dump_accessibility_tree_browsertest.cc
@@ -573,6 +573,10 @@ IN_PROC_BROWSER_TEST_F(DumpAccessibilityTreeTest,
  RunTest(FILE_PATH_LITERAL("iframe-coordinates.html"));
 }

+IN_PROC_BROWSER_TEST_F(DumpAccessibilityTreeTest, AccessibilityImg) {
+  RunTest(FILE_PATH_LITERAL("img.html"));
+}
+
 IN_PROC_BROWSER_TEST_F(DumpAccessibilityTreeTest, AccessibilityInputButton) {
  RunTest(FILE_PATH_LITERAL("input-button.html"));
 }

--- a/content/renderer/accessibility/blink_ax_tree_source.cc
+++ b/content/renderer/accessibility/blink_ax_tree_source.cc
@@ -75,6 +75,8 @@ std::string GetEquivalentAriaRoleString(const ui::AXRole role) {
    case ui::AX_ROLE_CONTENT_INFO:
    case ui::AX_ROLE_FOOTER:
      return "contentinfo";
+    case ui::AX_ROLE_IMAGE:
+      return "img";
    case ui::AX_ROLE_MAIN:
      return "main";
    case ui::AX_ROLE_NAVIGATION:

--- a/content/test/data/accessibility/img-expected-android.txt
+++ b/content/test/data/accessibility/img-expected-android.txt
+android.webkit.WebView focusable focused scrollable
+    android.view.View
+        android.widget.Image clickable name='pipe'
--- a/content/test/data/accessibility/img-expected-mac.txt
+++ b/content/test/data/accessibility/img-expected-mac.txt
+AXWebArea
+    AXGroup
+        AXImage AXRoleDescription='image'
--- a/content/test/data/accessibility/img-expected-win.txt
+++ b/content/test/data/accessibility/img-expected-win.txt
+ROLE_SYSTEM_DOCUMENT READONLY FOCUSABLE
+    IA2_ROLE_SECTION READONLY
+        ROLE_SYSTEM_GRAPHIC name='pipe' READONLY xml-roles:img
--- a/content/test/data/accessibility/img.html
+++ b/content/test/data/accessibility/img.html
+<!--
+@MAC-ALLOW:AXRoleDescription='image'
+@WIN-ALLOW:xml-roles:*
+-->
+<html>
+<body>
+  <img src="pipe.jpg" alt="pipe">
+  <img src="pipe.jpg" alt="">
+  <img src="pipe.jpg" alt=" ">
+</body>
+</html>