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

Commit 60dfb880 authored by Lorenzo Colitti's avatar Lorenzo Colitti Committed by Gerrit Code Review
Browse files

Merge changes from topic "connectivity_jar_in_apex" 

* changes:
  Move service-connectivity to the tethering APEX
  Set setCurrentProxyScriptUrl as public
parents 518d9d8f dbbf7a3b

packages/Connectivity/service/Android.bp

+5 −19
Original line number Diff line number Diff line
@@ -14,8 +14,8 @@ 
// limitations under the License.

//



cc_defaults {

    name: "libservice-connectivity-defaults",

cc_library_shared {

    name: "libservice-connectivity",

    // TODO: build against the NDK (sdk_version: "30" for example)

    cflags: [

        "-Wall",
@@ -26,6 +26,7 @@ cc_defaults { 
    srcs: [

        "jni/com_android_server_TestNetworkService.cpp",

        "jni/com_android_server_connectivity_Vpn.cpp",

        "jni/onload.cpp",

    ],

    shared_libs: [

        "libbase",
@@ -35,27 +36,11 @@ cc_defaults { 
        // addresses, and remove dependency on libnetutils.

        "libnetutils",

    ],

}



cc_library_shared {

    name: "libservice-connectivity",

    defaults: ["libservice-connectivity-defaults"],

    srcs: [

        "jni/onload.cpp",

    ],

    apex_available: [

        // TODO: move this library to the tethering APEX and remove libservice-connectivity-static

        // "com.android.tethering",

        "com.android.tethering",

    ],

}



// Static library linked into libservices.core until libservice-connectivity can be loaded from

// the tethering APEX instead.

cc_library_static {

    name: "libservice-connectivity-static",

    defaults: ["libservice-connectivity-defaults"],

}



java_library {

    name: "service-connectivity",

    srcs: [
@@ -75,5 +60,6 @@ java_library { 
    ],

    apex_available: [

        "//apex_available:platform",

        "com.android.tethering",

    ],

}

services/Android.bp

+0 −1
Original line number Diff line number Diff line
@@ -83,7 +83,6 @@ java_library { 
        "services.voiceinteraction",

        "services.wifi",

        "service-blobstore",

        "service-connectivity",

        "service-jobscheduler",

        "android.hidl.base-V1.0-java",

    ],

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

+2 −0
Original line number Diff line number Diff line
@@ -35,6 +35,8 @@ public final class ConnectivityServiceInitializer extends SystemService { 


    public ConnectivityServiceInitializer(Context context) {

        super(context);

        // Load JNI libraries used by ConnectivityService and its dependencies

        System.loadLibrary("service-connectivity");

        // TODO: Define formal APIs to get the needed services.

        mConnectivity = new ConnectivityService(context, getNetworkManagementService(),

                getNetworkStatsService());

services/core/java/com/android/server/connectivity/IpConnectivityMetrics.java

+2 −0
Original line number Diff line number Diff line
@@ -122,6 +122,8 @@ final public class IpConnectivityMetrics extends SystemService { 


    public IpConnectivityMetrics(Context ctx, ToIntFunction<Context> capacityGetter) {

        super(ctx);

        // Load JNI libraries used by the IpConnectivityMetrics service and its dependencies

        System.loadLibrary("service-connectivity");

        mCapacityGetter = capacityGetter;

        initBuffer();

    }

services/core/java/com/android/server/connectivity/PacManager.java

+1 −1
Original line number Diff line number Diff line
@@ -177,7 +177,7 @@ public class PacManager { 
     * @param proxy Proxy information that is about to be broadcast.

     * @return Returns whether the broadcast should be sent : either DO_ or DONT_SEND_BROADCAST

     */

    synchronized boolean setCurrentProxyScriptUrl(ProxyInfo proxy) {

    public synchronized boolean setCurrentProxyScriptUrl(ProxyInfo proxy) {

        if (!Uri.EMPTY.equals(proxy.getPacFileUrl())) {

            if (proxy.getPacFileUrl().equals(mPacUrl) && (proxy.getPort() > 0)) {

                // Allow to send broadcast, nothing to do.