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

Unified Diff: chrome/browser/chromeos/accessibility/accessibility_manager.cc

Issue 410783002: Corner Passthrough for Accessibility (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@side-gestures
Patch Set: Earcons through manager and delegate Created 6 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/chromeos/accessibility/accessibility_manager.cc
diff --git a/chrome/browser/chromeos/accessibility/accessibility_manager.cc b/chrome/browser/chromeos/accessibility/accessibility_manager.cc
index 1d40642b883dd56f8a5ddffa30be1973b09af17e..5444052917271f5d9aa68926f3eac04e8675511b 100644
--- a/chrome/browser/chromeos/accessibility/accessibility_manager.cc
+++ b/chrome/browser/chromeos/accessibility/accessibility_manager.cc
@@ -223,7 +223,7 @@ void InjectChromeVoxContentScript(
for (size_t i = 0; i < content_scripts.size(); i++) {
const extensions::UserScript& script = content_scripts[i];
for (size_t j = 0; j < script.js_scripts().size(); ++j) {
- const extensions::UserScript::File &file = script.js_scripts()[j];
+ const extensions::UserScript::File& file = script.js_scripts()[j];
extensions::ExtensionResource resource = extension->GetResource(
file.relative_path());
loader->AppendScript(resource);
@@ -374,6 +374,8 @@ AccessibilityManager::AccessibilityManager()
manager->Initialize(
SOUND_SPOKEN_FEEDBACK_DISABLED,
bundle.GetRawDataResource(IDR_SOUND_SPOKEN_FEEDBACK_DISABLED_WAV));
+ manager->Initialize(SOUND_PASSTHROUGH,
+ bundle.GetRawDataResource(IDR_SOUND_PASSTHROUGH_WAV));
}
AccessibilityManager::~AccessibilityManager() {
@@ -694,6 +696,16 @@ void AccessibilityManager::OnLocaleChanged() {
EnableSpokenFeedback(true, ash::A11Y_NOTIFICATION_NONE);
}
+void AccessibilityManager::PlayEarcon(int sound_key) {
+ switch (sound_key) {
+ case chromeos::SOUND_PASSTHROUGH:
+ ash::PlaySystemSoundIfSpokenFeedback(SOUND_PASSTHROUGH);
dmazzoni 2014/07/28 05:32:44 Do you need the switch? Why not just: ash::PlaySy
lisayin 2014/07/28 16:28:54 Done.
+ break;
+ default:
+ break;
+ }
+}
+
bool AccessibilityManager::IsHighContrastEnabled() {
return high_contrast_enabled_;
}

Powered by Google App Engine
This is Rietveld 408576698