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

Unified Diff: android_webview/javatests/src/org/chromium/android_webview/test/TestAwContentsClient.java

Issue 794023002: Remember user's decisions on SSL errors. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: comments Created 6 years 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
Index: android_webview/javatests/src/org/chromium/android_webview/test/TestAwContentsClient.java
diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/TestAwContentsClient.java b/android_webview/javatests/src/org/chromium/android_webview/test/TestAwContentsClient.java
index c9ee800cb311f3803464362089d2bbbd7866b974..a2b4b259b6864a715fe7c2978bc21ccdac27ee9a 100644
--- a/android_webview/javatests/src/org/chromium/android_webview/test/TestAwContentsClient.java
+++ b/android_webview/javatests/src/org/chromium/android_webview/test/TestAwContentsClient.java
@@ -5,7 +5,9 @@
package org.chromium.android_webview.test;
import android.graphics.Picture;
+import android.net.http.SslError;
import android.webkit.ConsoleMessage;
+import android.webkit.ValueCallback;
import org.chromium.base.ThreadUtils;
import org.chromium.content.browser.test.util.CallbackHelper;
@@ -19,9 +21,11 @@ import org.chromium.content.browser.test.util.TestCallbackHelperContainer.OnRece
*/
public class TestAwContentsClient extends NullContentsClient {
private String mUpdatedTitle;
+ private boolean mAllowSslError;
private final OnPageStartedHelper mOnPageStartedHelper;
private final OnPageFinishedHelper mOnPageFinishedHelper;
private final OnReceivedErrorHelper mOnReceivedErrorHelper;
+ private final OnReceivedSslErrorHelper mOnReceivedSslErrorHelper;
sgurun-gerrit only 2014/12/12 04:23:10 CallbackHelper
hush (inactive) 2014/12/12 19:43:40 Done.
private final OnDownloadStartHelper mOnDownloadStartHelper;
private final OnReceivedLoginRequestHelper mOnReceivedLoginRequestHelper;
private final OnEvaluateJavaScriptResultHelper mOnEvaluateJavaScriptResultHelper;
@@ -37,6 +41,7 @@ public class TestAwContentsClient extends NullContentsClient {
mOnPageStartedHelper = new OnPageStartedHelper();
mOnPageFinishedHelper = new OnPageFinishedHelper();
mOnReceivedErrorHelper = new OnReceivedErrorHelper();
+ mOnReceivedSslErrorHelper = new OnReceivedSslErrorHelper();
mOnDownloadStartHelper = new OnDownloadStartHelper();
mOnReceivedLoginRequestHelper = new OnReceivedLoginRequestHelper();
mOnEvaluateJavaScriptResultHelper = new OnEvaluateJavaScriptResultHelper();
@@ -46,6 +51,7 @@ public class TestAwContentsClient extends NullContentsClient {
mShouldOverrideUrlLoadingHelper = new ShouldOverrideUrlLoadingHelper();
mDoUpdateVisitedHistoryHelper = new DoUpdateVisitedHistoryHelper();
mOnCreateWindowHelper = new OnCreateWindowHelper();
+ mAllowSslError = true;
}
public OnPageStartedHelper getOnPageStartedHelper() {
@@ -60,6 +66,15 @@ public class TestAwContentsClient extends NullContentsClient {
return mOnReceivedErrorHelper;
}
+ /**
+ * Callback helper for onReceivedSslError.
+ */
+ public static class OnReceivedSslErrorHelper extends CallbackHelper {}
sgurun-gerrit only 2014/12/12 04:23:10 drop this and simply use callbackhelper.
hush (inactive) 2014/12/12 19:43:40 Done.
+
+ public OnReceivedSslErrorHelper getOnReceivedSslErrorHelper() {
+ return mOnReceivedSslErrorHelper;
+ }
+
public OnDownloadStartHelper getOnDownloadStartHelper() {
return mOnDownloadStartHelper;
}
@@ -146,6 +161,16 @@ public class TestAwContentsClient extends NullContentsClient {
mOnReceivedErrorHelper.notifyCalled(errorCode, description, failingUrl);
}
+ @Override
+ public void onReceivedSslError(ValueCallback<Boolean> callback, SslError error) {
+ callback.onReceiveValue(mAllowSslError);
+ mOnReceivedSslErrorHelper.notifyCalled();
+ }
+
+ public void setAllowSslError(boolean allow) {
+ mAllowSslError = allow;
+ }
+
/**
* CallbackHelper for OnDownloadStart.
*/
@@ -282,7 +307,7 @@ public class TestAwContentsClient extends NullContentsClient {
}
/**
- * Callback helper for onScaleChangedScaled.
+ * Callback helper for AddMessageToConsole.
*/
public static class AddMessageToConsoleHelper extends CallbackHelper {
private int mLevel;
@@ -325,7 +350,7 @@ public class TestAwContentsClient extends NullContentsClient {
}
/**
- * Callback helper for onScaleChangedScaled.
+ * Callback helper for PictureListener.
*/
public static class PictureListenerHelper extends CallbackHelper {
// Generally null, depending on |invalidationOnly| in enableOnNewPicture()
@@ -348,7 +373,7 @@ public class TestAwContentsClient extends NullContentsClient {
}
/**
- * Callback helper for onScaleChangedScaled.
+ * Callback helper for ShouldOverrideUrlLoading.
*/
public static class ShouldOverrideUrlLoadingHelper extends CallbackHelper {
private String mShouldOverrideUrlLoadingUrl;

Powered by Google App Engine
This is Rietveld 408576698