Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Move the implementation of ProxyErrorListener to libpac

Browse Source 
The implementation logs the error in Android.

libpac will provide C API, not C++ API. This change
prepares for the migration.

Bug: 121269980
Test: m droid
Change-Id: Ib39e76d1880c33656d5ddbffdc3c124e7e911c16
This commit is contained in:
Victor Chang
2019-01-02 19:52:20 +00:00
parent d274332a9d
commit 30428c4d53
1 changed files with 1 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
packages/services/PacProcessor/jni/com_android_pacprocessor_PacNative.cpp
View File

@@ -27,23 +27,7 @@
namespace android {
class ProxyErrorLogger : public net::ProxyErrorListener {
public:
~ProxyErrorLogger() {
}
void AlertMessage(String16 message) {
String8 str(message);
ALOGD("Alert: %s", str.string());
}
void ErrorMessage(String16 message) {
String8 str(message);
ALOGE("Error: %s", str.string());
}
};
net::ProxyResolverV8* proxyResolver = NULL;
ProxyErrorLogger* logger = NULL;
bool pacSet = false;
String16 jstringToString16(JNIEnv* env, jstring jstr) {
@@ -64,9 +48,7 @@ jstring string16ToJstring(JNIEnv* env, String16 string) {
static jboolean com_android_pacprocessor_PacNative_createV8ParserNativeLocked(JNIEnv* /* env */,
jobject) {
if (proxyResolver == NULL) {
logger = new ProxyErrorLogger();
proxyResolver = new net::ProxyResolverV8(net::ProxyResolverJSBindings::CreateDefault(),
logger);
proxyResolver = new net::ProxyResolverV8(net::ProxyResolverJSBindings::CreateDefault());
pacSet = false;
return JNI_FALSE;
}
@@ -76,9 +58,7 @@ static jboolean com_android_pacprocessor_PacNative_createV8ParserNativeLocked(JN
static jboolean com_android_pacprocessor_PacNative_destroyV8ParserNativeLocked(JNIEnv* /* env */,
jobject) {
if (proxyResolver != NULL) {
delete logger;
delete proxyResolver;
logger = NULL;
proxyResolver = NULL;
return JNI_FALSE;
}