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

Unified Diff: chrome/browser/resources/chromeos/chromevox/cvox2/background/output_test.extjs

Issue 1696443002: Re-land: Implement ChromeVox Next menus. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/chromeos/chromevox/cvox2/background/output_test.extjs
diff --git a/chrome/browser/resources/chromeos/chromevox/cvox2/background/output_test.extjs b/chrome/browser/resources/chromeos/chromevox/cvox2/background/output_test.extjs
index c1bcdc9b76b18be5a51da7b9ee201f69e46a8f0d..b6de34b5c4a6815895fbce3d642ac9c21aefaef0 100644
--- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/output_test.extjs
+++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/output_test.extjs
@@ -450,15 +450,16 @@ TEST_F('OutputE2ETest', 'Menu', function() {
var range = cursors.Range.fromNode(el);
var o = new Output().withSpeechAndBraille(range, null, 'navigate');
assertEqualsJSON({string_:
- '|Menu|with 1 item|a|Menu item| 1 of 1 ', spans_: [
- {value: 'name', start: 18, end: 19},
- {value: {earconId: 'BUTTON'}, start:18, end:19},
- {value: 'role', start:20, end: 29}
+ '|Menu|a|Menu item| 1 of 1 ', spans_: [
+ {value: {earconId: 'LISTBOX'}, start:0, end:0},
+ {value: 'name', start: 6, end: 7},
+ {value: {earconId: 'BUTTON'}, start:6, end:7},
+ {value: 'role', start:8, end: 17}
]}, o.speechOutputForTest);
checkBrailleOutput(
- 'mnu +1 a mnuitm 1/1',
- [{value: new Output.NodeSpan(el.parent), start: 0, end: 6},
- {value: new Output.NodeSpan(el), start: 7, end: 19}],
+ 'mnu a mnuitm 1/1',
+ [{value: new Output.NodeSpan(el.parent), start: 0, end: 3},
+ {value: new Output.NodeSpan(el), start: 4, end: 16}],
o);
});
});

Powered by Google App Engine
This is Rietveld 408576698