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

Merge "Use synchronize methods instead of synchronized blocks in PacWebView."

Browse Source
This commit is contained in:
Anna Malova
2020-02-13 08:37:54 +00:00
committed by Android (Google) Code Review
parent 98e89b627c 23782a50ab
commit f8bd2c5bdf
1 changed files with 7 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
packages/services/PacProcessor/src/com/android/pacprocessor/PacWebView.java
View File

@@ -19,8 +19,6 @@ package com.android.pacprocessor;
import android.util.Log;
import android.webkit.PacProcessor;
import com.android.internal.annotations.GuardedBy;
/**
* @hide
*/
@@ -28,10 +26,6 @@ public class PacWebView implements LibpacInterface {
private static final String TAG = "PacWebView";
private static final PacWebView sInstance = new PacWebView();
private Object mLock = new Object();
@GuardedBy("mLock")
private PacProcessor mProcessor = PacProcessor.getInstance();
public static PacWebView getInstance() {
@@ -39,20 +33,16 @@ public class PacWebView implements LibpacInterface {
}
@Override
public boolean setCurrentProxyScript(String script) {
synchronized (mLock) {
if (!mProcessor.setProxyScript(script)) {
Log.e(TAG, "Unable to parse proxy script.");
return false;
}
return true;
public synchronized boolean setCurrentProxyScript(String script) {
if (!mProcessor.setProxyScript(script)) {
Log.e(TAG, "Unable to parse proxy script.");
return false;
}
return true;
}
@Override
public String makeProxyRequest(String url, String host) {
synchronized (mLock) {
return mProcessor.findProxyForUrl(url);
}
public synchronized String makeProxyRequest(String url, String host) {
return mProcessor.findProxyForUrl(url);
}
}