LineageOS/android_packages_modules_Connectivity
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Replace clearAll with withoutDefaultCapabilities in NC#Builder am: 6d8141caa1

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/opt/net/ethernet/+/14518354

Change-Id: I872a9b2197c21e30f4a6e2aff8d4998f655d61c7
This commit is contained in:
Chiachang Wang
2021-05-14 09:36:56 +00:00
committed by Automerger Merge Worker
parent 80b0c3f84a 6d8141caa1
commit f40d0c5a3e
3 changed files with 6 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
service-t/src/com/android/server/ethernet/EthernetNetworkFactory.java
View File

@@ -167,8 +167,8 @@ public class EthernetNetworkFactory extends NetworkFactory {
} }
private void updateCapabilityFilter() { private void updateCapabilityFilter() {
NetworkCapabilities capabilitiesFilter = new NetworkCapabilities.Builder() NetworkCapabilities capabilitiesFilter =
.clearAll() NetworkCapabilities.Builder.withoutDefaultCapabilities()
.addTransportType(NetworkCapabilities.TRANSPORT_ETHERNET) .addTransportType(NetworkCapabilities.TRANSPORT_ETHERNET)
.build(); .build();

7
service-t/src/com/android/server/ethernet/EthernetTracker.java
View File

@@ -507,10 +507,9 @@ final class EthernetTracker {
boolean clearDefaultCapabilities, @Nullable String commaSeparatedCapabilities, boolean clearDefaultCapabilities, @Nullable String commaSeparatedCapabilities,
@Nullable String overrideTransport) { @Nullable String overrideTransport) {
final NetworkCapabilities.Builder builder = new NetworkCapabilities.Builder(); final NetworkCapabilities.Builder builder = clearDefaultCapabilities
if (clearDefaultCapabilities) { ? NetworkCapabilities.Builder.withoutDefaultCapabilities()
builder.clearAll(); // Remove default capabilities and transports : new NetworkCapabilities.Builder();
}
// Determine the transport type. If someone has tried to define an override transport then // Determine the transport type. If someone has tried to define an override transport then
// attempt to add it. Since we can only have one override, all errors with it will // attempt to add it. Since we can only have one override, all errors with it will

2
tests/ethernet/java/com/android/server/ethernet/EthernetTrackerTest.java
View File

@@ -107,7 +107,7 @@ public class EthernetTrackerTest {
private NetworkCapabilities.Builder makeEthernetCapabilitiesBuilder(boolean clearAll) { private NetworkCapabilities.Builder makeEthernetCapabilitiesBuilder(boolean clearAll) {
final NetworkCapabilities.Builder builder = final NetworkCapabilities.Builder builder =
clearAll ? new NetworkCapabilities.Builder().clearAll() clearAll ? NetworkCapabilities.Builder.withoutDefaultCapabilities()
: new NetworkCapabilities.Builder(); : new NetworkCapabilities.Builder();
return builder.addCapability(NetworkCapabilities.NET_CAPABILITY_NOT_ROAMING) return builder.addCapability(NetworkCapabilities.NET_CAPABILITY_NOT_ROAMING)
.addCapability(NetworkCapabilities.NET_CAPABILITY_NOT_CONGESTED) .addCapability(NetworkCapabilities.NET_CAPABILITY_NOT_CONGESTED)