diff options
author | Matt Pape <mpape@google.com> | 2019-04-19 12:31:24 -0700 |
---|---|---|
committer | Matt Pape <mpape@google.com> | 2019-04-30 16:35:16 +0000 |
commit | 15769e2ffab7f79b98df75e2d803c588a4fffc09 (patch) | |
tree | 640c0481755dd50addd48c6d3e5439502310cdd7 /services/contentcapture/java | |
parent | 28cb4d23db83de7b957e0b2fc6a3dae0371b4799 (diff) |
Migrate usage of single property lister to multi prop listener.
Test: atest CtsViewTestCases:TextClassificationManagerTest
atest FrameworksCoreTests:TextClassificationManagerTest
atest ExtServicesUnitTests:AssistantSettingsTest
atest SystemUITests:SmartReplyConstantsTest
atest FrameworksServicesTests:AutofillManagerServiceTest
atest FrameworksUiServicesTests:NotificationManagerServiceTest
Bug: 128902955
Change-Id: I00e524b615b3bd03b6b40535410ff7eb2ef6f8ee
Diffstat (limited to 'services/contentcapture/java')
-rw-r--r-- | services/contentcapture/java/com/android/server/contentcapture/ContentCaptureManagerService.java | 41 |
1 files changed, 22 insertions, 19 deletions
diff --git a/services/contentcapture/java/com/android/server/contentcapture/ContentCaptureManagerService.java b/services/contentcapture/java/com/android/server/contentcapture/ContentCaptureManagerService.java index a2d3d4c25b1d..5c6258ffe4cb 100644 --- a/services/contentcapture/java/com/android/server/contentcapture/ContentCaptureManagerService.java +++ b/services/contentcapture/java/com/android/server/contentcapture/ContentCaptureManagerService.java @@ -50,6 +50,7 @@ import android.os.ShellCallback; import android.os.UserHandle; import android.os.UserManager; import android.provider.DeviceConfig; +import android.provider.DeviceConfig.Properties; import android.provider.Settings; import android.service.contentcapture.ActivityEvent.ActivityEventType; import android.util.ArraySet; @@ -131,9 +132,9 @@ public final class ContentCaptureManagerService extends com.android.internal.R.string.config_defaultContentCaptureService), UserManager.DISALLOW_CONTENT_CAPTURE, /*packageUpdatePolicy=*/ PACKAGE_UPDATE_POLICY_NO_REFRESH); - DeviceConfig.addOnPropertyChangedListener(DeviceConfig.NAMESPACE_CONTENT_CAPTURE, + DeviceConfig.addOnPropertiesChangedListener(DeviceConfig.NAMESPACE_CONTENT_CAPTURE, ActivityThread.currentApplication().getMainExecutor(), - (namespace, key, value) -> onDeviceConfigChange(key, value)); + (properties) -> onDeviceConfigChange(properties)); setDeviceConfigProperties(); if (mDevCfgLogHistorySize > 0) { @@ -255,23 +256,25 @@ public final class ContentCaptureManagerService extends return enabled; } - private void onDeviceConfigChange(@NonNull String key, @Nullable String value) { - switch (key) { - case ContentCaptureManager.DEVICE_CONFIG_PROPERTY_SERVICE_EXPLICITLY_ENABLED: - setDisabledByDeviceConfig(value); - return; - case ContentCaptureManager.DEVICE_CONFIG_PROPERTY_LOGGING_LEVEL: - setLoggingLevelFromDeviceConfig(); - return; - case ContentCaptureManager.DEVICE_CONFIG_PROPERTY_MAX_BUFFER_SIZE: - case ContentCaptureManager.DEVICE_CONFIG_PROPERTY_IDLE_FLUSH_FREQUENCY: - case ContentCaptureManager.DEVICE_CONFIG_PROPERTY_LOG_HISTORY_SIZE: - case ContentCaptureManager.DEVICE_CONFIG_PROPERTY_TEXT_CHANGE_FLUSH_FREQUENCY: - case ContentCaptureManager.DEVICE_CONFIG_PROPERTY_IDLE_UNBIND_TIMEOUT: - setFineTuneParamsFromDeviceConfig(); - return; - default: - Slog.i(mTag, "Ignoring change on " + key); + private void onDeviceConfigChange(@NonNull Properties properties) { + for (String key : properties.getKeyset()) { + switch (key) { + case ContentCaptureManager.DEVICE_CONFIG_PROPERTY_SERVICE_EXPLICITLY_ENABLED: + setDisabledByDeviceConfig(properties.getString(key, null)); + return; + case ContentCaptureManager.DEVICE_CONFIG_PROPERTY_LOGGING_LEVEL: + setLoggingLevelFromDeviceConfig(); + return; + case ContentCaptureManager.DEVICE_CONFIG_PROPERTY_MAX_BUFFER_SIZE: + case ContentCaptureManager.DEVICE_CONFIG_PROPERTY_IDLE_FLUSH_FREQUENCY: + case ContentCaptureManager.DEVICE_CONFIG_PROPERTY_LOG_HISTORY_SIZE: + case ContentCaptureManager.DEVICE_CONFIG_PROPERTY_TEXT_CHANGE_FLUSH_FREQUENCY: + case ContentCaptureManager.DEVICE_CONFIG_PROPERTY_IDLE_UNBIND_TIMEOUT: + setFineTuneParamsFromDeviceConfig(); + return; + default: + Slog.i(mTag, "Ignoring change on " + key); + } } } |