Commit eb1d84f4 authored by Anastasia Helfinstein's avatar Anastasia Helfinstein Committed by Commit Bot

Change function name to match code convention

I recently realized that Switch Access did not follow the convention
used by all other accessibility component extensions for naming this
function. This re-aligns the naming for code clarity.

Bug: None
Change-Id: I0b015a25e3f7fa45add834ba4401504fa31151e9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1674640Reviewed-by: default avatarDominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: Anastasia Helfinstein <anastasi@google.com>
Cr-Commit-Position: refs/heads/master@{#703450}
parent 7df70dbd
...@@ -908,7 +908,7 @@ bool AccessibilityManager::IsSwitchAccessEnabled() const { ...@@ -908,7 +908,7 @@ bool AccessibilityManager::IsSwitchAccessEnabled() const {
return switch_access_enabled_; return switch_access_enabled_;
} }
void AccessibilityManager::UpdateSwitchAccessFromPref() { void AccessibilityManager::OnSwitchAccessChanged() {
if (!profile_) if (!profile_)
return; return;
...@@ -1115,7 +1115,7 @@ void AccessibilityManager::SetProfile(Profile* profile) { ...@@ -1115,7 +1115,7 @@ void AccessibilityManager::SetProfile(Profile* profile) {
base::Unretained(this))); base::Unretained(this)));
pref_change_registrar_->Add( pref_change_registrar_->Add(
ash::prefs::kAccessibilitySwitchAccessEnabled, ash::prefs::kAccessibilitySwitchAccessEnabled,
base::Bind(&AccessibilityManager::UpdateSwitchAccessFromPref, base::Bind(&AccessibilityManager::OnSwitchAccessChanged,
base::Unretained(this))); base::Unretained(this)));
pref_change_registrar_->Add( pref_change_registrar_->Add(
ash::prefs::kAccessibilityAutoclickEnabled, ash::prefs::kAccessibilityAutoclickEnabled,
...@@ -1150,11 +1150,11 @@ void AccessibilityManager::SetProfile(Profile* profile) { ...@@ -1150,11 +1150,11 @@ void AccessibilityManager::SetProfile(Profile* profile) {
else else
UpdateBrailleImeState(); UpdateBrailleImeState();
UpdateAlwaysShowMenuFromPref(); UpdateAlwaysShowMenuFromPref();
UpdateSwitchAccessFromPref();
// TODO(warx): reconcile to ash once the prefs registration above is moved to // TODO(warx): reconcile to ash once the prefs registration above is moved to
// ash. // ash.
OnSpokenFeedbackChanged(); OnSpokenFeedbackChanged();
OnSwitchAccessChanged();
OnSelectToSpeakChanged(); OnSelectToSpeakChanged();
OnAutoclickChanged(); OnAutoclickChanged();
} }
......
...@@ -384,8 +384,8 @@ class AccessibilityManager ...@@ -384,8 +384,8 @@ class AccessibilityManager
void OnFocusHighlightChanged(); void OnFocusHighlightChanged();
void OnTapDraggingChanged(); void OnTapDraggingChanged();
void OnSelectToSpeakChanged(); void OnSelectToSpeakChanged();
void UpdateSwitchAccessFromPref();
void OnAutoclickChanged(); void OnAutoclickChanged();
void OnSwitchAccessChanged();
void CheckBrailleState(); void CheckBrailleState();
void ReceiveBrailleDisplayState( void ReceiveBrailleDisplayState(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment