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

Merge "Fixing comments in Network Capabilities"

Browse Source 
am: b6477d22ca

Change-Id: Iacbcac2e740099ea6b6a6f7a646f046370fd16f5
This commit is contained in:
Pavel Maltsev
2018-03-12 07:57:07 +00:00
committed by android-build-merger
parent 97aaa25bf0 b6477d22ca
commit 2c36870f3d
2 changed files with 8 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
core/java/android/net/NetworkCapabilities.java
View File

@@ -382,7 +382,7 @@ public final class NetworkCapabilities implements Parcelable {
/** /**
* Removes (if found) the given capability from this {@code NetworkCapability} instance. * Removes (if found) the given capability from this {@code NetworkCapability} instance.
* <p> * <p>
* Note that this method removes capabilities that was added via {@link #addCapability(int)}, * Note that this method removes capabilities that were added via {@link #addCapability(int)},
* {@link #addUnwantedCapability(int)} or {@link #setCapabilities(int[], int[])} . * {@link #addUnwantedCapability(int)} or {@link #setCapabilities(int[], int[])} .
* *
* @param capability the capability to be removed. * @param capability the capability to be removed.
@@ -471,6 +471,7 @@ public final class NetworkCapabilities implements Parcelable {
&& ((mUnwantedNetworkCapabilities & (1 << capability)) != 0); && ((mUnwantedNetworkCapabilities & (1 << capability)) != 0);
} }
/** Note this method may result in having the same capability in wanted and unwanted lists. */
private void combineNetCapabilities(NetworkCapabilities nc) { private void combineNetCapabilities(NetworkCapabilities nc) {
this.mNetworkCapabilities |= nc.mNetworkCapabilities; this.mNetworkCapabilities |= nc.mNetworkCapabilities;
this.mUnwantedNetworkCapabilities |= nc.mUnwantedNetworkCapabilities; this.mUnwantedNetworkCapabilities |= nc.mUnwantedNetworkCapabilities;
@@ -1132,7 +1133,11 @@ public final class NetworkCapabilities implements Parcelable {
} }
/** /**
* Combine a set of Capabilities to this one. Useful for coming up with the complete set * Combine a set of Capabilities to this one. Useful for coming up with the complete set.
* <p>
* Note that this method may break an invariant of having a particular capability in either
* wanted or unwanted lists but never in both. Requests that have the same capability in
* both lists will never be satisfied.
* @hide * @hide
*/ */
public void combineCapabilities(NetworkCapabilities nc) { public void combineCapabilities(NetworkCapabilities nc) {

3
tests/net/java/android/net/NetworkCapabilitiesTest.java
View File

@@ -291,7 +291,7 @@ public class NetworkCapabilitiesTest {
assertTrue("Request: " + request + ", Network:" + network, assertTrue("Request: " + request + ", Network:" + network,
request.satisfiedByNetworkCapabilities(network)); request.satisfiedByNetworkCapabilities(network));
// Adding capabilities that doesn't exist in the network anyway // Requesting absence of capabilities that network doesn't have. Request should satisfy.
request.addUnwantedCapability(NET_CAPABILITY_WIFI_P2P); request.addUnwantedCapability(NET_CAPABILITY_WIFI_P2P);
request.addUnwantedCapability(NET_CAPABILITY_NOT_METERED); request.addUnwantedCapability(NET_CAPABILITY_NOT_METERED);
assertTrue(request.satisfiedByNetworkCapabilities(network)); assertTrue(request.satisfiedByNetworkCapabilities(network));
@@ -306,7 +306,6 @@ public class NetworkCapabilitiesTest {
assertTrue(request.hasUnwantedCapability(NET_CAPABILITY_NOT_RESTRICTED)); assertTrue(request.hasUnwantedCapability(NET_CAPABILITY_NOT_RESTRICTED));
assertFalse(request.hasCapability(NET_CAPABILITY_NOT_RESTRICTED)); assertFalse(request.hasCapability(NET_CAPABILITY_NOT_RESTRICTED));
// Now this request won't be satisfied because network contains NOT_RESTRICTED. // Now this request won't be satisfied because network contains NOT_RESTRICTED.
assertFalse(request.satisfiedByNetworkCapabilities(network)); assertFalse(request.satisfiedByNetworkCapabilities(network));
network.removeCapability(NET_CAPABILITY_NOT_RESTRICTED); network.removeCapability(NET_CAPABILITY_NOT_RESTRICTED);