Merge "Remove legacy network factories" am: 2e2dda5a9f am: 3ca0462c2b am: df1860329b (5bda8be5) · Commits · e / os / android_frameworks_base

packages/Connectivity/framework/src/android/net/ConnectivityManager.java

+0 −26

Original line number	Diff line number	Diff line
		@@ -3231,32 +3231,6 @@ public class ConnectivityManager {
		}
		}

		/** {@hide} - returns the factory serial number */
		@UnsupportedAppUsage
		@RequiresPermission(anyOf = {
		NetworkStack.PERMISSION_MAINLINE_NETWORK_STACK,
		android.Manifest.permission.NETWORK_FACTORY})
		public int registerNetworkFactory(Messenger messenger, String name) {
		try {
		return mService.registerNetworkFactory(messenger, name);
		} catch (RemoteException e) {
		throw e.rethrowFromSystemServer();
		}
		}

		/** {@hide} */
		@UnsupportedAppUsage(maxTargetSdk = Build.VERSION_CODES.P, trackingBug = 115609023)
		@RequiresPermission(anyOf = {
		NetworkStack.PERMISSION_MAINLINE_NETWORK_STACK,
		android.Manifest.permission.NETWORK_FACTORY})
		public void unregisterNetworkFactory(Messenger messenger) {
		try {
		mService.unregisterNetworkFactory(messenger);
		} catch (RemoteException e) {
		throw e.rethrowFromSystemServer();
		}
		}

		/**
		* Registers the specified {@link NetworkProvider}.
		* Each listener must only be registered once. The listener can be unregistered with

packages/Connectivity/framework/src/android/net/IConnectivityManager.aidl

+0 −3

Original line number	Diff line number	Diff line
		@@ -156,9 +156,6 @@ interface IConnectivityManager

		boolean requestBandwidthUpdate(in Network network);

		int registerNetworkFactory(in Messenger messenger, in String name);
		void unregisterNetworkFactory(in Messenger messenger);

		int registerNetworkProvider(in Messenger messenger, in String name);
		void unregisterNetworkProvider(in Messenger messenger);

services/core/java/com/android/server/ConnectivityService.java

+10 −53

Original line number	Diff line number	Diff line
		@@ -3357,7 +3357,6 @@ public class ConnectivityService extends IConnectivityManager.Stub
		if (VDBG) log("NetworkFactory connected");
		// Finish setting up the full connection
		NetworkProviderInfo npi = mNetworkProviderInfos.get(msg.replyTo);
		npi.completeConnection();
		sendAllRequestsToProvider(npi);
		} else {
		loge("Error connecting NetworkFactory");
		@@ -5443,27 +5442,21 @@ public class ConnectivityService extends IConnectivityManager.Stub
		private static class NetworkProviderInfo {
		public final String name;
		public final Messenger messenger;
		private final AsyncChannel mAsyncChannel;
		private final IBinder.DeathRecipient mDeathRecipient;
		public final int providerId;

		NetworkProviderInfo(String name, Messenger messenger, AsyncChannel asyncChannel,
		int providerId, IBinder.DeathRecipient deathRecipient) {
		int providerId, @NonNull IBinder.DeathRecipient deathRecipient) {
		this.name = name;
		this.messenger = messenger;
		this.providerId = providerId;
		mAsyncChannel = asyncChannel;
		mDeathRecipient = deathRecipient;

		if ((mAsyncChannel == null) == (mDeathRecipient == null)) {
		throw new AssertionError("Must pass exactly one of asyncChannel or deathRecipient");
		if (mDeathRecipient == null) {
		throw new AssertionError("Must pass a deathRecipient");
		}
		}

		boolean isLegacyNetworkFactory() {
		return mAsyncChannel != null;
		}

		void sendMessageToNetworkProvider(int what, int arg1, int arg2, Object obj) {
		try {
		messenger.send(Message.obtain(null /* handler */, what, arg1, arg2, obj));
		@@ -5474,27 +5467,15 @@ public class ConnectivityService extends IConnectivityManager.Stub
		}

		void requestNetwork(NetworkRequest request, int score, int servingProviderId) {
		if (isLegacyNetworkFactory()) {
		mAsyncChannel.sendMessage(android.net.NetworkFactory.CMD_REQUEST_NETWORK, score,
		servingProviderId, request);
		} else {
		sendMessageToNetworkProvider(NetworkProvider.CMD_REQUEST_NETWORK, score,
		servingProviderId, request);
		}
		}

		void cancelRequest(NetworkRequest request) {
		if (isLegacyNetworkFactory()) {
		mAsyncChannel.sendMessage(android.net.NetworkFactory.CMD_CANCEL_REQUEST, request);
		} else {
		sendMessageToNetworkProvider(NetworkProvider.CMD_CANCEL_REQUEST, 0, 0, request);
		}
		}

		void connect(Context context, Handler handler) {
		if (isLegacyNetworkFactory()) {
		mAsyncChannel.connect(context, handler, messenger);
		} else {
		try {
		messenger.getBinder().linkToDeath(mDeathRecipient, 0);
		} catch (RemoteException e) {
		@@ -5503,13 +5484,6 @@ public class ConnectivityService extends IConnectivityManager.Stub
		}
		}

		void completeConnection() {
		if (isLegacyNetworkFactory()) {
		mAsyncChannel.sendMessage(AsyncChannel.CMD_CHANNEL_FULL_CONNECTION);
		}
		}
		}

		private void ensureAllNetworkRequestsHaveType(List<NetworkRequest> requests) {
		for (int i = 0; i < requests.size(); i++) {
		ensureNetworkRequestHasType(requests.get(i));
		@@ -6000,15 +5974,6 @@ public class ConnectivityService extends IConnectivityManager.Stub
		EVENT_RELEASE_NETWORK_REQUEST, getCallingUid(), 0, networkRequest));
		}

		@Override
		public int registerNetworkFactory(Messenger messenger, String name) {
		enforceNetworkFactoryPermission();
		NetworkProviderInfo npi = new NetworkProviderInfo(name, messenger, new AsyncChannel(),
		nextNetworkProviderId(), null /* deathRecipient */);
		mHandler.sendMessage(mHandler.obtainMessage(EVENT_REGISTER_NETWORK_PROVIDER, npi));
		return npi.providerId;
		}

		private void handleRegisterNetworkProvider(NetworkProviderInfo npi) {
		if (mNetworkProviderInfos.containsKey(npi.messenger)) {
		// Avoid creating duplicates. even if an app makes a direct AIDL call.
		@@ -6022,11 +5987,8 @@ public class ConnectivityService extends IConnectivityManager.Stub
		if (DBG) log("Got NetworkProvider Messenger for " + npi.name);
		mNetworkProviderInfos.put(npi.messenger, npi);
		npi.connect(mContext, mTrackerHandler);
		if (!npi.isLegacyNetworkFactory()) {
		// Legacy NetworkFactories get their requests when their AsyncChannel connects.
		sendAllRequestsToProvider(npi);
		}
		}

		@Override
		public int registerNetworkProvider(Messenger messenger, String name) {
		@@ -6044,11 +6006,6 @@ public class ConnectivityService extends IConnectivityManager.Stub
		mHandler.sendMessage(mHandler.obtainMessage(EVENT_UNREGISTER_NETWORK_PROVIDER, messenger));
		}

		@Override
		public void unregisterNetworkFactory(Messenger messenger) {
		unregisterNetworkProvider(messenger);
		}

		private void handleUnregisterNetworkProvider(Messenger messenger) {
		NetworkProviderInfo npi = mNetworkProviderInfos.remove(messenger);
		if (npi == null) {