WKWebView: Implemented recoverable SSL interstitials.

ios/web/net/crw_cert_verification_controller.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef IOS_WEB_NET_CRW_CERT_VERIFICATION_CONTROLLER_H_
 #define IOS_WEB_NET_CRW_CERT_VERIFICATION_CONTROLLER_H_
 
 #import <Foundation/Foundation.h>
 
 #include "base/mac/scoped_cftyperef.h"
 #import "base/memory/ref_counted.h"
 #include "ios/web/public/security_style.h"
 #include "net/cert/cert_status_flags.h"
 
 namespace net {
 class X509Certificate;
 }
 
 namespace web {
 
 class BrowserState;
 
 // Accept policy for valid or invalid SSL cert.
 typedef NS_ENUM(NSInteger, CertAcceptPolicy) {
   // Cert status can't be determined due to an error. Caller should not proceed
   // with the load, but show net error page instead.
   CERT_ACCEPT_POLICY_NON_RECOVERABLE_ERROR = 0,
   // Cert is not valid. Caller may present SSL warning and ask user if they
   // want to proceed with the load.
-  CERT_ACCEPT_POLICY_RECOVERABLE_ERROR,
+  CERT_ACCEPT_POLICY_RECOVERABLE_ERROR_NOT_ACCEPTED_BY_USER,

[jww, 2015/10/17 20:47:19]

Is this comment still valid? Isn't it actually something more like:
// Cert is not valid. However caller should reject the load because
// user has decided not to proceed with this invalid cert.

and if that's not accurate, than you might consider renaming the variable to
something like "CERT_ACCEPT_POLICY_RECOVERABLE_ERROR_UNDECIDED_BY_USER"

[Eugene But (OOO till 7-30), 2015/10/17 22:43:55]

The original comment was correct (cert is indeed invalid, but error is
recoverable, so load decision will be up to the user). I changed enum name to
CERT_ACCEPT_POLICY_RECOVERABLE_ERROR_UNDECIDED_BY_USER.

+  // Cert is not valid. However caller should proceed with the load because
+  // user has decided to proceed with this invalid cert.
+  CERT_ACCEPT_POLICY_RECOVERABLE_ERROR_ACCEPTED_BY_USER,
   // Cert is valid. Caller should proceed with the load.
   CERT_ACCEPT_POLICY_ALLOW,
 };
 
-// Completion handler called by decidePolicyForCert:host:completionHandler:.
+// Completion handler called by decideLoadPolicyForTrust:host:completionHandler.
 typedef void (^PolicyDecisionHandler)(web::CertAcceptPolicy, net::CertStatus);
 // Completion handler called by querySSLStatusForTrust:host:completionHandler:.
 typedef void (^StatusQueryHandler)(web::SecurityStyle, net::CertStatus);
 
 }  // namespace web
 
 // Provides various cert verification API that can be used for blocking requests
 // with bad SSL cert, presenting SSL interstitials and determining SSL status
 // for Navigation Items. Must be used on UI thread.
 @interface CRWCertVerificationController : NSObject
 
 - (instancetype)init NS_UNAVAILABLE;
 
 // Initializes CRWCertVerificationController with the given |browserState| which
 // cannot be null and must outlive CRWCertVerificationController.
 - (instancetype)initWithBrowserState:(web::BrowserState*)browserState
     NS_DESIGNATED_INITIALIZER;
 
-// TODO(eugenebut): add API for:
-// - accepting bad SSL cert using CertPolicyCache
-
-// Decides the policy for the given |cert| for the given |host| and calls
+// Decides the policy for the given |serverTrust| for the given |host| and calls
 // |completionHandler| on completion. |host| should be in ASCII compatible form
 // (e.g. for "http://名がドメイン.com", it should be "xn--v8jxj3d1dzdz08w.com").
 // |completionHandler| cannot be null and will be called asynchronously on the
 // UI thread.
-- (void)decidePolicyForCert:(const scoped_refptr<net::X509Certificate>&)cert
-                       host:(NSString*)host
-          completionHandler:(web::PolicyDecisionHandler)completionHandler;
+- (void)decideLoadPolicyForTrust:(base::ScopedCFTypeRef<SecTrustRef>)serverTrust
+                            host:(NSString*)host
+               completionHandler:(web::PolicyDecisionHandler)completionHandler;
 
 // Asynchronously provides web::SecurityStyle and net::CertStatus for the given
 // |serverTrust| and |host|. |host| should be in ASCII compatible form.
 // Note: The web::SecurityStyle determines whether the certificate is trusted.
 // It is possible for an untrusted certificate to return a net::CertStatus with
 // no errors if the cause could not be determined. Callers must handle this case
 // gracefully.
 - (void)querySSLStatusForTrust:(base::ScopedCFTypeRef<SecTrustRef>)serverTrust
                           host:(NSString*)host
              completionHandler:(web::StatusQueryHandler)completionHandler;
 
+// Records that |cert| is permitted to be used for |host| in the future.
+// |host| should be in ASCII compatible form. Next time when
+// |decideLoadPolicyForTrust:| is called for the same server cert it will return
+// CERT_ACCEPT_POLICY_RECOVERABLE_ERROR_ACCEPTED_BY_USER if cert error is
+// recoverable.
+- (void)allowCert:(scoped_refptr<net::X509Certificate>)cert
+          forHost:(NSString*)host
+           status:(net::CertStatus)status;
+
 // Cancels all pending verification requests. Completion handlers will not be
 // called after |shutDown| call. Must always be called before object's
 // deallocation.
 - (void)shutDown;
 
 @end
 
 #endif  // IOS_WEB_NET_CRW_CERT_VERIFICATION_CONTROLLER_H_