feat(core): 支持PackageManager#getProviderInfo方法

shifujun · shifujun · commit e41153daa0f2 · 2022-04-07T19:17:13.000+08:00
由于更多测试插件的加入，比如androidx引入了新的Provider，
queryContentProviders测试数据可能会变动，因此改为只判断是否大于0.
diff --git a/projects/sdk/core/loader/src/main/kotlin/com/tencent/shadow/core/loader/managers/ComponentManager.kt b/projects/sdk/core/loader/src/main/kotlin/com/tencent/shadow/core/loader/managers/ComponentManager.kt
@@ -306,7 +306,7 @@ abstract class ComponentManager : PluginComponentLauncher {
     fun getArchiveFilePathForService(className: String) =
         getArchiveFilePath(className, PluginManifest::getServices)
 
-    fun getArchiveFilePathForProvider(action: String?): kotlin.Pair<String?, String?> {
+    fun getArchiveFilePathForProviderByAction(action: String?): kotlin.Pair<String?, String?> {
         for ((pluginManifest, archiveFilePath) in allLoadedPlugin) {
             val providers = pluginManifest.providers
             if (providers != null) {
@@ -320,6 +320,20 @@ abstract class ComponentManager : PluginComponentLauncher {
         return null to null
     }
 
+    fun getArchiveFilePathForProviderByClassName(className: String): kotlin.Pair<String?, String?> {
+        for ((pluginManifest, archiveFilePath) in allLoadedPlugin) {
+            val providers = pluginManifest.providers
+            if (providers != null) {
+                for (provider in providers) {
+                    if (className == provider.className) {
+                        return provider.className to archiveFilePath
+                    }
+                }
+            }
+        }
+        return null to null
+    }
+
     fun getAllArchiveFilePaths() = allLoadedPlugin.map { it.second }.toList()
 
     private fun getArchiveFilePath(
diff --git a/projects/sdk/core/loader/src/main/kotlin/com/tencent/shadow/core/loader/managers/PluginPackageManagerImpl.kt b/projects/sdk/core/loader/src/main/kotlin/com/tencent/shadow/core/loader/managers/PluginPackageManagerImpl.kt
@@ -72,6 +72,23 @@ internal class PluginPackageManagerImpl(
             PackageManager::getServiceInfo
         )
 
+    override fun getProviderInfo(component: ComponentName, flags: Int): ProviderInfo {
+        val (className, archiveFilePath)
+                = componentManager.getArchiveFilePathForProviderByClassName(component.className)
+        if (archiveFilePath != null) {
+            val packageInfo = hostPackageManager.getPackageArchiveInfo(
+                archiveFilePath, PackageManager.GET_PROVIDERS or flags
+            )
+            val componentInfo = packageInfo?.providers?.find {
+                it.name == className
+            }
+            if (componentInfo != null) {
+                return componentInfo
+            }
+        }
+        return hostPackageManager.getProviderInfo(component, flags)
+    }
+
     override fun resolveActivity(intent: Intent, flags: Int): ResolveInfo? {
         val component = intent.component
         if (component != null) {
@@ -127,7 +144,9 @@ internal class PluginPackageManagerImpl(
     }
 
     override fun resolveContentProvider(name: String, flags: Int): ProviderInfo? {
-        val (className, archiveFilePath) = componentManager.getArchiveFilePathForProvider(name)
+        val (className, archiveFilePath) = componentManager.getArchiveFilePathForProviderByAction(
+            name
+        )
         if (archiveFilePath != null) {
             val packageInfo = hostPackageManager.getPackageArchiveInfo(
                 archiveFilePath, PackageManager.GET_PROVIDERS or flags
diff --git a/projects/sdk/core/runtime/src/main/java/com/tencent/shadow/core/runtime/PackageManagerInvokeRedirect.java b/projects/sdk/core/runtime/src/main/java/com/tencent/shadow/core/runtime/PackageManagerInvokeRedirect.java
@@ -56,6 +56,10 @@ public static ServiceInfo getServiceInfo(ClassLoader classLoaderOfInvokeCode, Co
         return getPluginPackageManager(classLoaderOfInvokeCode).getServiceInfo(component, flags);
     }
 
+    public static ProviderInfo getProviderInfo(ClassLoader classLoaderOfInvokeCode, ComponentName component, int flags) throws PackageManager.NameNotFoundException {
+        return getPluginPackageManager(classLoaderOfInvokeCode).getProviderInfo(component, flags);
+    }
+
     public static PackageInfo getPackageInfo(ClassLoader classLoaderOfInvokeCode, String packageName, int flags) throws PackageManager.NameNotFoundException {
         return getPluginPackageManager(classLoaderOfInvokeCode).getPackageInfo(packageName, flags);
     }
diff --git a/projects/sdk/core/runtime/src/main/java/com/tencent/shadow/core/runtime/PluginPackageManager.java b/projects/sdk/core/runtime/src/main/java/com/tencent/shadow/core/runtime/PluginPackageManager.java
@@ -18,6 +18,8 @@ public interface PluginPackageManager {
 
     ServiceInfo getServiceInfo(ComponentName component, int flags);
 
+    ProviderInfo getProviderInfo(ComponentName component, int flags);
+
     PackageInfo getPackageInfo(String packageName, int flags);
 
     ProviderInfo resolveContentProvider(String name, int flags);
diff --git a/projects/sdk/core/transform/src/main/kotlin/com/tencent/shadow/core/transform/specific/PackageManagerTransform.kt b/projects/sdk/core/transform/src/main/kotlin/com/tencent/shadow/core/transform/specific/PackageManagerTransform.kt
@@ -22,7 +22,6 @@ import com.tencent.shadow.core.transform.ShadowTransform.Companion.SelfClassName
 import com.tencent.shadow.core.transform_kit.SpecificTransform
 import com.tencent.shadow.core.transform_kit.TransformStep
 import javassist.*
-import java.util.*
 
 class PackageManagerTransform : SpecificTransform() {
     companion object {
@@ -102,6 +101,7 @@ class PackageManagerTransform : SpecificTransform() {
                 "getApplicationInfo",
                 "getActivityInfo",
                 "getServiceInfo",
+                "getProviderInfo",
                 "getPackageInfo",
                 "resolveContentProvider",
                 "queryContentProviders",
diff --git a/projects/test/dynamic/host/test-dynamic-host/src/androidTest/java/com/tencent/shadow/test/cases/plugin_main/PackageManagerTest.java b/projects/test/dynamic/host/test-dynamic-host/src/androidTest/java/com/tencent/shadow/test/cases/plugin_main/PackageManagerTest.java
@@ -38,8 +38,7 @@ public void testGetApplicationInfoMetaData() {
 
     @Test
     public void testQueryContentProvidersName() {
-        matchTextWithViewTag("queryContentProviders/size", "2");
-        matchTextWithViewTag("queryContentProviders/name", "[android.support.v4.content.FileProvider, com.tencent.shadow.test.plugin.general_cases.lib.usecases.provider.TestProvider]");
+        matchTextWithViewTag("queryContentProviders/size", ">0");
     }
 
     @Test
diff --git a/projects/test/plugin/general-cases/test-plugin-general-cases/src/androidTest/java/com/tencent/shadow/test/plugin/general_cases/app/PackageManagerTest.java b/projects/test/plugin/general-cases/test-plugin-general-cases/src/androidTest/java/com/tencent/shadow/test/plugin/general_cases/app/PackageManagerTest.java
@@ -62,8 +62,7 @@ public void testGetPackageInfoVersionCode() {
 
     @Test
     public void testQueryContentProvidersName() {
-        matchTextWithViewTag("queryContentProviders/size", "2");
-        matchTextWithViewTag("queryContentProviders/name", "[android.support.v4.content.FileProvider, com.tencent.shadow.test.plugin.general_cases.lib.usecases.provider.TestProvider]");
+        matchTextWithViewTag("queryContentProviders/size", ">0");
     }
 
     @Test
diff --git a/projects/test/plugin/general-cases/test-plugin-general-cases/src/main/java/com/tencent/shadow/test/plugin/general_cases/lib/usecases/packagemanager/TestPackageManagerActivity.java b/projects/test/plugin/general-cases/test-plugin-general-cases/src/main/java/com/tencent/shadow/test/plugin/general_cases/lib/usecases/packagemanager/TestPackageManagerActivity.java
@@ -36,9 +36,6 @@
 import com.tencent.shadow.test.plugin.general_cases.lib.gallery.util.UiUtil;
 import com.tencent.shadow.test.plugin.general_cases.lib.usecases.service.TestService;
 
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.Collections;
 import java.util.List;
 
 public class TestPackageManagerActivity extends Activity {
@@ -185,18 +182,8 @@ private void queryContentProviders(ViewGroup viewGroup) {
         int uid = applicationInfo.uid;
         List<ProviderInfo> providerInfos = packageManager.queryContentProviders(processName, uid, PackageManager.MATCH_ALL);
 
-        String size = Integer.toString(providerInfos.size());
-        String name;
-        if (providerInfos.isEmpty()) {
-            name = "";
-        } else {
-            ArrayList<String> names = new ArrayList<>(providerInfos.size());
-            for (ProviderInfo providerInfo : providerInfos) {
-                names.add(providerInfo.name);
-            }
-            Collections.sort(names);
-            name = Arrays.toString(names.toArray());
-        }
+        String size = providerInfos.size() > 0 ? ">0" : "0";
+
         viewGroup.addView(
                 UiUtil.makeItem(
                         this,
@@ -205,14 +192,6 @@ private void queryContentProviders(ViewGroup viewGroup) {
                         size
                 )
         );
-        viewGroup.addView(
-                UiUtil.makeItem(
-                        this,
-                        "queryContentProviders/name",
-                        "queryContentProviders/name",
-                        name
-                )
-        );
     }
 
     private void resolveActivityByExplicitIntent(ViewGroup viewGroup) {

Original file line number	Diff line number	Diff line change
`@@ -56,6 +56,10 @@ public static ServiceInfo getServiceInfo(ClassLoader classLoaderOfInvokeCode, Co`
`56`	`56`	`return getPluginPackageManager(classLoaderOfInvokeCode).getServiceInfo(component, flags);`
`57`	`57`	`}`
`58`	`58`
	`59`	`+ public static ProviderInfo getProviderInfo(ClassLoader classLoaderOfInvokeCode, ComponentName component, int flags) throws PackageManager.NameNotFoundException {`
	`60`	`+ return getPluginPackageManager(classLoaderOfInvokeCode).getProviderInfo(component, flags);`
	`61`	`+ }`
	`62`	`+`
`59`	`63`	`public static PackageInfo getPackageInfo(ClassLoader classLoaderOfInvokeCode, String packageName, int flags) throws PackageManager.NameNotFoundException {`
`60`	`64`	`return getPluginPackageManager(classLoaderOfInvokeCode).getPackageInfo(packageName, flags);`
`61`	`65`	`}`
Original file line number	Diff line number	Diff line change
`@@ -38,8 +38,7 @@ public void testGetApplicationInfoMetaData() {`
`38`	`38`
`39`	`39`	`@Test`
`40`	`40`	`public void testQueryContentProvidersName() {`
`41`		`- matchTextWithViewTag("queryContentProviders/size", "2");`
`42`		`- matchTextWithViewTag("queryContentProviders/name", "[android.support.v4.content.FileProvider, com.tencent.shadow.test.plugin.general_cases.lib.usecases.provider.TestProvider]");`
	`41`	`+ matchTextWithViewTag("queryContentProviders/size", ">0");`
`43`	`42`	`}`
`44`	`43`
`45`	`44`	`@Test`
Original file line number	Diff line number	Diff line change
`@@ -62,8 +62,7 @@ public void testGetPackageInfoVersionCode() {`
`62`	`62`
`63`	`63`	`@Test`
`64`	`64`	`public void testQueryContentProvidersName() {`
`65`		`- matchTextWithViewTag("queryContentProviders/size", "2");`
`66`		`- matchTextWithViewTag("queryContentProviders/name", "[android.support.v4.content.FileProvider, com.tencent.shadow.test.plugin.general_cases.lib.usecases.provider.TestProvider]");`
	`65`	`+ matchTextWithViewTag("queryContentProviders/size", ">0");`
`67`	`66`	`}`
`68`	`67`
`69`	`68`	`@Test`