Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(7)

Side by Side Diff: third_party/WebKit/LayoutTests/accessibility/presentation-owned-elements.html

Issue 2827243002: Change default style of menu[type=context] (Closed)
Patch Set: Fix failing tests Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 <script src="../resources/js-test.js"></script> 1 <script src="../resources/js-test.js"></script>
2 <script src="../resources/accessibility-helper.js"></script> 2 <script src="../resources/accessibility-helper.js"></script>
3 <div>These lists have a static text role because they have a presentation role.< /div> 3 <div>These lists have a static text role because they have a presentation role.< /div>
4 <ul role="presentation"> 4 <ul role="presentation">
5 <li>Presentation Item 1</li> 5 <li>Presentation Item 1</li>
6 <li>Presentation Item 2</li> 6 <li>Presentation Item 2</li>
7 </ul> 7 </ul>
8 8
9 <div>The first list item has a listitem role even if ul has a presentation role because it has an explicit role.</div> 9 <div>The first list item has a listitem role even if ul has a presentation role because it has an explicit role.</div>
10 <ul role="presentation"> 10 <ul role="presentation">
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 <tr role="row"> 47 <tr role="row">
48 <th>Explicit th</th> 48 <th>Explicit th</th>
49 </tr> 49 </tr>
50 <tr> 50 <tr>
51 <td>Implicit td</td> 51 <td>Implicit td</td>
52 </tr> 52 </tr>
53 </table> 53 </table>
54 54
55 <div>The menu items except button have a static text role because it has a prese ntation role and is disabled.<br> 55 <div>The menu items except button have a static text role because it has a prese ntation role and is disabled.<br>
56 Button has a button role because it has an explicit role.</div> 56 Button has a button role because it has an explicit role.</div>
57 <menu type="context" role="presentation"> 57 <menu type="toolbar" role="presentation">
tkent 2017/04/24 05:20:18 I'm not sure if this change is safe. +je_julie.kim
dmazzoni 2017/04/24 15:54:50 This seems fine. At the time the test was written,
58 <input type="checkbox" name="vehicle" value="Bike" disabled> I have a bike 58 <input type="checkbox" name="vehicle" value="Bike" disabled> I have a bike
59 <input type="button" role="button" value="Click me" disabled> 59 <input type="button" role="button" value="Click me" disabled>
60 <input type="radio" name="gender" value="male" disabled> Male 60 <input type="radio" name="gender" value="male" disabled> Male
61 <input type="radio" name="gender" value="female" disabled> Female 61 <input type="radio" name="gender" value="female" disabled> Female
62 </menu> 62 </menu>
63 63
64 <div>The option A has a static text role because it has a presentation role and is disabled.<br> 64 <div>The option A has a static text role because it has a presentation role and is disabled.<br>
65 The option B has option role because it has an explicit role.</div> 65 The option B has option role because it has an explicit role.</div>
66 <div role="group"> 66 <div role="group">
67 <select multiple role="presentation" disabled> 67 <select multiple role="presentation" disabled>
68 <option value="A" disabled>A</option> 68 <option value="A" disabled>A</option>
69 <option value="B" disabled role="option">B</option> 69 <option value="B" disabled role="option">B</option>
70 </select> 70 </select>
71 </div> 71 </div>
72 72
73 <p>End of test</p> 73 <p>End of test</p>
74 <p id="description"></p> 74 <p id="description"></p>
75 <div id="console"></div> 75 <div id="console"></div>
76 <script> 76 <script>
77 description("This test make sure that presentation role is inherited to required owned elements."); 77 description("This test make sure that presentation role is inherited to required owned elements.");
78 78
79 if (window.testRunner) 79 if (window.testRunner)
80 testRunner.dumpAsText(); 80 testRunner.dumpAsText();
81 if (window.accessibilityController) { 81 if (window.accessibilityController) {
82 buildAccessibilityTree(accessibilityController.focusedElement, 0, 1); 82 buildAccessibilityTree(accessibilityController.focusedElement, 0, 1);
83 } 83 }
84 </script> 84 </script>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698