Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 7df36ed9 authored by Benedict Wong's avatar Benedict Wong
Browse files

Cleanup of Test Network service 

This follow-up change performs some cleanup changes without affecting
functionality

Bug: 72950854
Test: Compiles, CTS tests using this pass
Change-Id: Ic7394f24f11d713c9374b438182e29d2a02ea236
parent 46a93ae8

core/java/android/app/SystemServiceRegistry.java

+1 −1
Original line number Diff line number Diff line
@@ -324,7 +324,7 @@ final class SystemServiceRegistry { 
                            throw new ServiceNotFoundException(Context.TEST_NETWORK_SERVICE);

                        }

                        ITestNetworkManager tnMgr = ITestNetworkManager.Stub.asInterface(tnBinder);

                        return new TestNetworkManager(context, tnMgr);

                        return new TestNetworkManager(tnMgr);

                    }

                });

core/java/android/net/TestNetworkInterface.java

+0 −2
Original line number Diff line number Diff line
@@ -27,8 +27,6 @@ import android.os.Parcelable; 
 */

@TestApi

public final class TestNetworkInterface implements Parcelable {

    private static final String TAG = "TestNetworkInterface";



    private final ParcelFileDescriptor mFileDescriptor;

    private final String mInterfaceName;

core/java/android/net/TestNetworkManager.java

+1 −4
Original line number Diff line number Diff line
@@ -17,7 +17,6 @@ package android.net; 


import android.annotation.NonNull;

import android.annotation.TestApi;

import android.content.Context;

import android.os.IBinder;

import android.os.RemoteException;
@@ -33,11 +32,9 @@ public class TestNetworkManager { 
    @NonNull private static final String TAG = TestNetworkManager.class.getSimpleName();



    @NonNull private final ITestNetworkManager mService;

    @NonNull private final Context mContext;



    /** @hide */

    public TestNetworkManager(@NonNull Context context, @NonNull ITestNetworkManager service) {

        mContext = Preconditions.checkNotNull(context, "missing Context");

    public TestNetworkManager(@NonNull ITestNetworkManager service) {

        mService = Preconditions.checkNotNull(service, "missing ITestNetworkManager");

    }

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

+1 −1
Original line number Diff line number Diff line
@@ -310,7 +310,7 @@ class TestNetworkService extends ITestNetworkManager.Stub { 
    public void teardownTestNetwork(int netId) {

        enforceTestNetworkPermissions(mContext);



        TestNetworkAgent agent;

        final TestNetworkAgent agent;

        synchronized (mTestNetworkTracker) {

            agent = mTestNetworkTracker.get(netId);

        }