commit 85be035336af8d83eb24980026418207c85991cb
author Stevie Kideckel <kideckel@google.com> Mon May 24 14:03:52 2021 +0000
committer Stevie Kideckel <kideckel@google.com> Wed Jun 09 16:54:45 2021 +0000
tree 5c31e83173e5790a764e17abc12b4c38fcff05c3
parent ebcaa39d6b87e25e2f87319943186ea006405af5

Include widget providers enabled at runtime

The call to queryIntentReceivers only contains receivers that are
enabled. It does not return disabled receivers, whether disabled by
default and unmodified or disabled at runtime.

Therefore, the isEnabled check only has the effect of filtering out enabled widgets that were initially disabled in the manifest.

I have verified this locally with all 8 permutations of manifest-runtime
component enabled states.

Fix: 189087746
Test: locally, AppWidgetServiceImplTest
Change-Id: Ibc35005f8af31622134edb174469b39895b88ecc

services/appwidget/java/com/android/server/appwidget/AppWidgetServiceImpl.java

diff --git a/services/appwidget/java/com/android/server/appwidget/AppWidgetServiceImpl.java b/services/appwidget/java/com/android/server/appwidget/AppWidgetServiceImpl.java
index 4e14411..acbf487 100644
--- a/services/appwidget/java/com/android/server/appwidget/AppWidgetServiceImpl.java
+++ b/services/appwidget/java/com/android/server/appwidget/AppWidgetServiceImpl.java
@@ -2186,10 +2186,6 @@
             return false;
         }
 
-        if (!ri.activityInfo.isEnabled()) {
-            return false;
-        }
-
         ComponentName componentName = new ComponentName(ri.activityInfo.packageName,
                 ri.activityInfo.name);
         ProviderId providerId = new ProviderId(ri.activityInfo.applicationInfo.uid, componentName);