DPMS logging improvements.

Test: atest com.android.cts.devicepolicy.ProfileOwnerTest#testDevicePolicySafetyCheckerIntegration_onOperationSafetyStateChanged
Bug: 181627946

Change-Id: Id55acdd00eb4fe34b0458e6a8e7c4dc711f47f5a
This commit is contained in:
Felipe Leme
2021-03-04 10:47:14 -08:00
parent b797729b3f
commit 4d3a2681e0
2 changed files with 7 additions and 5 deletions

View File

@@ -7591,15 +7591,15 @@ public class DevicePolicyManagerService extends BaseIDevicePolicyManager {
private void sendActiveAdminCommand(String action, Bundle extras,
@UserIdInt int userId, ComponentName receiverComponent) {
if (VERBOSE_LOG) {
Slog.v(LOG_TAG, "sending intent " + action + " to "
+ receiverComponent.flattenToShortString() + " on user " + userId);
}
final Intent intent = new Intent(action);
intent.setComponent(receiverComponent);
if (extras != null) {
intent.putExtras(extras);
}
if (VERBOSE_LOG) {
Slog.v(LOG_TAG, "sendActiveAdminCommand(): broadcasting " + action + " to "
+ receiverComponent.flattenToShortString() + " on user " + userId);
}
mContext.sendBroadcastAsUser(intent, UserHandle.of(userId));
}
@@ -12437,10 +12437,12 @@ public class DevicePolicyManagerService extends BaseIDevicePolicyManager {
extras.putBoolean(DeviceAdminReceiver.EXTRA_OPERATION_SAFETY_STATE, isSafe);
if (mOwners.hasDeviceOwner()) {
if (VERBOSE_LOG) Slog.v(LOG_TAG, "Notifying DO");
sendDeviceOwnerCommand(DeviceAdminReceiver.ACTION_OPERATION_SAFETY_STATE_CHANGED,
extras);
}
for (int profileOwnerId : mOwners.getProfileOwnerKeys()) {
if (VERBOSE_LOG) Slog.v(LOG_TAG, "Notifying PO for user " + profileOwnerId);
sendProfileOwnerCommand(DeviceAdminReceiver.ACTION_OPERATION_SAFETY_STATE_CHANGED,
extras, profileOwnerId);
}

View File

@@ -60,7 +60,7 @@ final class OneTimeSafetyChecker implements DevicePolicySafetyChecker {
mOperation = operation;
mReason = reason;
mRealSafetyChecker = service.getDevicePolicySafetyChecker();
Slog.i(TAG, "OneTimeSafetyChecker constructor: operation= " + operationToString(operation)
Slog.i(TAG, "OneTimeSafetyChecker constructor: operation=" + operationToString(operation)
+ ", reason=" + operationSafetyReasonToString(reason)
+ ", realChecker=" + mRealSafetyChecker
+ ", maxDuration=" + SELF_DESTRUCT_TIMEOUT_MS + "ms");