From 9da4edace169cd6e29ddf176528143c416a23b8b Mon Sep 17 00:00:00 2001 From: Tor Norbye <tnorbye@google.com> Date: Wed, 20 Feb 2019 19:21:45 -0800 Subject: [PATCH] Remove MbmsDownloadServiceBase special case Test: make update-api and make checkapi Change-Id: I034f99483c53d86922f735143f6a16ef41d46093 --- .../com/android/tools/metalava/model/ClassItem.kt | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/src/main/java/com/android/tools/metalava/model/ClassItem.kt b/src/main/java/com/android/tools/metalava/model/ClassItem.kt index 0d072fa..a26b71b 100644 --- a/src/main/java/com/android/tools/metalava/model/ClassItem.kt +++ b/src/main/java/com/android/tools/metalava/model/ClassItem.kt @@ -670,18 +670,6 @@ interface ClassItem : Item { if (!predicate.test(superClass)) { superClass.filteredInterfaceTypes(predicate, types, true, includeParents, target) } else if (includeSelf && superClass.isInterface()) { - // Special case: Arguably, IInterface should be included in the system API by the - // general rules. However, this was just added to the system API in 28 at the same - // time as metalava, which did not include some hidden super classes in its analysis. - // This is now marked as an incompatible API change, so treat this the same way as in - // API 28 until this is clarified. - if (superClass.simpleName() == "IInterface" && - (target.qualifiedName() == "android.telephony.mbms.vendor.MbmsDownloadServiceBase" || - target.qualifiedName() == "android.telephony.mbms.vendor.MbmsStreamingServiceBase") - ) { - return types - } - types.add(superClassType) if (includeParents) { superClass.filteredInterfaceTypes(predicate, types, true, includeParents, target) -- GitLab