Merge "Add bit mask constants for the flags used in the ND packet." am: 98e9e80694 am: 41629888ac am: 4871fbf8b7
Original change: https://android-review.googlesource.com/c/platform/frameworks/libs/net/+/1619359
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: I0777274294202f82c5a2dd52228b39a0e12c2a0f
diff --git a/staticlibs/devicetests/com/android/testutils/CompatUtil.kt b/staticlibs/devicetests/com/android/testutils/CompatUtil.kt
deleted file mode 100644
index 4bb90a8..0000000
--- a/staticlibs/devicetests/com/android/testutils/CompatUtil.kt
+++ /dev/null
@@ -1,53 +0,0 @@
-/*
- * Copyright (C) 2021 The Android Open Source Project
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-package com.android.testutils
-
-import android.net.NetworkSpecifier
-import android.os.Build
-
-/**
- * Test utility to create [NetworkSpecifier]s on different SDK versions.
- */
-object CompatUtil {
- @JvmStatic
- fun makeTestNetworkSpecifier(ifName: String): NetworkSpecifier {
- // Until R, there was no TestNetworkSpecifier, StringNetworkSpecifier was used instead
- if (isDevSdkInRange(minExclusive = null, maxInclusive = Build.VERSION_CODES.R)) {
- makeNetworkSpecifierInternal("android.net.StringNetworkSpecifier", ifName)
- }
- // TestNetworkSpecifier is not part of the SDK in some branches using this utility
- // TODO: replace with a direct call to the constructor
- return makeNetworkSpecifierInternal("android.net.TestNetworkSpecifier", ifName)
- }
-
- @JvmStatic
- fun makeEthernetNetworkSpecifier(ifName: String): NetworkSpecifier {
- // Until R, there was no EthernetNetworkSpecifier, StringNetworkSpecifier was used instead
- if (isDevSdkInRange(minExclusive = null, maxInclusive = Build.VERSION_CODES.R)) {
- makeNetworkSpecifierInternal("android.net.StringNetworkSpecifier", ifName)
- }
- // EthernetNetworkSpecifier is not part of the SDK in some branches using this utility
- // TODO: replace with a direct call to the constructor
- return makeNetworkSpecifierInternal("android.net.EthernetNetworkSpecifier", ifName)
- }
-
- private fun makeNetworkSpecifierInternal(clazz: String, specifier: String): NetworkSpecifier {
- // StringNetworkSpecifier was removed after R (and was hidden API before that)
- return Class.forName(clazz)
- .getConstructor(String::class.java).newInstance(specifier) as NetworkSpecifier
- }
-}
\ No newline at end of file
diff --git a/staticlibs/devicetests/com/android/testutils/TestNetworkTracker.kt b/staticlibs/devicetests/com/android/testutils/TestNetworkTracker.kt
index 5b97861..4bd9ae8 100644
--- a/staticlibs/devicetests/com/android/testutils/TestNetworkTracker.kt
+++ b/staticlibs/devicetests/com/android/testutils/TestNetworkTracker.kt
@@ -23,6 +23,7 @@
import android.net.Network
import android.net.NetworkCapabilities
import android.net.NetworkRequest
+import android.net.StringNetworkSpecifier
import android.net.TestNetworkInterface
import android.net.TestNetworkManager
import android.os.Binder
@@ -67,7 +68,7 @@
// Test networks do not have NOT_VPN or TRUSTED capabilities by default
.removeCapability(NetworkCapabilities.NET_CAPABILITY_NOT_VPN)
.removeCapability(NetworkCapabilities.NET_CAPABILITY_TRUSTED)
- .setNetworkSpecifier(CompatUtil.makeTestNetworkSpecifier(iface.interfaceName))
+ .setNetworkSpecifier(StringNetworkSpecifier(iface.interfaceName))
.build()
networkCallback = object : NetworkCallback() {
override fun onAvailable(network: Network) {