Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1988)

Unified Diff: android_webview/glue/java/src/com/android/webview/chromium/WebViewContentsClientAdapter.java

Issue 876413007: [Android WebView] Rename ShouldInterceptRequestParams -> AwWebResourceRequest (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed glue code Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | android_webview/java/src/org/chromium/android_webview/AwContents.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: android_webview/glue/java/src/com/android/webview/chromium/WebViewContentsClientAdapter.java
diff --git a/android_webview/glue/java/src/com/android/webview/chromium/WebViewContentsClientAdapter.java b/android_webview/glue/java/src/com/android/webview/chromium/WebViewContentsClientAdapter.java
index 332bf402ffd11b371bb31ad915cf0049ac89c792..6f15bae961d0a0ab118950756f8207df7482ff49 100644
--- a/android_webview/glue/java/src/com/android/webview/chromium/WebViewContentsClientAdapter.java
+++ b/android_webview/glue/java/src/com/android/webview/chromium/WebViewContentsClientAdapter.java
@@ -285,35 +285,35 @@ public class WebViewContentsClientAdapter extends AwContentsClient {
}
private static class WebResourceRequestImpl implements WebResourceRequest {
- private final ShouldInterceptRequestParams mParams;
+ private final AwWebResourceRequest mRequest;
- public WebResourceRequestImpl(ShouldInterceptRequestParams params) {
- mParams = params;
+ public WebResourceRequestImpl(AwWebResourceRequest request) {
+ mRequest = request;
}
@Override
public Uri getUrl() {
- return Uri.parse(mParams.url);
+ return Uri.parse(mRequest.url);
}
@Override
public boolean isForMainFrame() {
- return mParams.isMainFrame;
+ return mRequest.isMainFrame;
}
@Override
public boolean hasGesture() {
- return mParams.hasUserGesture;
+ return mRequest.hasUserGesture;
}
@Override
public String getMethod() {
- return mParams.method;
+ return mRequest.method;
}
@Override
public Map<String, String> getRequestHeaders() {
- return mParams.requestHeaders;
+ return mRequest.requestHeaders;
}
}
@@ -321,12 +321,12 @@ public class WebViewContentsClientAdapter extends AwContentsClient {
* @see AwContentsClient#shouldInterceptRequest(java.lang.String)
*/
@Override
- public AwWebResourceResponse shouldInterceptRequest(ShouldInterceptRequestParams params) {
+ public AwWebResourceResponse shouldInterceptRequest(AwWebResourceRequest request) {
try {
TraceEvent.begin("WebViewContentsClientAdapter.shouldInterceptRequest");
- if (TRACE) Log.d(TAG, "shouldInterceptRequest=" + params.url);
+ if (TRACE) Log.d(TAG, "shouldInterceptRequest=" + request.url);
WebResourceResponse response = mWebViewClient.shouldInterceptRequest(mWebView,
- new WebResourceRequestImpl(params));
+ new WebResourceRequestImpl(request));
if (response == null) return null;
// AwWebResourceResponse should support null headers. b/16332774.
« no previous file with comments | « no previous file | android_webview/java/src/org/chromium/android_webview/AwContents.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698