From 57c032bf43eb4ff26f3035d6bf4cf00edc8a95e4 Mon Sep 17 00:00:00 2001 From: Ricky Wai Date: Fri, 27 May 2022 14:21:23 +0000 Subject: [PATCH] Only add boolean extra to bundle when the value is false As the default value is false, we don't need to put it in bundle if the value is false. Bug: 234070143 Test: atest android.app.cts.BroadcastOptionsTest#testTemporaryAppAllowlistBroadcastOptions_defaultValues Change-Id: Ia23d2a03f8c3c64d25b47ed9a8d02ae22ce9246b --- core/java/android/app/ComponentOptions.java | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/core/java/android/app/ComponentOptions.java b/core/java/android/app/ComponentOptions.java index 58732f0153187..4e5e384a27989 100644 --- a/core/java/android/app/ComponentOptions.java +++ b/core/java/android/app/ComponentOptions.java @@ -99,8 +99,10 @@ public class ComponentOptions { public Bundle toBundle() { Bundle b = new Bundle(); b.putBoolean(KEY_PENDING_INTENT_BACKGROUND_ACTIVITY_ALLOWED, mPendingIntentBalAllowed); - b.putBoolean(KEY_PENDING_INTENT_BACKGROUND_ACTIVITY_ALLOWED_BY_PERMISSION, - mPendingIntentBalAllowedByPermission); + if (mPendingIntentBalAllowedByPermission) { + b.putBoolean(KEY_PENDING_INTENT_BACKGROUND_ACTIVITY_ALLOWED_BY_PERMISSION, + mPendingIntentBalAllowedByPermission); + } return b; } }