From 26dbdc853b3cb754ca28045989c4f76e35820ba8 Mon Sep 17 00:00:00 2001 From: Remi NGUYEN VAN Date: Thu, 7 Jan 2021 10:47:11 +0900 Subject: [PATCH] Set setCurrentProxyScriptUrl as public PacManager is part of services.jar, while ProxyManager is in service-connectivity.jar. setCurrentProxyScriptUrl cannot be package-private and called across jars. As part of modularization of service-connectivity.jar, the PacManager interface will then be formally defined. Bug: 171540887 Test: Splitting service-connectivity, ProxyManager can reference the method Change-Id: Ie41a5b569a119f305bf647e55a65febbde459352 --- .../core/java/com/android/server/connectivity/PacManager.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/services/core/java/com/android/server/connectivity/PacManager.java b/services/core/java/com/android/server/connectivity/PacManager.java index 06721aea5540c..93930ae16de89 100644 --- a/services/core/java/com/android/server/connectivity/PacManager.java +++ b/services/core/java/com/android/server/connectivity/PacManager.java @@ -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.