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

Unified Diff: net/android/java/src/org/chromium/net/X509Util.java

Issue 1002883002: Fix up if statement curly braces issues. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 5 years, 9 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
Index: net/android/java/src/org/chromium/net/X509Util.java
diff --git a/net/android/java/src/org/chromium/net/X509Util.java b/net/android/java/src/org/chromium/net/X509Util.java
index 08ead31c640f4fccb0f4f8a5176180503b3dbef3..9c43a914e5d61d060cd7ae189c50ad1f229382e7 100644
--- a/net/android/java/src/org/chromium/net/X509Util.java
+++ b/net/android/java/src/org/chromium/net/X509Util.java
@@ -203,8 +203,9 @@ public class X509Util {
// Could not load AndroidCAStore. Continue anyway; isKnownRoot will always
// return false.
}
- if (!sDisableNativeCodeForTest)
+ if (!sDisableNativeCodeForTest) {
nativeRecordCertVerifyCapabilitiesHistogram(sSystemKeyStore != null);
+ }
sLoadedSystemKeyStore = true;
}
if (sSystemTrustAnchorCache == null) {
@@ -461,8 +462,9 @@ public class X509Util {
synchronized (sLock) {
// If no trust manager was found, fail without crashing on the null pointer.
- if (sDefaultTrustManager == null)
+ if (sDefaultTrustManager == null) {
return new AndroidCertVerifyResult(CertVerifyStatusAndroid.FAILED);
+ }
List<X509Certificate> verifiedChain;
try {

Powered by Google App Engine
This is Rietveld 408576698