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

Unified Diff: chrome/browser/resources/chromeos/chromevox/braille/braille_translator_manager_test.extjs

Issue 2813743002: Switch to selected 8-dot braille table in email and url text fields (Closed)
Patch Set: Fixes corner cases (liblouis callbacks sometimes drop when called quickly). 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/chromeos/chromevox/braille/braille_translator_manager_test.extjs
diff --git a/chrome/browser/resources/chromeos/chromevox/braille/braille_translator_manager_test.extjs b/chrome/browser/resources/chromeos/chromevox/braille/braille_translator_manager_test.extjs
index 24a414f9ebd6364ab8d1dad77aed5fa6562bd6b9..ea5dbddafd05acbea16cf8b5e76ff4991c41e703 100644
--- a/chrome/browser/resources/chromeos/chromevox/braille/braille_translator_manager_test.extjs
+++ b/chrome/browser/resources/chromeos/chromevox/braille/braille_translator_manager_test.extjs
@@ -97,7 +97,7 @@ TEST_F('CvoxBrailleTranslatorManagerTest', 'testRefreshWithoutChange',
this.manager.addChangeListener(function() {
assertNotReached('Refresh should not be called without a change.');
});
- this.manager.refresh();
+ this.manager.refresh(localStorage['brailleTable']);
});
});
@@ -107,10 +107,9 @@ TEST_F('CvoxBrailleTranslatorManagerTest', 'testRefreshWithChange',
assertNotEquals(null, this.manager.getExpandingTranslator());
this.addChangeListener(function() {
assertEquals('en-UEB-g2', this.manager.getDefaultTranslator().table.id);
- assertEquals('en-US-comp8',
+ assertEquals('en-UEB-g1',
this.manager.getUncontractedTranslator().table.id);
});
- localStorage['brailleTable'] = 'en-UEB-g2';
- this.manager.refresh();
+ this.manager.refresh('en-UEB-g2');
});
});

Powered by Google App Engine
This is Rietveld 408576698