commit 0cbccf9e012655177090f363bd3e84dd79aedaab
author Ashwini Oruganti <ashfall@google.com> Tue Mar 09 09:24:02 2021 -0800
committer Ashwini Oruganti <ashfall@google.com> Mon Mar 29 17:17:38 2021 +0000
tree 51666361f6ea92f0fc7d2bf630bb50f98e78a5cf
parent 2335acd9a2b83a7a7a9abe4ceed3d6ddc3569ce7

Enforce the PendingIntent mutability flag requirement for chrome

Bug: 178065720
Test: manual
Change-Id: Ic78716033fa91f4a16540aab72e3bff73c6391e9
Merged-In: Ic78716033fa91f4a16540aab72e3bff73c6391e9

core/java/android/app/PendingIntent.java

diff --git a/core/java/android/app/PendingIntent.java b/core/java/android/app/PendingIntent.java
index f4b9542..4cf3a80 100644
--- a/core/java/android/app/PendingIntent.java
+++ b/core/java/android/app/PendingIntent.java
@@ -379,8 +379,7 @@
         if (Compatibility.isChangeEnabled(PENDING_INTENT_EXPLICIT_MUTABILITY_REQUIRED)
                 && !flagImmutableSet && !flagMutableSet) {
 
-            //TODO(b/178065720) Remove check for chrome and enforce this requirement
-            if (packageName.equals("com.android.chrome") || mInstrumentation.isInstrumenting()) {
+            if (mInstrumentation.isInstrumenting()) {
                 Log.e(TAG, msg);
             } else {
                 throw new IllegalArgumentException(msg);