commit 1511c56b66ed0d95c58b06c094a8bce7a0d53a03
author Matt Banda <mbanda@google.com> Wed May 26 19:19:05 2021 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Wed May 26 19:19:05 2021 +0000
tree e0df5e8ec0c4c9394b6399a8608535eaa55819f6
parent 937a91bcb77f07d8b869a38e1a6d89f8ff9ef42d
parent 9eccc1ae92ca9ec1cf970f2cf7f8c6e8acd1d6a3

Changes Java/Kotlin AndroidLint API Finder check to also detect Protected methods. am: 9eccc1ae92

Original change: https://android-review.googlesource.com/c/platform/tools/apifinder/+/1718270

Change-Id: If6fd0d85f4534314d5725780f5b04099f0442130

checks/src/main/java/com/android/apifinder/ApiFinderDetector.kt

diff --git a/checks/src/main/java/com/android/apifinder/ApiFinderDetector.kt b/checks/src/main/java/com/android/apifinder/ApiFinderDetector.kt
index 1c72a83..5132fcf 100644
--- a/checks/src/main/java/com/android/apifinder/ApiFinderDetector.kt
+++ b/checks/src/main/java/com/android/apifinder/ApiFinderDetector.kt
@@ -41,13 +41,13 @@
           private fun visitGenericMethod(
               method: PsiMethod, node: UElement, isModuleMethod: Boolean = false
           ) {
-              // Exclude non-public calls.
-              if (!method.hasModifier(JvmModifier.PUBLIC)) {
+              // Exclude non-public/protected calls.
+              if (!method.hasModifier(JvmModifier.PUBLIC) && !method.hasModifier(JvmModifier.PROTECTED)) {
                   return
               }
               var containingClass = method.containingClass
               while (containingClass != null) {
-                  if (!containingClass.hasModifier(JvmModifier.PUBLIC)) {
+                  if (!containingClass.hasModifier(JvmModifier.PUBLIC) && !containingClass.hasModifier(JvmModifier.PROTECTED)) {
                       return
                   }
                   containingClass = containingClass.containingClass