diff --git a/core/java/android/net/http/SslError.java b/core/java/android/net/http/SslError.java index 5998f5f9ed6e7..863304c62b2a8 100644 --- a/core/java/android/net/http/SslError.java +++ b/core/java/android/net/http/SslError.java @@ -198,7 +198,8 @@ public class SslError { /** * Gets the most severe SSL error in this object's set of errors. - * @return The most severe SSL error. + * Returns -1 if the set is empty. + * @return The most severe SSL error, or -1 if the set is empty. */ public int getPrimaryError() { if (mErrors != 0) { @@ -208,9 +209,11 @@ public class SslError { return error; } } + // mErrors should never be set to an invalid value. + assert false; } - return 0; + return -1; } /** diff --git a/core/java/android/webkit/SslErrorHandlerImpl.java b/core/java/android/webkit/SslErrorHandlerImpl.java index 82cd3e80448ab..b2e4b131ffc68 100644 --- a/core/java/android/webkit/SslErrorHandlerImpl.java +++ b/core/java/android/webkit/SslErrorHandlerImpl.java @@ -159,7 +159,7 @@ class SslErrorHandlerImpl extends SslErrorHandler { if (DebugFlags.SSL_ERROR_HANDLER) { assert host != null; - assert primary != 0; + assert primary != -1; } if (mSslPrefTable.containsKey(host) && primary <= mSslPrefTable.getInt(host)) { @@ -260,7 +260,7 @@ class SslErrorHandlerImpl extends SslErrorHandler { if (DebugFlags.SSL_ERROR_HANDLER) { assert host != null; - assert primary != 0; + assert primary != -1; } boolean hasKey = mSslPrefTable.containsKey(host); if (!hasKey || primary > mSslPrefTable.getInt(host)) {