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

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

Issue 2144873002: Support first letter navigation in Panel. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix test. Created 4 years, 5 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 0515c041635e88d2daa7643c96015c1548a46e11..98e7f23a2bd68bb5893f7b434647b404a764c05c 100644
--- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/output_test.extjs
+++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/output_test.extjs
@@ -451,12 +451,13 @@ TEST_F('OutputE2ETest', 'Menu', function() {
var el = root.firstChild.firstChild;
var range = cursors.Range.fromNode(el);
var o = new Output().withSpeechAndBraille(range, null, 'navigate');
- checkSpeechOutput('a|Menu item| 1 of 1 ', [
+ checkSpeechOutput('a|Menu item| 1 of 1 |Menu', [
{value: 'name', start: 0, end: 1},
- {value: 'role', start: 2, end: 11}], o);
+ {value: 'role', start: 21, end: 25}], o);
checkBrailleOutput(
- 'a mnuitm 1/1',
- [{value: new Output.NodeSpan(el), start: 0, end: 12}],
+ 'a mnuitm 1/1 mnu',
+ [{value: new Output.NodeSpan(el), start: 0, end: 12},
+ {value: new Output.NodeSpan(el.parent), start: 13, end: 16}],
o);
});
});

Powered by Google App Engine
This is Rietveld 408576698