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

Unified Diff: chrome/browser/chromeos/arc/intent_helper/arc_settings_service.cc

Issue 2559663002: Support focus highlight in Android window (Closed)
Patch Set: Created 4 years 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/arc/intent_helper/arc_settings_service.cc
diff --git a/chrome/browser/chromeos/arc/intent_helper/arc_settings_service.cc b/chrome/browser/chromeos/arc/intent_helper/arc_settings_service.cc
index ff9c1c985bcecace3805b29c0ce2dc1712fc49f7..5b668d3699b50e8fe9a1106eb857339b14d54fc5 100644
--- a/chrome/browser/chromeos/arc/intent_helper/arc_settings_service.cc
+++ b/chrome/browser/chromeos/arc/intent_helper/arc_settings_service.cc
@@ -120,6 +120,7 @@ class ArcSettingsServiceImpl
// Send settings that need to be synced only on Android first start to
// Android.
void SyncInitialSettings() const;
+ void SyncFocusHighlightEnabled() const;
void SyncFontSize() const;
void SyncLocale() const;
void SyncProxySettings() const;
@@ -197,6 +198,7 @@ void ArcSettingsServiceImpl::StartObservingSettingsChanges() {
AddPrefToObserve(onc::prefs::kDeviceOpenNetworkConfiguration);
AddPrefToObserve(onc::prefs::kOpenNetworkConfiguration);
AddPrefToObserve(prefs::kAccessibilityVirtualKeyboardEnabled);
+ AddPrefToObserve(prefs::kAccessibilityFocusHighlightEnabled);
reporting_consent_subscription_ = CrosSettings::Get()->AddSettingsObserver(
chromeos::kStatsReportingPref,
@@ -237,6 +239,7 @@ void ArcSettingsServiceImpl::SyncRuntimeSettings() const {
SyncTimeZone();
SyncUse24HourClock();
SyncAccessibilityVirtualKeyboardEnabled();
+ SyncFocusHighlightEnabled();
const PrefService* const prefs =
ProfileManager::GetActiveUserProfile()->GetPrefs();
@@ -275,7 +278,9 @@ void ArcSettingsServiceImpl::AdapterPoweredChanged(
}
void ArcSettingsServiceImpl::OnPrefChanged(const std::string& pref_name) const {
- if (pref_name == prefs::kAccessibilitySpokenFeedbackEnabled) {
+ if (pref_name == prefs::kAccessibilityFocusHighlightEnabled) {
+ SyncFocusHighlightEnabled();
+ } else if (pref_name == prefs::kAccessibilitySpokenFeedbackEnabled) {
SyncSpokenFeedbackEnabled();
} else if (pref_name == prefs::kWebKitDefaultFixedFontSize ||
pref_name == prefs::kWebKitDefaultFontSize ||
@@ -344,6 +349,12 @@ void ArcSettingsServiceImpl::SendBoolPrefSettingsBroadcast(
SendSettingsBroadcast(action, extras);
}
+void ArcSettingsServiceImpl::SyncFocusHighlightEnabled() const {
+ SendBoolPrefSettingsBroadcast(
+ prefs::kAccessibilityFocusHighlightEnabled,
+ "org.chromium.arc.intent_helper.SET_FOCUS_HIGHLIGHT_ENABLED");
+}
+
void ArcSettingsServiceImpl::SyncSpokenFeedbackEnabled() const {
SendBoolPrefSettingsBroadcast(
prefs::kAccessibilitySpokenFeedbackEnabled,

Powered by Google App Engine
This is Rietveld 408576698