New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 884047 link

Starred by 1 user

Issue metadata

Status: Started
Owner:
Last visit > 30 days ago
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 2
Type: Feature



Sign in to add a comment

Implement handling of bad SSL page in //ios/web_view

Project Member Reported by ichikawa@chromium.org, Sep 14

Issue description

Add a new callback to CWVNavigationDelegate. Something like didFailNavigationWithSSLError:(NSError*)error decisionHandler:((^)(BOOL allow load))decisionHandler.

Call it from WebViewWebClient::AllowCertificateError().
 
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 17

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b7e2941bd81ea51bb003486cd70e5b3c509deaf0

commit b7e2941bd81ea51bb003486cd70e5b3c509deaf0
Author: Hiroshi Ichikawa <ichikawa@chromium.org>
Date: Wed Oct 17 02:31:10 2018

Add CWVSSLStatus class.

This will be used (in other CLs) in:
- a property of CWVWebView to report SSL status of the current page
- a method of CWVNavigationDelegate to report SSL error of a page.

Bug: 786891 884047
Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs
Change-Id: I63c37429e4d1dbd92333778b14504947948b1d62
Reviewed-on: https://chromium-review.googlesource.com/c/1272917
Commit-Queue: Hiroshi Ichikawa <ichikawa@chromium.org>
Reviewed-by: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#600255}
[modify] https://crrev.com/b7e2941bd81ea51bb003486cd70e5b3c509deaf0/ios/web_view/BUILD.gn
[add] https://crrev.com/b7e2941bd81ea51bb003486cd70e5b3c509deaf0/ios/web_view/internal/cwv_ssl_status.mm
[add] https://crrev.com/b7e2941bd81ea51bb003486cd70e5b3c509deaf0/ios/web_view/internal/cwv_ssl_status_internal.h
[add] https://crrev.com/b7e2941bd81ea51bb003486cd70e5b3c509deaf0/ios/web_view/internal/cwv_ssl_status_unittest.mm
[add] https://crrev.com/b7e2941bd81ea51bb003486cd70e5b3c509deaf0/ios/web_view/public/cwv_ssl_status.h

Status: Started (was: Assigned)

Sign in to add a comment