New issue
Advanced search Search tips

Issue 921916 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 3
Type: Bug



Sign in to add a comment

DidFinishNavigation is not called for SSL error navigations

Project Member Reported by eugene...@chromium.org, Jan 15

Issue description

Steps to reproduce: 
Load https://expired.badssl.com/

Observed behavior: 
Only WebStateObserver::DidStartNavigation is called

Expected behavior: 
Both WebStateObserver::DidStartNavigation and WebStateObserver::DidFinishNavigation should be called.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 16 (6 days ago)

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

commit 0eec38e65d3388d78196d4f4d5c4b480131c802f
Author: Eugene But <eugenebut@chromium.org>
Date: Wed Jan 16 21:09:14 2019

Add WebStateObserverTest.FailedSslConnection integration test.

The test sets up a local HTTPS server and loads a page. This results in
navigation failure because System Certificate Store does not recognize
self-signed certificate. The test verifies DidStartLoading,
DidStartNavigation and DidStopLoading WebStateObserver callbacks.
DidFinishNavigation is not called because of a bug.

Bug: 921916
Change-Id: Ib7efc8cf7af16ca47c29ab5e757399d6ac6f0741
Reviewed-on: https://chromium-review.googlesource.com/c/1411213
Reviewed-by: Kurt Horimoto <kkhorimoto@chromium.org>
Commit-Queue: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#623367}
[modify] https://crrev.com/0eec38e65d3388d78196d4f4d5c4b480131c802f/ios/web/web_state/web_state_observer_inttest.mm

Sign in to add a comment